refactor: rebase master branch

logto-io · Jun 12, 2024 · 182991a · 182991a
1 parent 1ed8c0a
commit 182991a
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 13 deletions.
diff --git a/packages/core/src/libraries/user.ts b/packages/core/src/libraries/user.ts
@@ -1,3 +1,4 @@
+/* eslint-disable max-lines */
 import type { BindMfa, CreateUser, MfaVerification, Scope, User } from '@logto/schemas';
 import { MfaFactor, RoleType, Users, UsersPasswordEncryptionMethod } from '@logto/schemas';
 import { generateStandardShortId, generateStandardId } from '@logto/shared';
@@ -122,15 +123,14 @@ export const createUserLibrary = (queries: Queries) => {
     set: Partial<OmitAutoSetFields<CreateUser>>,
     jsonbMode?: 'replace' | 'merge'
   ) => {
+    const { primaryPhone, ...rest } = set;
     const validPhoneNumber = conditional(
-      'primaryPhone' in set &&
-        typeof set.primaryPhone === 'string' &&
-        getValidPhoneNumber(set.primaryPhone)
+      typeof primaryPhone === 'string' && getValidPhoneNumber(primaryPhone)
     );
 
     return updateUserByIdQuery(
       id,
-      { ...set, ...conditional(validPhoneNumber && { primaryPhone: validPhoneNumber }) },
+      { ...rest, ...conditional(validPhoneNumber && { primaryPhone: validPhoneNumber }) },
       jsonbMode
     );
   };
@@ -147,10 +147,9 @@ export const createUserLibrary = (queries: Queries) => {
 
     assertThat(parameterRoles.length === roleNames.length, 'role.default_role_missing');
 
+    const { primaryPhone, ...rest } = data;
     const validPhoneNumber = conditional(
-      'primaryPhone' in data &&
-        typeof data.primaryPhone === 'string' &&
-        getValidPhoneNumber(data.primaryPhone)
+      typeof primaryPhone === 'string' && getValidPhoneNumber(primaryPhone)
     );
 
     return pool.transaction(async (connection) => {
@@ -159,7 +158,7 @@ export const createUserLibrary = (queries: Queries) => {
       });
 
       const user = await insertUserQuery({
-        ...data,
+        ...rest,
         ...conditional(validPhoneNumber && { primaryPhone: validPhoneNumber }),
       });
       const roles = deduplicateByKey([...parameterRoles, ...defaultRoles], 'id');
@@ -173,7 +172,7 @@ export const createUserLibrary = (queries: Queries) => {
 
       const provisionOrganizations = async (): Promise<readonly string[]> => {
         // Just-in-time organization provisioning
-        const userEmailDomain = data.primaryEmail?.split('@')[1];
+        const userEmailDomain = rest.primaryEmail?.split('@')[1];
         // TODO: Remove this check when launching
         if (EnvSet.values.isDevFeaturesEnabled && userEmailDomain) {
           const organizationQueries = new OrganizationQueries(connection);
@@ -368,3 +367,4 @@ export const createUserLibrary = (queries: Queries) => {
     updateUserById,
   };
 };
+/* eslint-enable max-lines */
diff --git a/packages/core/src/routes/admin-user/mfa-verifications.test.ts b/packages/core/src/routes/admin-user/mfa-verifications.test.ts
@@ -8,6 +8,7 @@ import {
   mockUserTotpMfaVerification,
   mockUserWithMfaVerifications,
 } from '#src/__mocks__/index.js';
+import { type InsertUserResult } from '#src/libraries/user.js';
 import type Libraries from '#src/tenants/Libraries.js';
 import type Queries from '#src/tenants/Queries.js';
 import { MockTenant, type Partial2 } from '#src/test-utils/tenant.js';
@@ -44,10 +45,13 @@ const mockLibraries = {
   users: {
     generateUserId: jest.fn(async () => 'fooId'),
     insertUser: jest.fn(
-      async (user: CreateUser): Promise<User> => ({
-        ...mockUser,
-        ...removeUndefinedKeys(user), // No undefined values will be returned from database
-      })
+      async (user: CreateUser): Promise<InsertUserResult> => [
+        {
+          ...mockUser,
+          ...removeUndefinedKeys(user), // No undefined values will be returned from database
+        },
+        { organizationIds: [] },
+      ]
     ),
     updateUserById: jest.fn(
       async (_, data: Partial<CreateUser>): Promise<User> => ({