test: fix tests for mongoose v6 & v5

Author: nodkz

src/__tests__/__snapshots__/integration-test.ts.snap

@@ -2,24 +2,24 @@
 
 exports[`integration tests projection should request all fields to rawData field: projection from all fields 1`] = `
 Array [
-  "someDeep",
-  "users",
-  "skills",
-  "employment",
-  "contacts",
+  "__v",
   "_id",
-  "n",
   "age",
+  "contacts",
+  "createdAt",
+  "employment",
   "gender",
-  "relocation",
+  "id",
   "languages",
-  "periods",
-  "createdAt",
-  "updatedAt",
-  "__v",
+  "n",
   "name",
   "nameVirtual",
-  "id",
+  "periods",
+  "relocation",
+  "skills",
+  "someDeep",
+  "updatedAt",
+  "users",
 ]
 `;
 

src/__tests__/github_issues/271-test.ts

@@ -99,8 +99,8 @@ describe('nested projections with aliases - issue #271', () => {
     expect(book?.author?.age).toEqual(115);
     expect(book?.toObject({ virtuals: true })).toEqual({
       _id: 1,
-      a: { ag: 115, age: 115, id: null },
-      author: { ag: 115, age: 115, id: null },
+      a: expect.objectContaining({ ag: 115, age: 115 }),
+      author: expect.objectContaining({ ag: 115, age: 115 }),
       id: '1',
       pageCount: 1168,
       pc: 1168,

src/__tests__/integration-test.ts

@@ -165,7 +165,9 @@ describe('integration tests', () => {
         schema,
         '{ user(_id: "100000000000000000000000") { rawData } }'
       );
-      expect(Object.keys(res.data.user.rawData)).toMatchSnapshot('projection from all fields');
+      expect(Object.keys(res.data.user.rawData).sort()).toMatchSnapshot(
+        'projection from all fields'
+      );
     });
   });
 });

src/resolvers/__tests__/createOne-test.ts

@@ -89,7 +89,7 @@ describe('createOne() ->', () => {
       });
 
       const doc = await UserModel.collection.findOne({ _id: res.record._id });
-      expect(doc.n).toBe(checkedName);
+      expect(doc?.n).toBe(checkedName);
     });
 
     it('should return payload.record when it requested', async () => {

src/resolvers/__tests__/dataLoader-test.ts

@@ -161,7 +161,7 @@ describe('dataLoader() ->', () => {
       ...resolveParams,
     });
 
-    const nonExistedId = mongoose.Types.ObjectId('5cefda4616156200084e5170');
+    const nonExistedId = new mongoose.Types.ObjectId('5cefda4616156200084e5170');
     const resultPromise5 = resolver.resolve({
       args: { _id: nonExistedId },
       ...resolveParams,

src/resolvers/__tests__/dataLoaderMany-test.ts

@@ -157,7 +157,7 @@ describe('dataLoaderMany() ->', () => {
       ...resolveParams,
     });
 
-    const nonExistedId = mongoose.Types.ObjectId('5cefda4616156200084e5170');
+    const nonExistedId = new mongoose.Types.ObjectId('5cefda4616156200084e5170');
     const resultPromise3 = resolver.resolve({
       args: { _ids: [user._id, nonExistedId] },
       ...resolveParams,

src/resolvers/__tests__/removeById-test.ts

@@ -164,7 +164,7 @@ describe('removeById() ->', () => {
       });
       await expect(result).rejects.toThrow('Denied due context ReadOnly');
       const exist = await UserModel.collection.findOne({ _id: user._id });
-      expect(exist.n).toBe(user.name);
+      expect(exist?.n).toBe(user.name);
     });
 
     it('should call `beforeQuery` method with non-executed `query` as arg', async () => {
@@ -180,14 +180,14 @@ describe('removeById() ->', () => {
         beforeQuery(query: any, rp: ExtendedResolveParams) {
           expect(rp.model).toBe(UserModel);
           expect(rp.query).toHaveProperty('exec');
-          return query.where({ _id: user._id, canDelete: false });
+          return query.where({ _id: user._id, gender: 'some' });
         },
       };
 
       const result = await removeById(UserModel, UserTC).resolve(resolveParams);
 
       expect(mongooseActions).toEqual([
-        ['users', 'findOne', { _id: user._id, canDelete: false }, { projection: {} }],
+        ['users', 'findOne', { _id: user._id, gender: 'some' }, { projection: {} }],
       ]);
 
       expect(result).toBeNull();

src/resolvers/__tests__/removeOne-test.ts

@@ -215,7 +215,7 @@ describe('removeOne() ->', () => {
       });
       await expect(result).rejects.toThrow('Denied due context ReadOnly');
       const exist = await UserModel.collection.findOne({ _id: user1._id });
-      expect(exist.n).toBe(user1.name);
+      expect(exist?.n).toBe(user1.name);
     });
 
     it('should call `beforeQuery` method with non-executed `query` as arg', async () => {

src/resolvers/__tests__/updateById-test.ts

@@ -240,7 +240,7 @@ describe('updateById() ->', () => {
       });
       await expect(result).rejects.toThrow('Denied due context ReadOnly');
       const exist = await UserModel.collection.findOne({ _id: user1._id });
-      expect(exist.n).toBe(user1.name);
+      expect(exist?.n).toBe(user1.name);
     });
 
     it('should call `beforeQuery` method with non-executed `query` as arg', async () => {

src/resolvers/__tests__/updateOne-test.ts

@@ -294,7 +294,7 @@ describe('updateOne() ->', () => {
       });
       await expect(result).rejects.toThrow('Denied due context ReadOnly');
       const exist = await UserModel.collection.findOne({ _id: user1._id });
-      expect(exist.n).toBe(user1.name);
+      expect(exist?.n).toBe(user1.name);
     });
 
     it('should call `beforeQuery` method with non-executed `query` as arg', async () => {