From 2f46931ad59ffb546f3e1f82dea542d18a63d7c2 Mon Sep 17 00:00:00 2001 From: Ettore Di Giacinto Date: Wed, 21 Oct 2026 00:59:54 +0000 Subject: [PATCH] test(ui): fix the Home answers in the stale pin spec The spec asserts that the page shows no alert. The Home page raises its own alert when the model list fails or a staging operation has failed, and both answers come from the shared test server. A slow reply or a failed install left there counted as an alert about the pin. The spec now fixes both answers, so the alert count tells only about the pin. The assertions are unchanged. Assisted-by: Claude Code:claude-sonnet-5-5 node Signed-off-by: Ettore Di Giacinto --- core/http/react-ui/e2e/addon-navigation.spec.js | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/core/http/react-ui/e2e/addon-navigation.spec.js b/core/http/react-ui/e2e/addon-navigation.spec.js index cfb6eea95..9eda5b19b 100644 --- a/core/http/react-ui/e2e/addon-navigation.spec.js +++ b/core/http/react-ui/e2e/addon-navigation.spec.js @@ -160,6 +160,12 @@ test.describe('Add-on navigation', () => { test('a pin of an add-on that is no longer listed is not drawn and shows no error', async ({ page }) => { await setup(page, { pins: ['studio', 'removed-addon'] }) + // The Home page raises its own alert when the model list or a staging + // operation fails. Both come from the shared test server, so a slow answer + // or a failed install left there would count as an alert about the pin. + // Fix their answers so the alert count tells only about the pin. + await page.route('**/v1/models', route => route.fulfill({ json: { object: 'list', data: [] } })) + await page.route('**/api/operations', route => route.fulfill({ json: { operations: [] } })) // Hold the Apps list back so the assertions run after it has arrived and // been drawn from, not before, when a missing pin proves nothing. await page.route(/\/api\/addons\/apps$/, async route => {