Removes super user authorisation for Task requests list and details API (#1683)

* chore : removes super user restriction on task requests api

* chore: changes test expect for not a super user case

Author: Ajeyakrishna-k

routes/taskRequests.js

@@ -7,8 +7,8 @@ const taskRequests = require("../controllers/tasksRequests");
 const { validateUser } = require("../middlewares/taskRequests");
 const validators = require("../middlewares/validators/task-requests");
 
-router.get("/", authenticate, authorizeRoles([SUPERUSER]), validators.getTaskRequests, taskRequests.fetchTaskRequests);
-router.get("/:id", authenticate, authorizeRoles([SUPERUSER]), taskRequests.fetchTaskRequestById);
+router.get("/", authenticate, taskRequests.fetchTaskRequests);
+router.get("/:id", authenticate, taskRequests.fetchTaskRequestById);
 router.post("/addOrUpdate", authenticate, validateUser, taskRequests.addOrUpdate);
 router.patch("/approve", authenticate, authorizeRoles([SUPERUSER]), validateUser, taskRequests.approveTaskRequest);
 router.post("/", authenticate, validators.postTaskRequests, taskRequests.addTaskRequests);

test/integration/taskRequests.test.js

@@ -119,7 +119,7 @@ describe("Task Requests", function () {
         await taskRequestsModel.addOrUpdate(taskId, userId);
       });
 
-      it("should return 401 unauthorized user response", function (done) {
+      it("should be successful when the user is not a super user", function (done) {
         chai
           .request(app)
           .get("/taskRequests")
@@ -129,7 +129,7 @@ describe("Task Requests", function () {
               return done(err);
             }
 
-            expect(res).to.have.status(401);
+            expect(res).to.have.status(200);
             return done();
           });
       });
@@ -196,7 +196,7 @@ describe("Task Requests", function () {
         await taskRequestsModel.addOrUpdate(taskId, userId);
       });
 
-      it("should return 401 status code", function (done) {
+      it("should be successful when the user is not a super user", function (done) {
         chai
           .request(app)
           .get(`/taskRequests/taskrequstid`)
@@ -206,7 +206,7 @@ describe("Task Requests", function () {
               return done(err);
             }
 
-            expect(res).to.have.status(401);
+            expect(res).to.have.status(200);
             return done();
           });
       });