fix: test cases

Author: surajmaity1

test/integration/requests.test.ts

@@ -41,6 +41,7 @@ import { deleteUserStatus, updateUserStatus } from "../../models/userStatus";
 import * as requestsQuery from "../../models/requests";
 import { userState } from "../../constants/userStatus";
 import * as logUtils from "../../services/logService";
+import { OldOooRequest, OooStatusRequest } from "../../types/oooRequest";
 
 const userData = userDataFixture();
 chai.use(chaiHttp);
@@ -691,6 +692,34 @@ describe("/requests OOO", function () {
           done();
         });
     });
+
+    it("should return OOO requests with state and message field when dev is true", (done) => {
+      chai
+        .request(app)
+        .get("/requests?dev=true&type=OOO")
+        .end(function (err, res) {
+          expect(res).to.have.status(200);
+          res.body.data.forEach(request => {
+            expect(request).to.have.property('state');
+            expect(request).to.have.property('message');
+          });
+          done();
+        });
+    });
+
+    it("should return OOO requests with status and reason field when dev is false", (done) => {
+      chai
+        .request(app)
+        .get("/requests?dev=false&type=OOO")
+        .end(function (err, res) {
+          expect(res).to.have.status(200);
+          res.body.data.forEach(request => {
+            expect(request).to.have.property('status');
+            expect(request).to.have.property('reason');
+          });
+          done();
+        });
+    });
   });
 });
 

test/unit/utils/requests.test.ts

@@ -1,6 +1,10 @@
-import { convertDateStringToMilliseconds, getNewDeadline } from "../../../utils/requests"
+import { convertDateStringToMilliseconds, getNewDeadline, transformGetOooRequest } from "../../../utils/requests"
 import { convertDaysToMilliseconds } from "../../../utils/time";
 import {expect} from "chai";
+import { createdOOORequest, oooStatusRequests } from "../../fixtures/oooRequest/oooRequest";
+import sinon from "sinon";
+import * as usersModel from "../../../models/users";
+import { ERROR_WHILE_FETCHING_REQUEST } from "../../../constants/requests";
 
 describe("Test getNewDeadline", () => {
     const currentDate = Date.now();
@@ -42,4 +46,29 @@ describe("Test convertDateStringInMilliseconds", () => {
         expect(res.isDate).to.equal(true);
         expect(res.milliseconds).to.equal(Date.parse(validDateString));
     })
-})
+})
+
+describe("Test transformGetOooRequest", async () => {
+    it("should return OOO requests with old format when dev is true", async () => {
+        const response = await transformGetOooRequest(true, [ createdOOORequest, ...oooStatusRequests ]);
+        expect(response[0].state).to.equal(createdOOORequest.status);
+        expect(response[0].message).to.equal(createdOOORequest.reason);
+        expect(response[0].requestedBy).to.equal(createdOOORequest.userId);
+    });
+
+    it("should return OOO requests with new format when dev is false", async () => {
+        const response = await transformGetOooRequest(false, [ createdOOORequest, ...oooStatusRequests ]);
+        expect(response[1].status).to.equal(oooStatusRequests[0].state);
+        expect(response[1].reason).to.equal(oooStatusRequests[0].message);
+        expect(response[1].userId).to.equal(oooStatusRequests[0].requestedBy);
+    });
+
+    it("should throw ERROR_WHILE_FETCHING_REQUEST ", async () => {
+        await transformGetOooRequest(false, [createdOOORequest, ...oooStatusRequests])
+            .then(() => { throw new Error(ERROR_WHILE_FETCHING_REQUEST); })
+            .catch(error => {
+                expect(error).to.be.instanceOf(Error);
+                expect(error.message).to.equal(ERROR_WHILE_FETCHING_REQUEST);
+            });
+    });
+});

utils/requests.ts

@@ -1,6 +1,5 @@
 import { ERROR_WHILE_FETCHING_REQUEST } from "../constants/requests";
 import { fetchUser } from "../models/users";
-import { userData } from "../types/global";
 import { OldOooRequest, OooStatusRequest } from "../types/oooRequest";
 
 /**