From df818767d44ac18a991f6c22c12acb1093c39cfb Mon Sep 17 00:00:00 2001 From: beatz174-bit Date: Mon, 24 Nov 2025 08:09:49 +1000 Subject: [PATCH 1/4] Add e2e test for deleting pick list items --- e2e/picklist.spec.ts | 32 ++++++++++++++++++++++++++++++++ 1 file changed, 32 insertions(+) diff --git a/e2e/picklist.spec.ts b/e2e/picklist.spec.ts index 066f6f6..292f8a8 100644 --- a/e2e/picklist.spec.ts +++ b/e2e/picklist.spec.ts @@ -53,4 +53,36 @@ test.describe('Active pick list', () => { await expect(page.getByText('Playwright Cola Zero')).toBeVisible(); }); + + test('removes a product from the pick list when deleted', async ({ page }) => { + await page.goto('/'); + + await expect(page.getByRole('heading', { name: 'StockFill' })).toBeVisible(); + await page.getByRole('link', { name: 'Create Pick List' }).click(); + await page.getByLabel('Area').click(); + await page.getByRole('option', { name: areaName }).first().click(); + await page.getByRole('button', { name: 'Save Pick List' }).click(); + + await expect(page.getByRole('heading', { name: `${areaName} List` })).toBeVisible(); + + const searchInput = page.getByPlaceholder('Search products'); + await searchInput.click(); + await searchInput.fill(additionalProduct); + await page + .getByRole('option', { name: new RegExp(`${additionalProduct} \\(${areaName}\\)`, 'i') }) + .first() + .click(); + + const productRow = page.getByText(additionalProduct).locator( + 'xpath=ancestor::div[contains(@class, "MuiStack-root")]//button[@aria-label="Delete item"]', + ); + + await expect(page.getByText(additionalProduct).first()).toBeVisible(); + await productRow.first().click(); + + await expect(page.getByRole('dialog', { name: 'Delete item' })).toBeVisible(); + await page.getByRole('button', { name: 'Delete', exact: true }).click(); + + await expect(page.getByText(additionalProduct).first()).not.toBeVisible(); + }); }); From 185be8f7bb58adea51651a963973dd29dfe58fa8 Mon Sep 17 00:00:00 2001 From: beatz174-bit Date: Mon, 24 Nov 2025 08:10:11 +1000 Subject: [PATCH 2/4] Add e2e test for editing pick lists --- e2e/picklist.spec.ts | 32 ++++++++++++++++++++++++++++++++ 1 file changed, 32 insertions(+) diff --git a/e2e/picklist.spec.ts b/e2e/picklist.spec.ts index 066f6f6..e493007 100644 --- a/e2e/picklist.spec.ts +++ b/e2e/picklist.spec.ts @@ -27,6 +27,38 @@ test.describe('Active pick list', () => { await expect(page.getByRole('button', { name: 'Save and Return' })).toBeEnabled(); }); + test('adds a product to an existing pick list from the search input', async ({ page }) => { + await page.goto('/'); + + await expect(page.getByRole('heading', { name: 'StockFill' })).toBeVisible(); + await page.getByRole('link', { name: 'Create Pick List' }).click(); + await page.getByLabel('Area').click(); + await page.getByRole('option', { name: areaName }).first().click(); + await page.getByRole('button', { name: 'Save Pick List' }).click(); + + await expect(page.getByRole('heading', { name: `${areaName} List` })).toBeVisible(); + + await page.getByRole('button', { name: 'Save and Return' }).click(); + + await expect(page.getByRole('heading', { name: 'Pick Lists' })).toBeVisible(); + await page.getByRole('link', { name: new RegExp(areaName, 'i') }).click(); + + await expect(page.getByRole('heading', { name: `${areaName} List` })).toBeVisible(); + + const searchInput = page.getByPlaceholder('Search products'); + await searchInput.click(); + await searchInput.fill(additionalProduct); + + const productOption = page.getByRole('option', { + name: new RegExp(`${additionalProduct} \\(${areaName}\\)`, 'i'), + }); + await expect(productOption).toBeVisible(); + await productOption.click(); + + await expect(page.getByText(additionalProduct).first()).toBeVisible(); + await expect(page.getByText(/Qty:\s*1\s+unit/i)).toBeVisible(); + }); + test('allows adding and editing products from the manage products screen', async ({ page }) => { await page.goto('/'); From 789212951e021c38ea2ca8f9151d3f9316c40337 Mon Sep 17 00:00:00 2001 From: beatz174-bit Date: Mon, 24 Nov 2025 08:10:35 +1000 Subject: [PATCH 3/4] Add e2e test for toggling picked status --- e2e/picklist.spec.ts | 33 +++++++++++++++++++++++++++++++++ 1 file changed, 33 insertions(+) diff --git a/e2e/picklist.spec.ts b/e2e/picklist.spec.ts index 066f6f6..c3290a3 100644 --- a/e2e/picklist.spec.ts +++ b/e2e/picklist.spec.ts @@ -53,4 +53,37 @@ test.describe('Active pick list', () => { await expect(page.getByText('Playwright Cola Zero')).toBeVisible(); }); + + test('lets a user mark an item as picked then revert it back to pending', async ({ page }) => { + await page.goto('/'); + + await page.getByRole('link', { name: 'Create Pick List' }).click(); + await page.getByLabel('Area').click(); + await page.getByRole('option', { name: areaName }).first().click(); + await page.getByRole('button', { name: 'Save Pick List' }).click(); + + const searchInput = page.getByPlaceholder('Search products'); + await searchInput.click(); + await searchInput.fill(additionalProduct); + await page + .getByRole('option', { name: new RegExp(`${additionalProduct} \\(${areaName}\\)`, 'i') }) + .first() + .click(); + + const itemStatusToggle = page.getByLabel('Toggle picked status').first(); + const showPickedToggle = page.getByLabel('Show picked'); + + await expect(itemStatusToggle).not.toBeChecked(); + await expect(showPickedToggle).toBeEnabled(); + + await itemStatusToggle.check(); + + await expect(itemStatusToggle).toBeChecked(); + await expect(showPickedToggle).toBeDisabled(); + + await itemStatusToggle.uncheck(); + + await expect(itemStatusToggle).not.toBeChecked(); + await expect(showPickedToggle).toBeEnabled(); + }); }); From 962b80bed38c6667f5e12b49ee7e1e34bd9c76c5 Mon Sep 17 00:00:00 2001 From: beatz174-bit Date: Mon, 24 Nov 2025 08:10:54 +1000 Subject: [PATCH 4/4] Add e2e coverage for packaging toggle --- e2e/picklist.spec.ts | 46 ++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 46 insertions(+) diff --git a/e2e/picklist.spec.ts b/e2e/picklist.spec.ts index 066f6f6..f2a0619 100644 --- a/e2e/picklist.spec.ts +++ b/e2e/picklist.spec.ts @@ -53,4 +53,50 @@ test.describe('Active pick list', () => { await expect(page.getByText('Playwright Cola Zero')).toBeVisible(); }); + + test('toggles packaging type and persists the selection', async ({ page }) => { + await page.goto('/'); + + await page.getByRole('link', { name: 'Create Pick List' }).click(); + await page.getByLabel('Area').click(); + await page.getByRole('option', { name: areaName }).first().click(); + await page.getByRole('button', { name: 'Save Pick List' }).click(); + + await expect(page.getByRole('heading', { name: `${areaName} List` })).toBeVisible(); + + const searchInput = page.getByPlaceholder('Search products'); + await searchInput.click(); + await searchInput.fill(additionalProduct); + await page + .getByRole('option', { name: new RegExp(`${additionalProduct} \\(${areaName}\\)`, 'i') }) + .first() + .click(); + + const increaseButton = page.getByLabel('Increase quantity'); + await increaseButton.click(); + await expect(page.getByText('Qty: 2 unit')).toBeVisible(); + + await page.getByLabel('Switch to carton packaging').click(); + await expect(page.getByText('Qty: 2 carton')).toBeVisible(); + + await page.getByLabel('Decrease quantity').click(); + await expect(page.getByText('Qty: 1 carton')).toBeVisible(); + + await page.getByLabel('Switch to unit packaging').click(); + await expect(page.getByText('Qty: 1 unit')).toBeVisible(); + + await increaseButton.click(); + await increaseButton.click(); + await page.getByLabel('Switch to carton packaging').click(); + await expect(page.getByText('Qty: 3 carton')).toBeVisible(); + + await page.getByRole('button', { name: 'Save and Return' }).click(); + await expect(page.getByRole('heading', { name: 'Pick Lists' })).toBeVisible(); + + await page.reload(); + await page.getByRole('link', { name: areaName }).first().click(); + await expect(page.getByRole('heading', { name: `${areaName} List` })).toBeVisible(); + await expect(page.getByText('Qty: 3 carton')).toBeVisible(); + await expect(page.getByLabel('Switch to unit packaging')).toBeVisible(); + }); });