Fix users batch invite with roles (#400)

+ test
descope · Oct 6, 2024 · 5944578 · 5944578
1 parent 8fb2553
commit 5944578
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/lib/management/user.test.ts b/lib/management/user.test.ts
@@ -387,8 +387,8 @@ describe('Management User', () => {
 
       const resp: SdkResponse<InviteBatchResponse> = await management.user.inviteBatch(
         [
-          { loginId: 'one', email: 'one@one', password: 'clear', seed: 'aaa' },
-          { loginId: 'two', email: 'two@two', hashedPassword: hashed },
+          { loginId: 'one', roles: ['r1'], email: 'one@one', password: 'clear', seed: 'aaa' },
+          { loginId: 'two', roles: ['r1'], email: 'two@two', hashedPassword: hashed },
         ],
         'https://invite.me',
         true,
@@ -400,12 +400,14 @@ describe('Management User', () => {
           users: [
             {
               loginId: 'one',
+              roleNames: ['r1'],
               email: 'one@one',
               password: 'clear',
               seed: 'aaa',
             },
             {
               loginId: 'two',
+              roleNames: ['r1'],
               email: 'two@two',
               hashedPassword: {
                 firebase: {

diff --git a/lib/management/user.ts b/lib/management/user.ts
@@ -434,7 +434,14 @@ const withUser = (sdk: CoreSdk, managementKey?: string) => {
         sdk.httpClient.post(
           apiPaths.user.createBatch,
           {
-            users,
+            users: users.map((u) => {
+              const res = {
+                ...u,
+                roleNames: u.roles,
+              };
+              delete res.roles;
+              return res;
+            }),
             invite: true,
             inviteUrl,
             sendMail,