From 5d911d40fe4cb2638512ef8214ff8eb2effe39db Mon Sep 17 00:00:00 2001 From: kristianwind Date: Tue, 15 Sep 2026 14:11:26 +0200 Subject: [PATCH] =?UTF-8?q?test(noter):=20vent=20p=C3=A5=20at=20noten=20er?= =?UTF-8?q?=20gemt,=20f=C3=B8r=20den=20n=C3=A6ste=20laves?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Prøven fra i går fejlede i CI på en PR, der kun rørte dokumentation — altså flakkede den, og det er min. Løkken laver tre noter og klikkede "Ny note" igen, uden at vente på at den forrige gemning var kvitteret. Her på maskinen nåede den altid med; på en belastet runner gjorde den ikke, og listen manglede noten. Fem sekunders tålmodighed hjalp ikke, for det var ikke ventetid, der manglede — det var et signal. Der ventes nu på "Gemt". Kørt tre gange i træk grønt. En prøve, der er hurtigere end programmet, måler den maskine, den kører på. Co-Authored-By: Claude Opus 5 (1M context) --- web/e2e/smoke.spec.js | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/web/e2e/smoke.spec.js b/web/e2e/smoke.spec.js index b26785a..453f1ba 100644 --- a/web/e2e/smoke.spec.js +++ b/web/e2e/smoke.spec.js @@ -4656,11 +4656,17 @@ test('en søgning i ⌘K efterlader de træffere, den fandt', async ({ page }) = // Gemt ved at forlade feltet, som resten af fladen gør det — der er ingen // Gem-knap at huske. + // + // Der ventes på "Gemt", før den næste note laves. Uden det klikkes "Ny note", + // mens den forrige stadig er undervejs, og listen når ikke at få den med — det + // fejlede i CI én gang ud af tre kørsler, og bestod hver gang her på maskinen. + // En prøve, der er hurtigere end programmet, måler maskinen den kører på. for (const title of ['Fyrreskov nord', 'Fyrreskov syd', 'Egeskov']) { await page.getByRole('button', { name: 'Ny note' }).click(); const body = page.getByLabel('Notens tekst'); await body.fill(`${title}\n\nnoget om den`); await body.blur(); + await expect(page.getByText('Gemt', { exact: true })).toBeVisible(); await expect(page.locator('.list').getByText(title, { exact: true })).toBeVisible(); }