Fix: Reverted changes which occured due to departed-users api.

Author: VinuB-Dev

models/users.js

@@ -8,12 +8,8 @@ const firestore = require("../utils/firestore");
 const { fetchWallet, createWallet } = require("../models/wallets");
 const { updateUserStatus } = require("../models/userStatus");
 const { arraysHaveCommonItem, chunks } = require("../utils/array");
-const {
-  ALLOWED_FILTER_PARAMS,
-  FIRESTORE_IN_CLAUSE_SIZE,
-  USERS_PATCH_HANDLER_SUCCESS_MESSAGES,
-  USERS_PATCH_HANDLER_ERROR_MESSAGES,
-} = require("../constants/users");
+const { archiveUsers } = require("../services/users");
+const { ALLOWED_FILTER_PARAMS, FIRESTORE_IN_CLAUSE_SIZE } = require("../constants/users");
 const { DOCUMENT_WRITE_SIZE } = require("../constants/constants");
 const { userState } = require("../constants/userStatus");
 const { BATCH_SIZE_IN_CLAUSE } = require("../constants/firebase");
@@ -31,46 +27,6 @@ const { formatUsername } = require("../utils/username");
 const { logType } = require("../constants/logs");
 const { addLog } = require("../services/logService");
 
-const archiveUsers = async (usersData) => {
-  const batch = firestore.batch();
-  const usersBatch = [];
-  const summary = {
-    totalUsersArchived: 0,
-    totalOperationsFailed: 0,
-    updatedUserDetails: [],
-    failedUserDetails: [],
-  };
-
-  usersData.forEach((user) => {
-    const { id, first_name: firstName, last_name: lastName } = user;
-    const updatedUserData = {
-      ...user,
-      roles: {
-        ...user.roles,
-        archived: true,
-      },
-      updated_at: Date.now(),
-    };
-    batch.update(userModel.doc(id), updatedUserData);
-    usersBatch.push({ id, firstName, lastName });
-  });
-
-  try {
-    await batch.commit();
-    summary.totalUsersArchived += usersData.length;
-    summary.updatedUserDetails = [...usersBatch];
-    return {
-      message: USERS_PATCH_HANDLER_SUCCESS_MESSAGES.ARCHIVE_USERS.SUCCESSFULLY_COMPLETED_BATCH_UPDATES,
-      ...summary,
-    };
-  } catch (err) {
-    logger.error("Firebase batch Operation Failed!");
-    summary.totalOperationsFailed += usersData.length;
-    summary.failedUserDetails = [...usersBatch];
-    return { message: USERS_PATCH_HANDLER_ERROR_MESSAGES.ARCHIVE_USERS.BATCH_DATA_UPDATED_FAILED, ...summary };
-  }
-};
-
 /**
  * Adds or updates the user data
  *
@@ -1088,7 +1044,6 @@ const fetchUsersNotInDiscordServer = async () => {
 };
 
 module.exports = {
-  archiveUsers,
   addOrUpdate,
   fetchPaginatedUsers,
   fetchUser,

test/unit/services/users.test.js

@@ -5,8 +5,8 @@ const firestore = require("../../../utils/firestore");
 const userModel = firestore.collection("users");
 const cleanDb = require("../../utils/cleanDb");
 const userDataArray = require("../../fixtures/user/user")();
-const { generateUniqueUsername } = require("../../../services/users");
-const { addOrUpdate, archiveUsers } = require("../../../models/users");
+const { archiveUsers, generateUniqueUsername } = require("../../../services/users");
+const { addOrUpdate } = require("../../../models/users");
 
 describe("Users services", function () {
   describe("archive inactive discord users in bulk", function () {