fix: addToCart takes actual product code into account

feature-libs/cart/base/components/added-to-cart-dialog/added-to-cart-dialog-event.listener.spec.ts

@@ -1,19 +1,29 @@
-import { ElementRef, ViewContainerRef } from '@angular/core';
+import { AbstractType, ElementRef, ViewContainerRef } from '@angular/core';
 import { TestBed } from '@angular/core/testing';
 import {
   CartAddEntryFailEvent,
+  CartAddEntrySuccessEvent,
   CartUiEventAddToCart,
 } from '@spartacus/cart/base/root';
-import { CxEvent, EventService } from '@spartacus/core';
-import { LaunchDialogService, LAUNCH_CALLER } from '@spartacus/storefront';
+import { CxEvent, EventService, FeatureConfigService } from '@spartacus/core';
+import { LAUNCH_CALLER, LaunchDialogService } from '@spartacus/storefront';
 import { BehaviorSubject, EMPTY, Observable } from 'rxjs';
 import { AddedToCartDialogEventListener } from './added-to-cart-dialog-event.listener';
+import { OrderEntry } from '../../root/models';
 
 const mockEventStream$ = new BehaviorSubject<CxEvent>({});
+const mockEventSuccessStream$ = new BehaviorSubject<CxEvent>({});
 
 class MockEventService implements Partial<EventService> {
-  get(): Observable<any> {
-    return mockEventStream$.asObservable();
+  get<T>(eventType: AbstractType<T>): Observable<T> {
+    if (
+      eventType.name === CartUiEventAddToCart.type ||
+      eventType.name === CartAddEntryFailEvent.type
+    ) {
+      return mockEventStream$.asObservable() as Observable<T>;
+    } else {
+      return mockEventSuccessStream$.asObservable() as Observable<T>;
+    }
   }
 }
 
@@ -28,18 +38,23 @@ class MockLaunchDialogService implements Partial<LaunchDialogService> {
   closeDialog(_reason: string): void {}
 }
 
+const entry: OrderEntry = { quantity: 0 };
+
 const mockEvent = new CartUiEventAddToCart();
+const mockSuccessEvent = new CartAddEntrySuccessEvent();
 mockEvent.productCode = 'test';
 mockEvent.quantity = 3;
 mockEvent.numberOfEntriesBeforeAdd = 1;
 mockEvent.pickupStoreName = 'testStore';
+mockSuccessEvent.entry = entry;
 
 const mockFailEvent = new CartAddEntryFailEvent();
 mockFailEvent.error = {};
 
-describe('AddToCartDialogEventListener', () => {
+describe('AddedToCartDialogEventListener', () => {
   let listener: AddedToCartDialogEventListener;
   let launchDialogService: LaunchDialogService;
+  let featureConfigService: FeatureConfigService;
 
   beforeEach(() => {
     TestBed.configureTestingModule({
@@ -56,37 +71,94 @@ describe('AddToCartDialogEventListener', () => {
       ],
     });
 
-    listener = TestBed.inject(AddedToCartDialogEventListener);
+    featureConfigService = TestBed.inject(FeatureConfigService);
     launchDialogService = TestBed.inject(LaunchDialogService);
   });
 
   describe('onAddToCart', () => {
-    it('Should open modal on event', () => {
+    it('should open modal on event CartAddEntrySuccessEvent in case toggle adddedToCartDialogDrivenBySuccessEvent is active', () => {
+      spyOn(featureConfigService, 'isEnabled').and.returnValue(true);
+      listener = TestBed.inject(AddedToCartDialogEventListener);
+      spyOn(listener as any, 'openModalAfterSuccess').and.stub();
+      mockEventSuccessStream$.next(mockSuccessEvent);
+      expect(listener['openModalAfterSuccess']).toHaveBeenCalledWith(
+        mockSuccessEvent
+      );
+    });
+
+    it('should not open modal on event CartAddEntrySuccessEvent in case toggle adddedToCartDialogDrivenBySuccessEvent is inactive', () => {
+      spyOn(featureConfigService, 'isEnabled').and.returnValue(false);
+      listener = TestBed.inject(AddedToCartDialogEventListener);
+      spyOn(listener as any, 'openModalAfterSuccess').and.stub();
+      mockEventSuccessStream$.next(mockSuccessEvent);
+      expect(listener['openModalAfterSuccess']).not.toHaveBeenCalled();
+    });
+
+    it('should open modal on event CartUiEventAddToCart in case toggle adddedToCartDialogDrivenBySuccessEvent is inactive', () => {
+      spyOn(featureConfigService, 'isEnabled').and.returnValue(false);
+      listener = TestBed.inject(AddedToCartDialogEventListener);
       spyOn(listener as any, 'openModal').and.stub();
       mockEventStream$.next(mockEvent);
       expect(listener['openModal']).toHaveBeenCalledWith(mockEvent);
     });
 
-    it('Should close modal on fail event', () => {
+    it('should not open modal on event CartUiEventAddToCart in case toggle adddedToCartDialogDrivenBySuccessEvent is active', () => {
+      spyOn(featureConfigService, 'isEnabled').and.returnValue(true);
+      listener = TestBed.inject(AddedToCartDialogEventListener);
+      spyOn(listener as any, 'openModal').and.stub();
+      mockEventStream$.next(mockEvent);
+      expect(listener['openModal']).not.toHaveBeenCalled();
+    });
+
+    it('should close modal on fail event in case toggle is inactive', () => {
+      spyOn(featureConfigService, 'isEnabled').and.returnValue(false);
+      listener = TestBed.inject(AddedToCartDialogEventListener);
       spyOn(listener as any, 'closeModal').and.stub();
       mockEventStream$.next(mockFailEvent);
       expect(listener['closeModal']).toHaveBeenCalledWith(mockFailEvent);
     });
   });
 
   describe('openModal', () => {
-    it('Should open the add to cart dialog', () => {
+    it('should open the add to cart dialog', () => {
+      listener = TestBed.inject(AddedToCartDialogEventListener);
       spyOn(launchDialogService, 'openDialog').and.callThrough();
       listener['openModal'](mockEvent);
       expect(launchDialogService.openDialog).toHaveBeenCalled();
     });
   });
 
   describe('closeModal', () => {
-    it('Should close the add to cart dialog', () => {
+    it('should close the add to cart dialog', () => {
+      listener = TestBed.inject(AddedToCartDialogEventListener);
       spyOn(launchDialogService, 'closeDialog').and.stub();
       listener['closeModal']('reason');
       expect(launchDialogService.closeDialog).toHaveBeenCalledWith('reason');
     });
   });
+
+  describe('calculateEntryWasMerged', () => {
+    it('should return true in case no quantityAdded is present (which happens if no stock is available)', () => {
+      mockSuccessEvent.quantityAdded = undefined;
+      expect(listener['calculateEntryWasMerged'](mockSuccessEvent)).toBe(true);
+    });
+
+    it('should return true in case the resulting entries quantity exceeds the quantity that was added to the cart', () => {
+      mockSuccessEvent.quantityAdded = 1;
+      entry.quantity = 2;
+      expect(listener['calculateEntryWasMerged'](mockSuccessEvent)).toBe(true);
+    });
+
+    it('should return false in case the resulting entries quantity equals the quantity that was added to the cart', () => {
+      mockSuccessEvent.quantityAdded = 3;
+      entry.quantity = 3;
+      expect(listener['calculateEntryWasMerged'](mockSuccessEvent)).toBe(false);
+    });
+
+    it('should return false in case the resulting entries quantity is undefined (which can happen only in exceptional situations) ', () => {
+      mockSuccessEvent.quantityAdded = 1;
+      entry.quantity = undefined;
+      expect(listener['calculateEntryWasMerged'](mockSuccessEvent)).toBe(false);
+    });
+  });
 });

feature-libs/cart/base/components/added-to-cart-dialog/added-to-cart-dialog-event.listener.ts

@@ -4,21 +4,24 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import { Injectable, OnDestroy } from '@angular/core';
+import { Injectable, OnDestroy, inject } from '@angular/core';
 import {
   CartAddEntryFailEvent,
+  CartAddEntrySuccessEvent,
   CartUiEventAddToCart,
 } from '@spartacus/cart/base/root';
-import { EventService } from '@spartacus/core';
-import { LaunchDialogService, LAUNCH_CALLER } from '@spartacus/storefront';
+import { EventService, FeatureConfigService } from '@spartacus/core';
+import { LAUNCH_CALLER, LaunchDialogService } from '@spartacus/storefront';
 import { Subscription } from 'rxjs';
 import { take } from 'rxjs/operators';
+import { AddedToCartDialogComponentData } from './added-to-cart-dialog.component';
 
 @Injectable({
   providedIn: 'root',
 })
 export class AddedToCartDialogEventListener implements OnDestroy {
   protected subscription = new Subscription();
+  private featureConfig = inject(FeatureConfigService);
 
   constructor(
     protected eventService: EventService,
@@ -28,21 +31,39 @@ export class AddedToCartDialogEventListener implements OnDestroy {
   }
 
   protected onAddToCart() {
-    this.subscription.add(
-      this.eventService.get(CartUiEventAddToCart).subscribe((event) => {
-        this.openModal(event);
-      })
-    );
-
-    this.subscription.add(
-      this.eventService.get(CartAddEntryFailEvent).subscribe((event) => {
-        this.closeModal(event);
-      })
-    );
+    if (
+      this.featureConfig.isEnabled('adddedToCartDialogDrivenBySuccessEvent')
+    ) {
+      this.subscription.add(
+        this.eventService
+          .get(CartAddEntrySuccessEvent)
+          .subscribe((successEvent) => {
+            this.openModalAfterSuccess(successEvent);
+          })
+      );
+    } else {
+      this.subscription.add(
+        this.eventService.get(CartUiEventAddToCart).subscribe((event) => {
+          this.openModal(event);
+        })
+      );
+      this.subscription.add(
+        this.eventService.get(CartAddEntryFailEvent).subscribe((event) => {
+          this.closeModal(event);
+        })
+      );
+    }
   }
-
+  /**
+   * @deprecated since 2211.24. With activation of feature toggle 'adddedToCartDialogDrivenBySuccessEvent'
+   * the method is no longer called, instead method openModalAfterSuccess takes care of launching
+   * the addedToCart dialogue.
+   *
+   * Opens modal based on CartUiEventAddToCart.
+   * @param event Signals that a product has been added to the cart.
+   */
   protected openModal(event: CartUiEventAddToCart): void {
-    const addToCartData = {
+    const addToCartData: AddedToCartDialogComponentData = {
       productCode: event.productCode,
       quantity: event.quantity,
       numberOfEntriesBeforeAdd: event.numberOfEntriesBeforeAdd,
@@ -61,6 +82,45 @@ export class AddedToCartDialogEventListener implements OnDestroy {
     }
   }
 
+  /**
+   * Opens modal after addToCart has been successfully performed.
+   * @param successEvent Signals that the backend call succeeded.
+   */
+  protected openModalAfterSuccess(
+    successEvent: CartAddEntrySuccessEvent
+  ): void {
+    const addToCartData: AddedToCartDialogComponentData = {
+      productCode: successEvent.productCode,
+      quantity: successEvent.quantity,
+      pickupStoreName: successEvent.pickupStore,
+      addedEntryWasMerged: this.calculateEntryWasMerged(successEvent),
+    };
+
+    const dialog = this.launchDialogService.openDialog(
+      LAUNCH_CALLER.ADDED_TO_CART,
+      undefined,
+      undefined,
+      addToCartData
+    );
+
+    if (dialog) {
+      dialog.pipe(take(1)).subscribe();
+    }
+  }
+
+  protected calculateEntryWasMerged(
+    successEvent: CartAddEntrySuccessEvent
+  ): boolean {
+    // In case quantityAdded zero, the system could have run into stock issues.
+    // Then we continue and still launch the dialogue in 'merge' mode in order
+    // to not break the existing behaviour.
+    const quantityAdded = successEvent.quantityAdded ?? 0;
+    return (
+      quantityAdded === 0 ||
+      (successEvent.entry?.quantity ?? 0) - quantityAdded > 0
+    );
+  }
+
   protected closeModal(reason?: any): void {
     this.launchDialogService.closeDialog(reason);
   }