updated tests and replaced res.send

Author: iamYashSinha

controllers/users.js

@@ -495,12 +495,14 @@ const migrate = async (req, res) => {
       await batchWrite.commit();
     }
 
-    return res.send({
-      statusCode: 201,
+    return res.staus(200).json({
+      statusCode: 200,
       message: "All Users github_user_id added successfully",
     });
   } catch (error) {
-    return res.status(500).send("Internal server error");
+    return res.status(500).json({
+      message: "Internal Server Error",
+    });
   }
 };
 

test/integration/users.test.js

@@ -1039,14 +1039,6 @@ describe("Users", function () {
   });
 
   describe("POST /users/migrate", function () {
-    beforeEach(async function () {
-      superUserId = await addUser(superUser);
-      superUserAuthToken = authService.generateAuthToken({ userId: superUserId });
-    });
-
-    afterEach(async function () {
-      await cleanDb();
-    });
     it("Should return 401 when user is unauthorize", function (done) {
       chai
         .request(app)

test/unit/models/users.test.js

@@ -102,7 +102,7 @@ describe("users", function () {
 
     it("should update the github_id in the user collection", async function () {
       const userData = userDataArray[0];
-      userData.github_user_id = "Yash Sinha";
+      userData.github_id = "Yash Sinha";
 
       // Add the user the first time
       const { userId } = await users.addOrUpdate(userData);