feat: extend User interface with identifier

src/users/domain/models/User.ts

@@ -5,5 +5,6 @@ export interface User {
   firstName: string
   lastName: string
   email: string
+  identifier: string
   affiliation?: string
 }

src/users/infrastructure/mappers/JSUserMapper.ts

@@ -0,0 +1,21 @@
+import { User } from '@/users/domain/models/User'
+import { AuthenticatedUser } from '@iqss/dataverse-client-javascript'
+
+export class JSUserMapper {
+  static toUser(authenticatedUser: AuthenticatedUser): User {
+    return {
+      displayName: authenticatedUser.displayName,
+      persistentId: authenticatedUser.persistentUserId,
+      firstName: authenticatedUser.firstName,
+      lastName: authenticatedUser.lastName,
+      email: authenticatedUser.email,
+      affiliation: authenticatedUser.affiliation,
+      superuser: authenticatedUser.superuser,
+      identifier: this.removeAtSymbol(authenticatedUser.identifier)
+    }
+  }
+
+  static removeAtSymbol(identifier: string): string {
+    return identifier.startsWith('@') ? identifier.slice(1) : identifier
+  }
+}

src/users/infrastructure/repositories/UserJSDataverseRepository.ts

@@ -9,6 +9,7 @@ import {
   deleteCurrentApiToken
 } from '@iqss/dataverse-client-javascript/dist/users'
 import { logout, ReadError, WriteError } from '@iqss/dataverse-client-javascript'
+import { JSUserMapper } from '../mappers/JSUserMapper'
 
 interface ApiTokenInfoPayload {
   apiToken: string
@@ -20,15 +21,7 @@ export class UserJSDataverseRepository implements UserRepository {
     return getCurrentAuthenticatedUser
       .execute()
       .then((authenticatedUser: AuthenticatedUser) => {
-        return {
-          displayName: authenticatedUser.displayName,
-          persistentId: authenticatedUser.persistentUserId,
-          firstName: authenticatedUser.firstName,
-          lastName: authenticatedUser.lastName,
-          email: authenticatedUser.email,
-          affiliation: authenticatedUser.affiliation,
-          superuser: authenticatedUser.superuser
-        }
+        return JSUserMapper.toUser(authenticatedUser)
       })
       .catch((error: ReadError) => {
         throw error

tests/component/users/domain/models/UserMother.ts

@@ -9,7 +9,8 @@ export class UserMother {
       lastName: 'Potts',
       email: '[email protected]',
       affiliation: 'Harvard University',
-      superuser: false
+      superuser: false,
+      identifier: 'jamespotts'
     }
   }
   static createSuperUser(): User {
@@ -20,7 +21,8 @@ export class UserMother {
       firstName: 'James',
       lastName: 'Potts',
       email: '[email protected]',
-      affiliation: 'Harvard University'
+      affiliation: 'Harvard University',
+      identifier: 'jamespotts'
     }
   }
 }