From 43f4ff4183973a36876f8ea29e1a6e39792eaad4 Mon Sep 17 00:00:00 2001 From: Ettore Di Giacinto Date: Tue, 20 Oct 2026 16:12:42 +0000 Subject: [PATCH] test(ui): drop Settings from the 404 places Settings is a tab of the Operate hub and not a destination of its own, so the 404 page does not list it. Assert that it is absent. Assisted-by: Claude Code:claude-sonnet-5-5 playwright Signed-off-by: Ettore Di Giacinto --- core/http/react-ui/e2e/not-found.spec.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/core/http/react-ui/e2e/not-found.spec.js b/core/http/react-ui/e2e/not-found.spec.js index 370f12130..2ee6f85a6 100644 --- a/core/http/react-ui/e2e/not-found.spec.js +++ b/core/http/react-ui/e2e/not-found.spec.js @@ -9,9 +9,11 @@ test.describe('404', () => { await expect(nf).toContainText('/app/no-such-page') await expect(nf.getByRole('link', { name: 'Go home' })).toHaveAttribute('href', '/app') const places = nf.getByRole('navigation', { name: 'Where to go' }) - for (const name of ['Home', 'Models', 'Chat', 'Add-ons', 'Settings', 'Build', 'Operate']) { + for (const name of ['Home', 'Models', 'Chat', 'Add-ons', 'Build', 'Operate']) { await expect(places.getByRole('link', { name })).toBeVisible() } + // Settings is a tab of the Operate hub, so the places do not list it. + await expect(places.getByRole('link', { name: 'Settings' })).toHaveCount(0) }) test('has no search, because the app has none to offer', async ({ page }) => {