Added test for id filterin in addOrUpdate

Author: RahulGoyal-tech

test/unit/models/users.test.js

@@ -114,6 +114,16 @@ describe("users", function () {
       const { user } = await users.fetchUser({ githubUsername });
       expect(user).to.haveOwnProperty("github_created_at");
     });
+
+    it("it should filter out the id field while updating profileDiff", async function () {
+      const userData = userDataArray[0];
+      const { userId } = await users.addOrUpdate(userData);
+      const profileDiff = { id: "random-id", diff: "random-diff" };
+      await users.addOrUpdate(profileDiff, userId);
+      const data = (await userModel.doc(userId).get()).data();
+      expect(data).to.haveOwnProperty("diff"); // Ensures profile diffs are pushed
+      expect(data.id).not.equal("random-id"); // Ensures id is not added / updated by profile Diffs
+    });
   });
 
   describe("fetch user details based on discord id", function () {