User deletion

packages/core-payment/src/module/configurePaymentCredentialsModule.ts

@@ -36,7 +36,10 @@ export const configurePaymentCredentialsModule = (
 
   return {
     markPreferred,
-
+    async count(query: mongodb.Filter<PaymentCredentialsType>): Promise<number> {
+      const credentials = await PaymentCredentials.countDocuments(query);
+      return credentials;
+    },
     credentialsExists: async ({
       paymentCredentialsId,
     }: {
@@ -128,6 +131,10 @@ export const configurePaymentCredentialsModule = (
       const paymentCredentials = await PaymentCredentials.findOneAndDelete(selector, {});
       return paymentCredentials;
     },
+    deleteUserPaymentCredentials: async (userId: string): Promise<number> => {
+      const { deletedCount } = await PaymentCredentials.deleteMany({ userId }, {});
+      return deletedCount;
+    },
   };
 };
 

packages/core-users/src/module/configureUsersModule.ts

@@ -631,6 +631,9 @@ export const configureUsersModule = async ({
     },
 
     markDeleted: async (userId: string): Promise<User> => {
+      await db.collection('sessions').deleteMany({
+        session: { $regex: `"user":"${userId}"` },
+      });
       const user = await Users.findOneAndUpdate(
         { _id: userId },
         {

packages/core-users/src/module/configureUsersWebAuthnModule.ts

@@ -19,6 +19,7 @@ export interface UsersWebAuthnModule {
     extensionOptions?: any,
   ) => Promise<any>;
   verifyCredentialRequest: (userPublicKeys: any[], username: string, credentials: any) => Promise<any>;
+  deleteUserWebAuthnCredentials: (username: string) => Promise<number>;
 }
 
 const logger = createLogger('unchained:core-users');
@@ -231,5 +232,11 @@ export const configureUsersWebAuthnModule = async ({
       const loginResult = await f2l.assertionResult(assertionResponse, assertionExpectations);
       return { userHandle: loginResult?.authnrData?.get('userHandle') };
     },
+    deleteUserWebAuthnCredentials: async (username: string) => {
+      const { deletedCount } = await WebAuthnCredentialsCreationRequests.deleteMany({
+        username,
+      });
+      return deletedCount;
+    },
   };
 };

packages/core/src/services/deleteUser.ts

@@ -8,6 +8,8 @@ export async function deleteUserService(this: Modules, { userId }: { userId: str
   await this.bookmarks.deleteByUserId(userId);
   await this.quotations.deleteRequestedUserQuotations(userId);
   await this.enrollments.deleteInactiveUserEnrollments(userId);
+  await this.payment.paymentCredentials.deleteUserPaymentCredentials(userId);
+  await this.users.webAuthn.deleteUserWebAuthnCredentials(user.username);
 
   const carts = await this.orders.findOrders({ userId, status: null });
 
@@ -24,8 +26,16 @@ export async function deleteUserService(this: Modules, { userId }: { userId: str
   const reviewsCount = await this.products.reviews.count({ authorId: userId });
   const enrollmentsCount = await this.enrollments.count({ userId });
   const tokens = await this.warehousing.findTokensForUser({ userId });
-
-  if (!ordersCount && !reviewsCount && !enrollmentsCount && !quotationsCount && !tokens?.length) {
+  const paymentCredentials = await this.payment.paymentCredentials.count({ userId });
+
+  if (
+    !ordersCount &&
+    !reviewsCount &&
+    !enrollmentsCount &&
+    !quotationsCount &&
+    !tokens?.length &&
+    !paymentCredentials
+  ) {
     await this.users.deletePermanently({ userId });
   }
 

packages/core/src/services/migrateUserData.ts

@@ -10,13 +10,13 @@ export async function migrateUserDataService(userIdBeforeLogin: string, userId:
     fromUserId: userIdBeforeLogin,
     toUserId: userId,
     shouldMerge: userSettings.mergeUserCartsOnLogin,
-    countryContext: userBeforeLogin.lastLogin?.countryCode || user.lastLogin?.countryCode,
+    countryContext: userBeforeLogin?.lastLogin?.countryCode || user?.lastLogin?.countryCode,
   });
 
   await migrateBookmarksService.bind(this)({
     fromUserId: userIdBeforeLogin,
     toUserId: userId,
     shouldMerge: userSettings.mergeUserCartsOnLogin,
-    countryContext: userBeforeLogin.lastLogin?.countryCode || user.lastLogin?.countryCode,
+    countryContext: userBeforeLogin?.lastLogin?.countryCode || user?.lastLogin?.countryCode,
   });
 }

tests/products.test.js

@@ -1762,7 +1762,6 @@ describe('Products', () => {
     it('return minimum and maximum simulated price range of a configurable product', async () => {
       const {
         data: { product = {} },
-        ...rest
       } = await graphqlFetchAsAdmin({
         query: /* GraphQL */ `
           query simulatedPriceRange($productId: ID!) {