Merge pull request #2087 from CarnegieLearningWeb/code-cleanup/remove…

…-experimentusercontroller-issue-2084 Remove ExperimentUserController as its not used in the frontend
CarnegieLearningWeb · Nov 4, 2024 · 733dc51 · 733dc51
2 parents e885041 + 69f4a55
commit 733dc51
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 273 deletions.
diff --git a/backend/packages/Upgrade/src/api/controllers/ExperimentUserController.ts b/backend/packages/Upgrade/src/api/controllers/ExperimentUserController.ts
diff --git a/backend/packages/Upgrade/src/api/services/ExperimentUserService.ts b/backend/packages/Upgrade/src/api/services/ExperimentUserService.ts
@@ -233,12 +233,6 @@ export class ExperimentUserService {
     return this.userRepository.save(newDocument);
   }
 
-  public update(id: string, user: Partial<ExperimentUser>, logger: UpgradeLogger): Promise<ExperimentUser> {
-    logger.info({ message: `Update a user ${user.toString()}` });
-    user.id = id;
-    return this.userRepository.save(user);
-  }
-
   // TODO should we check for workingGroup as a subset over here?
   public async updateGroupMembership(
     userId: string,

diff --git a/backend/packages/Upgrade/test/unit/controllers/ExperimentUserController.test.ts b/backend/packages/Upgrade/test/unit/controllers/ExperimentUserController.test.ts