From df818767d44ac18a991f6c22c12acb1093c39cfb Mon Sep 17 00:00:00 2001 From: beatz174-bit Date: Mon, 24 Nov 2025 08:09:49 +1000 Subject: [PATCH 1/2] 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/2] 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('/');