feat: tests for OOO acknowledgement feature (#2474)

RishiChaubey31 · MayankBansal12 · web-flow · commit c3626469007a · 2025-09-07T00:14:22.000+05:30
* initial commit

* added-more-test

* fix-test-title

* removed-try/catch-block

* fix-try-catch-cases

* added-try-catch

* removed-functon

* test: update assertions in oooRequest tests to use include for response validation

---------

Co-authored-by: Mayank Bansal &lt;mayankbansal125@gmail.com&gt;
diff --git a/test/fixtures/oooRequest/oooRequest.ts b/test/fixtures/oooRequest/oooRequest.ts
@@ -167,7 +167,7 @@ export const createOooRequests3 = {
   status: REQUEST_STATE.PENDING
 };
 
-export const acknowledgeOooRequest = {
+export const testAcknowledgeOooRequest = {
   type: REQUEST_TYPE.OOO,
   status: REQUEST_STATE.APPROVED,
   comment: "OOO request approved as it's emergency."
diff --git a/test/integration/requests.test.ts b/test/integration/requests.test.ts
@@ -15,7 +15,7 @@ import {
   validOooStatusRequests,
   validOooStatusUpdate,
   createOooRequests2,
-  acknowledgeOooRequest,
+  testAcknowledgeOooRequest,
   createOooRequests3,
 } from "../fixtures/oooRequest/oooRequest";
 import { createRequest, updateRequest } from "../../models/requests";
@@ -351,7 +351,7 @@ describe("/requests OOO", function () {
       chai
         .request(app)
         .patch(`/requests/${testOooRequest.id}?dev=true`)
-        .send(acknowledgeOooRequest)
+        .send(testAcknowledgeOooRequest)
         .end(function (err, res) {
           expect(res).to.have.status(401);
           expect(res.body.error).to.equal("Unauthorized");
@@ -365,7 +365,7 @@ describe("/requests OOO", function () {
         .request(app)
         .patch(`/requests/${testOooRequest.id}?dev=false`)
         .set("cookie", `${cookieName}=${superUserToken}`)
-        .send(acknowledgeOooRequest)
+        .send(testAcknowledgeOooRequest)
         .end(function (err, res) {
           if (err) {
             return done(err);
@@ -381,7 +381,7 @@ describe("/requests OOO", function () {
         .request(app)
         .patch(`/requests/11111111111111?dev=true`)
         .set("cookie", `${cookieName}=${superUserToken}`)
-        .send(acknowledgeOooRequest)
+        .send(testAcknowledgeOooRequest)
         .end(function (err, res) {
           if (err) {
             return done(err);
@@ -397,13 +397,13 @@ describe("/requests OOO", function () {
         .request(app)
         .patch(`/requests/${testOooRequest.id}?dev=true`)
         .set("cookie", `${cookieName}=${authToken}`)
-        .send(acknowledgeOooRequest)
+        .send(testAcknowledgeOooRequest)
         .end(function (err, res) {
           if (err) {
             return done(err);
           }
           expect(res.statusCode).to.equal(403);
-          // expect(res.body.message).to.equal(UNAUTHORIZED_TO_ACKNOWLEDGE_OOO_REQUEST);
+          expect(res.body.message).to.equal(UNAUTHORIZED_TO_CREATE_OOO_REQUEST);
           done();
         });
     });
@@ -413,7 +413,7 @@ describe("/requests OOO", function () {
         .request(app)
         .patch(`/requests/${approvedOooRequest.id}?dev=true`)
         .set("cookie", `${cookieName}=${superUserToken}`)
-        .send(acknowledgeOooRequest)
+        .send(testAcknowledgeOooRequest)
         .end(function (err, res) {
           if (err) {
             return done(err);
@@ -429,7 +429,7 @@ describe("/requests OOO", function () {
         .request(app)
         .patch(`/requests/${rejectedOooRequest.id}?dev=true`)
         .set("cookie", `${cookieName}=${superUserToken}`)
-        .send(acknowledgeOooRequest)
+        .send(testAcknowledgeOooRequest)
         .end(function (err, res) {
           if (err) {
             return done(err);
@@ -445,7 +445,7 @@ describe("/requests OOO", function () {
         .request(app)
         .patch(`/requests/${onboardingRequest.id}?dev=true`)
         .set("cookie", `${cookieName}=${superUserToken}`)
-        .send(acknowledgeOooRequest)
+        .send(testAcknowledgeOooRequest)
         .end(function (err, res) {
           if (err) {
             return done(err);
@@ -461,7 +461,7 @@ describe("/requests OOO", function () {
         .request(app)
         .patch(`/requests/${testOooRequest.id}?dev=true`)
         .set("cookie", `${cookieName}=${superUserToken}`)
-        .send(acknowledgeOooRequest)
+        .send(testAcknowledgeOooRequest)
         .end(function (err, res) {
           if (err) {
             return done(err);
@@ -477,7 +477,7 @@ describe("/requests OOO", function () {
         .request(app)
         .patch(`/requests/${testOooRequest.id}?dev=true`)
         .set("cookie", `${cookieName}=${superUserToken}`)
-        .send({...acknowledgeOooRequest, status: REQUEST_STATE.REJECTED})
+        .send({...testAcknowledgeOooRequest, status: REQUEST_STATE.REJECTED})
         .end(function (err, res) {
           if (err) {
             return done(err);
@@ -494,7 +494,7 @@ describe("/requests OOO", function () {
         .request(app)
         .patch(`/requests/${testOooRequest.id}?dev=true`)
         .set("cookie", `${cookieName}=${superUserToken}`)
-        .send(acknowledgeOooRequest)
+        .send(testAcknowledgeOooRequest)
         .end(function (err, res) {
           if (err) return done(err);
           expect(res.statusCode).to.equal(500);
diff --git a/test/unit/middlewares/oooRequests.test.ts b/test/unit/middlewares/oooRequests.test.ts
@@ -4,15 +4,17 @@ const { expect } = chai;
 
 import {
   createOooStatusRequestValidator,
-  // acknowledgeOOORequestsValidator,
+   acknowledgeOooRequestValidator,
 } from "./../../../middlewares/validators/oooRequests";
-import { acknowledgeOooRequest, validOooStatusRequests, validOooStatusUpdate } from "../../fixtures/oooRequest/oooRequest";
+import { testAcknowledgeOooRequest, validOooStatusRequests, validOooStatusUpdate } from "../../fixtures/oooRequest/oooRequest";
 import _ from "lodash";
+import { AcknowledgeOooRequest, OooRequestResponse } from "../../../types/oooRequest";
 
 describe("OOO Status Request Validators", function () {
   let req: any;
   let res: any;
   let nextSpy;
+  
   beforeEach(function () {
     res = {
       boom: {
@@ -91,40 +93,46 @@ describe("OOO Status Request Validators", function () {
     });
   });
 
-  describe.skip("acknowledgeOOORequestsValidator", function () {
+  describe("acknowledgeOOORequestsValidator", function () {
     it("should not validate for an invalid request for invalid request type", async function () {
       req = {
-        body: { ...acknowledgeOooRequest, type: "XYZ"}
+        body: { ...testAcknowledgeOooRequest, type: "XYZ"},
+        params: { id: "test-id" }
       };
 
-      // await acknowledgeOOORequestsValidator(req, res, nextSpy);
+      await acknowledgeOooRequestValidator(req as AcknowledgeOooRequest, res as OooRequestResponse, nextSpy);
+      expect(res.boom.badRequest.calledOnce).to.be.true;
       expect(nextSpy.notCalled).to.be.true;
     });
 
     it("should not validate for an invalid request if status is incorrect", async function () {
       req = {
-        body: { ...acknowledgeOooRequest, status: "PENDING"}
+        body: { ...testAcknowledgeOooRequest, status: "PENDING"},
+        params: { id: "test-id" }
       };
 
-      // await acknowledgeOOORequestsValidator(req, res, nextSpy);
+      await acknowledgeOooRequestValidator(req as AcknowledgeOooRequest, res as OooRequestResponse, nextSpy);
+      expect(res.boom.badRequest.calledOnce).to.be.true;
       expect(nextSpy.notCalled).to.be.true;
     });
 
     it("should validate for a valid acknowledge OOO request if comment not provided by superusers", async function() {
       req = {
-        body: _.omit(acknowledgeOooRequest, "comment")
+        body: _.omit(testAcknowledgeOooRequest, "comment"),
+        params: { id: "test-id" }
       };
       res = {};
-      // await acknowledgeOOORequestsValidator(req, res, nextSpy);
+      await acknowledgeOooRequestValidator(req as AcknowledgeOooRequest, res as OooRequestResponse, nextSpy);
       expect(nextSpy.calledOnce).to.be.true;
     });
 
     it("should validate for a valid acknowledge OOO request", async function() {
       req = {
-        body: acknowledgeOooRequest
+        body: testAcknowledgeOooRequest,
+        params: { id: "test-id" }
       };
       res = {};
-      // await acknowledgeOOORequestsValidator(req, res, nextSpy);
+      await acknowledgeOooRequestValidator(req as AcknowledgeOooRequest, res as OooRequestResponse, nextSpy);
       expect(nextSpy.calledOnce).to.be.true;
     });
   });
diff --git a/test/unit/services/oooRequest.test.ts b/test/unit/services/oooRequest.test.ts
