fix tests

Author: janishar

tests/auth/authorization/mock.ts

@@ -44,7 +44,7 @@ export const mockUserFindById = jest.fn(async (id: Types.ObjectId) => {
 });
 
 export const mockRoleRepoFindByCode = jest.fn(
-  async (code: string): Promise<Role> => {
+  async (code: string): Promise<Role | null> => {
     switch (code) {
       case RoleCode.WRITER:
         return {

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

@@ -7,7 +7,7 @@ export const BLOG_ID = new Types.ObjectId();
 export const BLOG_URL = 'abc';
 
 export const mockBlogFindByUrl = jest.fn(
-  async (blogUrl: string): Promise<Blog> => {
+  async (blogUrl: string): Promise<Blog | null> => {
     if (blogUrl === BLOG_URL)
       return {
         _id: BLOG_ID,
@@ -18,7 +18,7 @@ export const mockBlogFindByUrl = jest.fn(
 );
 
 export const mockFindInfoWithTextById = jest.fn(
-  async (id: Types.ObjectId): Promise<Blog> => {
+  async (id: Types.ObjectId): Promise<Blog | null> => {
     if (BLOG_ID.equals(id))
       return {
         _id: BLOG_ID,

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

@@ -9,7 +9,7 @@ export const BLOG_ID_2 = new Types.ObjectId();
 export const BLOG_URL = 'abc';
 
 export const mockBlogFindUrlIfExists = jest.fn(
-  async (blogUrl: string): Promise<Blog> => {
+  async (blogUrl: string): Promise<Blog | null> => {
     if (blogUrl === BLOG_URL)
       return {
         _id: BLOG_ID,
@@ -29,7 +29,7 @@ export const mockBlogCreate = jest.fn(
 export const mockBlogUpdate = jest.fn(async (blog: Blog): Promise<Blog> => blog);
 
 export const mockFindBlogAllDataById = jest.fn(
-  async (id: Types.ObjectId): Promise<Blog> => {
+  async (id: Types.ObjectId): Promise<Blog | null> => {
     if (BLOG_ID.equals(id))
       return {
         _id: BLOG_ID,

tests/routes/v1/login/integration.test.ts

@@ -22,7 +22,7 @@ describe('Login basic route', () => {
   const password = '123456';
 
   let user: User;
-  let apikey: ApiKey;
+  let apikey: ApiKey | null;
 
   beforeAll(async () => {
     await UserModel.remove({}); // delete all data from user table
@@ -168,5 +168,5 @@ describe('Login basic route', () => {
   });
 });
 
-export const addHeaders = (request: any, apikey: ApiKey) =>
-  request.set('Content-Type', 'application/json').set('x-api-key', apikey.key);
+export const addHeaders = (request: any, apikey: ApiKey | null) =>
+  request.set('Content-Type', 'application/json').set('x-api-key', apikey?.key);

tests/routes/v1/login/mock.ts

@@ -25,7 +25,7 @@ export const mockKeystoreCreate = jest.fn(
 );
 
 export const mockUserFindByEmail = jest.fn(
-  async (email: string): Promise<User> => {
+  async (email: string): Promise<User | null> => {
     if (email === USER_EMAIL)
       return {
         _id: USER_ID,

tests/routes/v1/login/unit.test.ts

@@ -112,10 +112,13 @@ describe('Login basic route', () => {
 
   it('Should send success response for correct credentials', async () => {
     const response = await addHeaders(
-      request.post(endpoint).send({
-        email: USER_EMAIL,
-        password: USER_PASSWORD,
-      }),
+      request
+        .post(endpoint)
+        .send({
+          email: USER_EMAIL,
+          password: USER_PASSWORD,
+        })
+        .timeout(5000),
     );
     expect(response.status).toBe(200);
     expect(response.body.message).toMatch(/Success/i);