Mark neo4j user update queries as private and skip reimpl in edgedb

Author: CarsonF

src/components/user/user.edgedb.repository.ts

@@ -3,7 +3,6 @@ import {
   DuplicateException,
   ID,
   NotFoundException,
-  Role,
   ServerException,
   Session,
 } from '~/common';
@@ -99,36 +98,6 @@ export class UserEdgedbRepository extends UserRepository {
     }
   }
 
-  async updateEmail(
-    user: User,
-    email: string | null | undefined,
-  ): Promise<void> {
-    const query = e.update(e.User, () => ({
-      filter_single: { id: user.id },
-      set: { email },
-    }));
-    try {
-      await this.edgedb.run(query);
-    } catch (e) {
-      if (isExclusivityViolation(e, 'email')) {
-        throw new DuplicateException(
-          'person.email',
-          'Email address is already in use',
-          e,
-        );
-      }
-      throw e;
-    }
-  }
-
-  async updateRoles(user: User, roles: Role[]): Promise<void> {
-    const query = e.update(e.User, () => ({
-      filter_single: { id: user.id },
-      set: { roles },
-    }));
-    await this.edgedb.run(query);
-  }
-
   async delete(id: ID, _session: Session, _object: User): Promise<void> {
     const query = e.delete(e.User, () => ({
       filter_single: { id },

src/components/user/user.repository.ts

@@ -121,7 +121,7 @@ export class UserRepository extends DtoRepository<typeof User, [Session | ID]>(
         );
   }
 
-  async updateEmail(
+  private async updateEmail(
     user: User,
     email: string | null | undefined,
   ): Promise<void> {
@@ -151,7 +151,7 @@ export class UserRepository extends DtoRepository<typeof User, [Session | ID]>(
     }
   }
 
-  async updateRoles(user: User, roles: Role[]): Promise<void> {
+  private async updateRoles(user: User, roles: Role[]): Promise<void> {
     const removals = difference(user.roles.value, roles);
     const additions = difference(roles, user.roles.value);