add test and refactor

Author: sahsisunny

services/contributions.js

@@ -12,7 +12,6 @@ const getUserContributions = async (username) => {
   const { data } = await githubService.fetchPRsByUser(username);
   const allUserTasks = await tasks.fetchUserTasks(username);
   const noteworthy = [];
-  const collapsed = [];
   const all = [];
 
   if (data.total_count) {
@@ -27,11 +26,9 @@ const getUserContributions = async (username) => {
 
     for (const task of allUserTasks) {
       const noteworthyObject = {};
-      const collapsedTaskObject = {};
       const participantsDetails = [];
 
       noteworthyObject.task = extractTaskdetails(task);
-      collapsedTaskObject.task = extractTaskdetails(task);
 
       if (Array.isArray(task.participants)) {
         for (const userId of task.participants) {
@@ -48,7 +45,6 @@ const getUserContributions = async (username) => {
       }
 
       noteworthyObject.task.participants = participantsDetails;
-      collapsedTaskObject.task.participants = participantsDetails;
       const prList = [];
 
       task.links?.forEach((link) => {
@@ -66,10 +62,6 @@ const getUserContributions = async (username) => {
       } else {
         all.push(noteworthyObject);
       }
-
-      collapsedTaskObject.prList = prList;
-
-      task.isCollapsed ? collapsed.push(collapsedTaskObject) : all.push(collapsedTaskObject);
     }
 
     for (const prDetails of prMaps.values()) {
@@ -81,7 +73,6 @@ const getUserContributions = async (username) => {
     }
   }
   contributions.noteworthy = noteworthy;
-  contributions.collapsed = collapsed;
   contributions.all = all;
   return contributions;
 };

test/integration/tasks.test.js

@@ -290,6 +290,23 @@ describe("Tasks", function () {
           return done();
         });
     });
+    it("Should update the task status collapsed for the given taskid", function (done) {
+      chai
+        .request(app)
+        .patch("/tasks/" + taskId1)
+        .set("cookie", `${cookieName}=${jwt}`)
+        .send({
+          isCollapsed: true,
+        })
+        .end((err, res) => {
+          if (err) {
+            return done(err);
+          }
+          expect(res).to.have.status(204);
+
+          return done();
+        });
+    });
     it("Should return fail response if task data has a non-acceptable status value to update the task for the given taskid", function (done) {
       chai
         .request(app)