Merge pull request #919 from nestjsx/feat/update-nestjs-version

feat: update nestjs version

Author: bashleigh

.github/workflows/main.yml

@@ -32,7 +32,7 @@ jobs:
 
       # Runs a single command using the runners shell
       - name: build docker db
-        run: docker-compose up -d
+        run: docker compose up -d
 
       - name: install
         run: yarn install --ignore-scripts
@@ -41,10 +41,10 @@ jobs:
         run: yarn build
 
       - name: check docker
-        run: docker-compose up -d
+        run: docker compose up -d
 
       - name: docker logs
-        run: docker-compose logs && docker-compose ps
+        run: docker compose logs && docker compose ps
 
       # Runs a set of commands using the runners shell
       - name: tests

package.json

@@ -18,19 +18,19 @@
     "paginate"
   ],
   "devDependencies": {
-    "@nestjs/common": "^9.0.5",
-    "@nestjs/core": "9.4.3",
-    "@nestjs/testing": "^9.0.5",
-    "@nestjs/typeorm": "^7.1.0",
+    "@nestjs/common": "^11.0.0",
+    "@nestjs/core": "^11.0.0",
+    "@nestjs/testing": "^11.0.0",
+    "@nestjs/typeorm": "^11.0.0",
     "@types/jest": "^27.0.0",
     "@types/node": "^20.9.0",
     "coveralls": "^3.0.5",
-    "jest": "^26.6.3",
+    "jest": "^29.7.0",
     "mysql": "^2.17.1",
     "prettier": "^3.1.0",
     "reflect-metadata": "^0.1.13",
-    "rxjs": "^6.5.2",
-    "ts-jest": "^26.4.4",
+    "rxjs": "^7.8.2",
+    "ts-jest": "^29.2.6",
     "ts-node": "^10.0.0",
     "typeorm": "0.3.17",
     "typescript": "^4.0.5"
@@ -46,7 +46,7 @@
     "prepublish": "npm run format && npm run build"
   },
   "peerDependencies": {
-    "@nestjs/common": "^6.1.1 || ^5.6.2 || ^7.0.0 || ^8.0.0 || ^9.0.0 || ^10.0.0",
+    "@nestjs/common": "^6.1.1 || ^5.6.2 || ^7.0.0 || ^8.0.0 || ^9.0.0 || ^10.0.0 || ^11.0.0",
     "typeorm": "^0.3.0"
   },
   "jest": {

src/__tests__/mocks.ts

@@ -6,7 +6,7 @@ import {
 } from 'typeorm';
 
 export class MockRepository extends Repository<any> {
-  items = [];
+  items: Entity[] = [];
   constructor(entityAmount: number) {
     super({} as EntityTarget<{}>, {} as EntityManager);
     for (let i = 0; i < entityAmount; i++) this.items.push(new Entity());
@@ -19,7 +19,7 @@ export class MockRepository extends Repository<any> {
   };
 
   find = async (options?: FindManyOptions<any>): Promise<any[]> => {
-    const startIndex = options.skip;
+    const startIndex = options?.skip;
     const endIndex = startIndex + options.take;
 
     const localItems = this.items.slice(startIndex, endIndex);

src/__tests__/paginate.repository.spec.ts

@@ -62,14 +62,14 @@ describe('Test paginate function', () => {
       route: 'http://example.com/something',
     });
 
-    expect(results.links.first).toBe('http://example.com/something?limit=4');
-    expect(results.links.previous).toBe(
+    expect(results.links?.first).toBe('http://example.com/something?limit=4');
+    expect(results.links?.previous).toBe(
       'http://example.com/something?page=1&limit=4',
     );
-    expect(results.links.next).toBe(
+    expect(results.links?.next).toBe(
       'http://example.com/something?page=3&limit=4',
     );
-    expect(results.links.last).toBe(
+    expect(results.links?.last).toBe(
       'http://example.com/something?page=3&limit=4',
     );
   });
@@ -87,16 +87,16 @@ describe('Test paginate function', () => {
       },
     });
 
-    expect(results.links.first).toBe(
+    expect(results.links?.first).toBe(
       'http://example.com/something?page-size=4',
     );
-    expect(results.links.previous).toBe(
+    expect(results.links?.previous).toBe(
       'http://example.com/something?current-page=1&page-size=4',
     );
-    expect(results.links.next).toBe(
+    expect(results.links?.next).toBe(
       'http://example.com/something?current-page=3&page-size=4',
     );
-    expect(results.links.last).toBe(
+    expect(results.links?.last).toBe(
       'http://example.com/something?current-page=3&page-size=4',
     );
   });
@@ -113,14 +113,14 @@ describe('Test paginate function', () => {
       },
     });
 
-    expect(results.links.first).toBe('http://example.com/something?limit=4');
-    expect(results.links.previous).toBe(
+    expect(results.links?.first).toBe('http://example.com/something?limit=4');
+    expect(results.links?.previous).toBe(
       'http://example.com/something?current-page=1&limit=4',
     );
-    expect(results.links.next).toBe(
+    expect(results.links?.next).toBe(
       'http://example.com/something?current-page=3&limit=4',
     );
-    expect(results.links.last).toBe(
+    expect(results.links?.last).toBe(
       'http://example.com/something?current-page=3&limit=4',
     );
   });
@@ -137,16 +137,16 @@ describe('Test paginate function', () => {
       },
     });
 
-    expect(results.links.first).toBe(
+    expect(results.links?.first).toBe(
       'http://example.com/something?page-size=4',
     );
-    expect(results.links.previous).toBe(
+    expect(results.links?.previous).toBe(
       'http://example.com/something?page=1&page-size=4',
     );
-    expect(results.links.next).toBe(
+    expect(results.links?.next).toBe(
       'http://example.com/something?page=3&page-size=4',
     );
-    expect(results.links.last).toBe(
+    expect(results.links?.last).toBe(
       'http://example.com/something?page=3&page-size=4',
     );
   });
@@ -160,12 +160,12 @@ describe('Test paginate function', () => {
       route: 'http://example.com/something',
     });
 
-    expect(results.links.first).toBe('http://example.com/something?limit=4');
-    expect(results.links.previous).toBe('');
-    expect(results.links.next).toBe(
+    expect(results.links?.first).toBe('http://example.com/something?limit=4');
+    expect(results.links?.previous).toBe('');
+    expect(results.links?.next).toBe(
       'http://example.com/something?page=2&limit=4',
     );
-    expect(results.links.last).toBe(
+    expect(results.links?.last).toBe(
       'http://example.com/something?page=3&limit=4',
     );
   });
@@ -179,12 +179,12 @@ describe('Test paginate function', () => {
       route: 'http://example.com/something',
     });
 
-    expect(results.links.first).toBe('http://example.com/something?limit=4');
-    expect(results.links.previous).toBe(
+    expect(results.links?.first).toBe('http://example.com/something?limit=4');
+    expect(results.links?.previous).toBe(
       'http://example.com/something?page=2&limit=4',
     );
-    expect(results.links.next).toBe('');
-    expect(results.links.last).toBe(
+    expect(results.links?.next).toBe('');
+    expect(results.links?.last).toBe(
       'http://example.com/something?page=3&limit=4',
     );
   });
@@ -198,12 +198,12 @@ describe('Test paginate function', () => {
       route: 'http://example.com/something',
     });
 
-    expect(results.links.first).toBe('http://example.com/something?limit=4');
-    expect(results.links.previous).toBe('');
-    expect(results.links.next).toBe(
+    expect(results.links?.first).toBe('http://example.com/something?limit=4');
+    expect(results.links?.previous).toBe('');
+    expect(results.links?.next).toBe(
       'http://example.com/something?page=2&limit=4',
     );
-    expect(results.links.last).toBe(
+    expect(results.links?.last).toBe(
       'http://example.com/something?page=3&limit=4',
     );
   });
@@ -222,12 +222,12 @@ describe('Test paginate function', () => {
     });
 
     expect(results.items.length).toBe(5);
-    expect(results.links.first).toBe('http://example.com/something?limit=10');
-    expect(results.links.previous).toBe(
+    expect(results.links?.first).toBe('http://example.com/something?limit=10');
+    expect(results.links?.previous).toBe(
       'http://example.com/something?page=1&limit=10',
     );
-    expect(results.links.next).toBe('');
-    expect(results.links.last).toBe(
+    expect(results.links?.next).toBe('');
+    expect(results.links?.last).toBe(
       'http://example.com/something?page=2&limit=10',
     );
     expect(consoleMock).toHaveBeenCalledWith(
@@ -249,12 +249,12 @@ describe('Test paginate function', () => {
     });
 
     expect(results.items.length).toBe(4);
-    expect(results.links.first).toBe('http://example.com/something?limit=4');
-    expect(results.links.previous).toBe('');
-    expect(results.links.next).toBe(
+    expect(results.links?.first).toBe('http://example.com/something?limit=4');
+    expect(results.links?.previous).toBe('');
+    expect(results.links?.next).toBe(
       'http://example.com/something?page=2&limit=4',
     );
-    expect(results.links.last).toBe(
+    expect(results.links?.last).toBe(
       'http://example.com/something?page=3&limit=4',
     );
     expect(consoleMock).toHaveBeenCalledWith(
@@ -276,12 +276,12 @@ describe('Test paginate function', () => {
     });
 
     expect(results.items.length).toBe(4);
-    expect(results.links.first).toBe('http://example.com/something?limit=4');
-    expect(results.links.previous).toBe('');
-    expect(results.links.next).toBe(
+    expect(results.links?.first).toBe('http://example.com/something?limit=4');
+    expect(results.links?.previous).toBe('');
+    expect(results.links?.next).toBe(
       'http://example.com/something?page=2&limit=4',
     );
-    expect(results.links.last).toBe(
+    expect(results.links?.last).toBe(
       'http://example.com/something?page=3&limit=4',
     );
     expect(consoleMock).toHaveBeenCalledWith(
@@ -303,12 +303,12 @@ describe('Test paginate function', () => {
     });
 
     expect(results.items.length).toBe(4);
-    expect(results.links.first).toBe('http://example.com/something?limit=4');
-    expect(results.links.previous).toBe('');
-    expect(results.links.next).toBe(
+    expect(results.links?.first).toBe('http://example.com/something?limit=4');
+    expect(results.links?.previous).toBe('');
+    expect(results.links?.next).toBe(
       'http://example.com/something?page=2&limit=4',
     );
-    expect(results.links.last).toBe(
+    expect(results.links?.last).toBe(
       'http://example.com/something?page=3&limit=4',
     );
     expect(consoleMock).toHaveBeenCalledWith(
@@ -375,7 +375,7 @@ describe('Test paginate function', () => {
       route: '/test?test=test',
     });
 
-    expect(results.links.next).toBe('/test?test=test&page=2&limit=4');
+    expect(results.links?.next).toBe('/test?test=test&page=2&limit=4');
   });
 
   it('when page is 0, return empty pagination object', async () => {
@@ -388,10 +388,10 @@ describe('Test paginate function', () => {
     });
 
     expect(results.items.length).toBe(0);
-    expect(results.links.first).toBe('/test?test=test&limit=4');
-    expect(results.links.previous).toBe('');
-    expect(results.links.next).toBe('');
-    expect(results.links.last).toBe('');
+    expect(results.links?.first).toBe('/test?test=test&limit=4');
+    expect(results.links?.previous).toBe('');
+    expect(results.links?.next).toBe('');
+    expect(results.links?.last).toBe('');
   });
 
   it('Can use skip and take', async () => {
@@ -405,10 +405,10 @@ describe('Test paginate function', () => {
     });
 
     expect(results.items.length).toBe(0);
-    expect(results.links.first).toBe('/test?test=test&limit=4');
-    expect(results.links.previous).toBe('');
-    expect(results.links.next).toBe('');
-    expect(results.links.last).toBe('');
+    expect(results.links?.first).toBe('/test?test=test&limit=4');
+    expect(results.links?.previous).toBe('');
+    expect(results.links?.next).toBe('');
+    expect(results.links?.last).toBe('');
   });
 
   it('Can call paginate with no count queries', async () => {