@designcrowd/library.brand-page 6.2.2-live-preview-1 → 6.2.2

package/src/shared/page-settings/store/page-settings-store.test.ts

@@ -22,6 +22,7 @@ import { setLocaleAsync } from '@src/composables/useTranslate';
 import { usePageNavigationStore } from '@src/shared/page-navigation/store/page-navigation-store';
 import { useShopConfigStore } from '@shared/ecommerce-settings/store/shop-settings-store';
 import { useEventTracker } from '@brand-page-maker/composables/useEventTracker';
+import { useHistoryStore } from '@brand-page-maker/stores/history/history-store';
 import { toPageModels } from '../utils';
 
 vi.mock('../../../clients/brand-page-client', () => ({
@@ -677,6 +678,53 @@ describe('duplicatePage', () => {
     ).toBe(true);
   });
 
+  test("deleting the copy's product leaves the source's product alone", async () => {
+    // The consequence the id rotation exists to prevent. `deleteProduct` filters
+    // the site-wide catalogue BY ID, so a copy that kept the source's ids would
+    // take the original's row down with it - silently, until the next save.
+    const { pageSettingsStore } = await initialiseAsync();
+    const shopConfigStore = useShopConfigStore();
+
+    const source = pageSettingsStore.addNewPage({ title: 'Shop' });
+
+    pageSettingsStore.setPageContents({
+      pageId: source.id,
+      contents: [
+        {
+          id: 'shop-1',
+          type: 'shop-section',
+          settings: {
+            content: {
+              id: 'shop-1',
+              type: 'shop-section',
+              products: [
+                { id: 'product-1', name: 'Latte', variant: [], skus: [] },
+              ],
+              shopPaymentConfig: { paymentConfig: { id: 'stripe-1' } },
+            },
+          },
+        } as unknown as ContentItem<Content>,
+      ],
+    });
+
+    shopConfigStore.initialiseShopStore({
+      paymentConfig: { id: 'stripe-1' } as never,
+      pages: toPageModels(pageSettingsStore.pages) as never,
+    });
+
+    const copy = await pageSettingsStore.duplicatePage({ pageId: source.id });
+
+    const copyProductId = (
+      copy?.contents[0].settings.content as unknown as {
+        products: { id: string }[];
+      }
+    ).products[0].id;
+
+    shopConfigStore.deleteProduct(copyProductId);
+
+    expect(shopConfigStore.products.map((p) => p.id)).toEqual(['product-1']);
+  });
+
   test('adds the copy shop products to the site-wide catalogue', async () => {
     // The catalogue is derived from the shop blocks and is what the save
     // upserts. Without a re-derive the copy's fresh product ids never reach the
@@ -723,6 +771,29 @@ describe('duplicatePage', () => {
     expect(shopConfigStore.paymentConfig?.id).toEqual('stripe-1');
   });
 
+  test('undo removes both halves of a duplicate', async () => {
+    // duplicatePage writes to two stores - the page into page-settings, the
+    // route into page-navigation. A snapshot that caught only one would leave
+    // an orphan behind on undo.
+    const { pageSettingsStore, pageNavigationStore } = await initialiseAsync();
+    const historyStore = useHistoryStore();
+
+    const source = pageSettingsStore.addNewPage({ title: 'About' });
+
+    historyStore.captureBaseline();
+
+    const copy = await pageSettingsStore.duplicatePage({ pageId: source.id });
+
+    historyStore.capture({ source: 'manual_edit', actor: 'user' });
+
+    await historyStore.undo();
+
+    expect(pageSettingsStore.pages.map((p) => p.id)).not.toContain(copy?.id);
+    expect(pageNavigationStore.routes.some((r) => r.pageId === copy?.id)).toBe(
+      false,
+    );
+  });
+
   test('is a no-op for an unknown page id', async () => {
     const { pageSettingsStore } = await initialiseAsync();