test: users

linode · Oct 2, 2024 · a89275d · a89275d
1 parent e6e8fc4
commit a89275d
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 12 deletions.
diff --git a/src/operator/keycloak.ts b/src/operator/keycloak.ts
@@ -710,8 +710,8 @@ async function manageGroups(connection: KeycloakConnection) {
 }
 
 async function createUpdateUser(api: any, user: any) {
-  const { username, email, firstName, lastName, isPlatformAdmin, isTeamAdmin, teams, teamId } = user
-  const userConf = createTeamUser(username, email, firstName, lastName, isPlatformAdmin, isTeamAdmin, teams, teamId)
+  const { email, firstName, lastName, isPlatformAdmin, isTeamAdmin, teams } = user
+  const userConf = createTeamUser(email, firstName, lastName, isPlatformAdmin, isTeamAdmin, teams)
   const existingUsersByUserEmail = (await doApiCall([], `Getting users`, () =>
     api.users.realmUsersGet(keycloakRealm, false, `${email}`),
   )) as UserRepresentation[]
@@ -723,11 +723,11 @@ async function createUpdateUser(api: any, user: any) {
       const updatedUserConf = existingUser.requiredActions?.includes('UPDATE_PASSWORD')
         ? userConf
         : omit(userConf, ['credentials'])
-      await doApiCall(errors, `Updating user ${username}`, async () =>
+      await doApiCall(errors, `Updating user ${email}`, async () =>
         api.users.realmUsersIdPut(keycloakRealm, existingUser.id as string, updatedUserConf),
       )
     } else {
-      await doApiCall(errors, `Creating user ${username}`, () => api.users.realmUsersPost(keycloakRealm, userConf))
+      await doApiCall(errors, `Creating user ${email}`, () => api.users.realmUsersPost(keycloakRealm, userConf))
     }
   } catch (error) {
     console.error('Error in internalIDP: ', error)

diff --git a/src/tasks/keycloak/config.ts b/src/tasks/keycloak/config.ts
@@ -71,16 +71,14 @@ export const adminUserCfgTpl = (username: string, password: string): Record<stri
 })
 
 export const teamUserCfgTpl = (
-  username: string,
   email: string,
   firstName: string,
   lastName: string,
   isPlatformAdmin: boolean,
   isTeamAdmin: boolean,
   teams: string[],
-  teamId: string,
 ): Record<string, unknown> => ({
-  username,
+  username: email,
   enabled: true,
   email,
   emailVerified: true,
@@ -90,13 +88,12 @@ export const teamUserCfgTpl = (
   groups: [
     ...(isPlatformAdmin ? ['platform-admin'] : []),
     ...(isTeamAdmin ? ['team-admin'] : []),
-    `team-${teamId}`,
     ...(teams.length > 0 ? teams.map((team) => `team-${team}`) : []),
   ],
   credentials: [
     {
       type: 'password',
-      value: `${username}@APL`,
+      value: `${email}`,
       temporary: true,
     },
   ],

diff --git a/src/tasks/keycloak/realm-factory.ts b/src/tasks/keycloak/realm-factory.ts
@@ -120,18 +120,16 @@ export function createAdminUser(username: string, password: string): UserReprese
   return userRepresentation
 }
 export function createTeamUser(
-  username: string,
   email: string,
   firstName: string,
   lastName: string,
   isPlatformAdmin: boolean,
   isTeamAdmin: boolean,
   teams: string[],
-  teamId: string,
 ): UserRepresentation {
   const userRepresentation = defaultsDeep(
     new UserRepresentation(),
-    teamUserCfgTpl(username, email, firstName, lastName, isPlatformAdmin, isTeamAdmin, teams, teamId),
+    teamUserCfgTpl(email, firstName, lastName, isPlatformAdmin, isTeamAdmin, teams),
   )
   return userRepresentation
 }