fix: rename controller function

Author: surajmaity1

controllers/oooRequests.ts

@@ -21,7 +21,8 @@ import { addLog } from "../models/logs";
 import { getRequestByKeyValues, getRequests, updateRequest } from "../models/requests";
 import { createUserFutureStatus } from "../models/userFutureStatus";
 import { getUserStatus, addFutureStatus } from "../models/userStatus";
-import { createOooRequest, validateUserStatus, acknowledgeOooRequest } from "../services/oooRequest";
+import { createOooRequest, validateUserStatus } from "../services/oooRequest";
+import * as oooRequestService from "../services/oooRequest";
 import { CustomResponse } from "../typeDefinitions/global";
 import { AcknowledgeOooRequest, OooRequestCreateRequest, OooRequestResponse, OooStatusRequest } from "../types/oooRequest";
 import { UpdateRequest } from "../types/requests";
@@ -159,7 +160,7 @@ export const updateOooRequestController = async (req: UpdateRequest, res: Custom
  * @param {OooRequestResponse} res - The response object.
  * @returns {Promise<OooRequestResponse>} Resolves with success or failure.
  */
-export const acknowledgeOooRequestController = async (
+export const acknowledgeOooRequest = async (
   req: AcknowledgeOooRequest,
   res: OooRequestResponse,
   next: NextFunction
@@ -182,7 +183,7 @@ export const acknowledgeOooRequestController = async (
 
     try {
 
-      const response = await acknowledgeOooRequest(requestId, requestBody, superUserId);
+      const response = await oooRequestService.acknowledgeOooRequest(requestId, requestBody, superUserId);
 
       return res.status(200).json({
         message: response.message,

controllers/requests.ts

@@ -5,7 +5,7 @@ import {
 } from "../constants/requests";
 import { getRequests } from "../models/requests";
 import { getPaginatedLink } from "../utils/helper";
-import { acknowledgeOooRequestController, createOooRequestController, updateOooRequestController } from "./oooRequests";
+import { acknowledgeOooRequest, createOooRequestController, updateOooRequestController } from "./oooRequests";
 import { AcknowledgeOooRequest, OooRequestCreateRequest, OooRequestResponse } from "../types/oooRequest";
 import { CustomResponse } from "../typeDefinitions/global";
 import { ExtensionRequestRequest, ExtensionRequestResponse } from "../types/extensionRequests";
@@ -125,7 +125,7 @@ export const updateRequestBeforeAcknowledgedController = async (req: Request, re
 
   switch(type){
     case REQUEST_TYPE.OOO:
-      await acknowledgeOooRequestController(req as AcknowledgeOooRequest, res as OooRequestResponse, next);
+      await acknowledgeOooRequest(req as AcknowledgeOooRequest, res as OooRequestResponse, next);
       break;
     case REQUEST_TYPE.ONBOARDING:
       await updateOnboardingExtensionRequestController(req as UpdateOnboardingExtensionRequest, res as OnboardingExtensionResponse);

test/unit/models/requests.test.ts

@@ -1,14 +1,15 @@
 import { expect } from "chai";
 import cleanDb from "../../utils/cleanDb";
-import { createRequest, getRequests, updateRequest, getRequestByKeyValues } from "../../../models/requests";
+import { createRequest, getRequests, updateRequest, getRequestByKeyValues, getRequestById } from "../../../models/requests";
 import {
   createOooRequests,
   createOooRequests2,
+  createOooRequests3,
   createOooStatusRequests,
   updateOooApprovedRequests,
   updateOooRejectedRequests,
 } from "./../../fixtures/oooRequest/oooRequest";
-import { REQUEST_STATE, REQUEST_TYPE } from "../../../constants/requests";
+import { REQUEST_DOES_NOT_EXIST, REQUEST_STATE, REQUEST_TYPE } from "../../../constants/requests";
 import userDataFixture from "./../../fixtures/user/user";
 import addUser from "../../utils/addUser";
 const userData = userDataFixture();
@@ -179,4 +180,20 @@ describe("models/oooRequests", () => {
       expect(oooRequestData).to.be.equal(null);
     });
   });
+
+  describe("getRequestById", () => {
+
+    it("should return request using request id", async () => {
+      const oooRequest = await createRequest(createOooRequests3);
+      const response = await getRequestById(oooRequest.id);
+      expect(response).to.deep.include(createOooRequests3);
+    });
+
+    it("should return REQUEST_DOES_NOT_EXIST for invalid request id", async () => {
+      await getRequestById("111111111111").catch((error) => {
+        expect(error).to.be.not.undefined;
+        expect(error.message).to.equal(REQUEST_DOES_NOT_EXIST);
+      });
+    });
+  });
 });