update mock function

Author: janishar

tests/auth/authentication/mock.ts

@@ -39,15 +39,11 @@ export const mockKeystoreFindForKey = jest.fn(
 );
 
 jest.mock('../../../src/database/repository/UserRepo', () => ({
-  get findById() {
-    return mockUserFindById;
-  },
+  findById: mockUserFindById,
 }));
 
 jest.mock('../../../src/database/repository/KeystoreRepo', () => ({
-  get findforKey() {
-    return mockKeystoreFindForKey;
-  },
+  findforKey: mockKeystoreFindForKey,
 }));
 
 JWT.validate = mockJwtValidate;

tests/auth/authorization/mock.ts

@@ -68,7 +68,7 @@ export const mockRoleRepoFindByCode = jest.fn(async (code: string): Promise<Role
 });
 
 export const mockJwtValidate = jest.fn(async (token: string): Promise<JwtPayload> => {
-  let subject = null;
+  let subject: null | string = null;
   switch (token) {
     case ACCESS_TOKEN:
       subject = USER_ID.toHexString();
@@ -93,15 +93,11 @@ export const mockJwtValidate = jest.fn(async (token: string): Promise<JwtPayload
 });
 
 jest.mock('../../../src/database/repository/UserRepo', () => ({
-  get findById() {
-    return mockUserFindById;
-  },
+  findById: mockUserFindById,
 }));
 
 jest.mock('../../../src/database/repository/RoleRepo', () => ({
-  get findByCode() {
-    return mockRoleRepoFindByCode;
-  },
+  findByCode: mockRoleRepoFindByCode,
 }));
 
 JWT.validate = mockJwtValidate;

tests/routes/v1/blog/index/mock.ts

@@ -27,10 +27,6 @@ export const mockFindInfoWithTextById = jest.fn(
 );
 
 jest.mock('../../../../../src/database/repository/BlogRepo', () => ({
-  get findByUrl() {
-    return mockBlogFindByUrl;
-  },
-  get findInfoForPublishedById() {
-    return mockFindInfoWithTextById;
-  },
+  findByUrl: mockBlogFindByUrl,
+  findInfoForPublishedById: mockFindInfoWithTextById,
 }));

tests/routes/v1/blog/writer/mock.ts

@@ -45,16 +45,8 @@ export const mockFindBlogAllDataById = jest.fn(async (id: Types.ObjectId): Promi
 });
 
 jest.mock('../../../../../src/database/repository/BlogRepo', () => ({
-  get findUrlIfExists() {
-    return mockBlogFindUrlIfExists;
-  },
-  get create() {
-    return mockBlogCreate;
-  },
-  get update() {
-    return mockBlogUpdate;
-  },
-  get findBlogAllDataById() {
-    return mockFindBlogAllDataById;
-  },
+  findUrlIfExists: mockBlogFindUrlIfExists,
+  create: mockBlogCreate,
+  update: mockBlogUpdate,
+  findBlogAllDataById: mockFindBlogAllDataById,
 }));

tests/routes/v1/login/mock.ts

@@ -39,15 +39,11 @@ export const mockUserFindByEmail = jest.fn(async (email: string): Promise<User |
 });
 
 jest.mock('../../../../src/database/repository/KeystoreRepo', () => ({
-  get create() {
-    return mockKeystoreCreate;
-  },
+  create: mockKeystoreCreate,
 }));
 
 jest.mock('../../../../src/database/repository/UserRepo', () => ({
-  get findByEmail() {
-    return mockUserFindByEmail;
-  },
+  findByEmail: mockUserFindByEmail,
 }));
 
 jest.unmock('../../../../src/auth/authUtils'); // remove any override made anywhere

tests/routes/v1/signup/mock.ts

@@ -32,10 +32,6 @@ export const mockUserCreate = jest.fn(
 );
 
 jest.mock('../../../../src/database/repository/UserRepo', () => ({
-  get findByEmail() {
-    return mockUserFindByEmail;
-  }, // utilising already defined mock
-  get create() {
-    return mockUserCreate;
-  },
+  findByEmail: mockUserFindByEmail, // utilising already defined mock
+  create: mockUserCreate,
 }));