fix(noter): en søgning i ⌘K efterlader de træffere, den fandt
Paletten viste de noter, der passede, i sin egen liste — og når man valgte en, åbnede den noten på en side, hvis liste stod helt uændret. De andre træffere var væk, og det eneste sted, de nogensinde havde været, var en popup, der lige var lukket. Feltet i selve notelisten har filtreret hele tiden, og det er prøvet to steder. Men den søgning, man finder, er den store i toppen, og den er palettens. Så det er ikke "søgning i noter virker ikke" — det er, at den søgning, man bruger, ikke efterlod noget. Adressen bærer nu søgningen. ⌘K sender `?note=…&q=…`, notesiden læser begge, og listen ved siden af den åbne note er de andre træffere. Den skrives tilbage, når feltet ændres, så adressen kan gemmes og deles — og frem for alt ikke lyver, når feltet er tømt. To effekter, der deler en markør, og hvorfor de læser den med `untrack`: Den nederste sætter `settled` og kalder `replaceState`. Det vækker den øverste, og på det tidspunkt har `$page.url` endnu ikke skiftet — så den læste den gamle adresse, fandt en søgning, der ikke var enig med markøren, og skrev den tilbage i feltet. Man tømte søgefeltet, og ordet kom igen. Hver effekt skal reagere på sin egen kilde, adressen og feltet, og aldrig på den fælles markør. Den fejl fandt jeg kun, fordi prøven fejlede på den sidste linje og fejlbilledets tilgængelighedstræ stadig viste "Fyrreskov" i feltet. En prøve, der kun havde tjekket adressen, havde været grøn. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Opus 5
parent
54359f8d68
commit
1bbd36c79b
@@ -4635,3 +4635,57 @@ test('et projekt kan deles med en, der allerede har en konto', async ({ browser,
|
||||
|
||||
expect(trouble).toEqual([]);
|
||||
});
|
||||
|
||||
/**
|
||||
* En søgning i ⌘K efterlader resultaterne, den fandt.
|
||||
*
|
||||
* Paletten viste de noter, der passede, i sin egen liste — og når man valgte en,
|
||||
* åbnede den noten på en side, hvis liste stod helt uændret. De andre træffere var
|
||||
* væk, og det eneste sted, de nogensinde havde været, var en popup, der lige var
|
||||
* lukket. Rapporteret sådan her: "jeg får kun præsenteret en popup liste med
|
||||
* resultater, men hvis jeg trykker retur, kommer jeg tilbage til den normale liste
|
||||
* — ikke resultater."
|
||||
*
|
||||
* Prøven laver sine egne noter med et ord, ingen anden note i basen har. Et ord
|
||||
* lånt fra en tidligere prøve ville gøre den her afhængig af rækkefølgen, og den
|
||||
* kobling har allerede narret mig én gang i den her fil.
|
||||
*/
|
||||
test('en søgning i ⌘K efterlader de træffere, den fandt', async ({ page }) => {
|
||||
const trouble = watchForTrouble(page);
|
||||
await page.goto('/noter');
|
||||
|
||||
// Gemt ved at forlade feltet, som resten af fladen gør det — der er ingen
|
||||
// Gem-knap at huske.
|
||||
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.locator('.list').getByText(title, { exact: true })).toBeVisible();
|
||||
}
|
||||
|
||||
await page.goto('/');
|
||||
await page.getByRole('button', { name: /Søg/ }).click();
|
||||
const palette = page.getByRole('dialog');
|
||||
await palette.getByRole('textbox').fill('Fyrreskov');
|
||||
await expect(palette.locator('li button', { hasText: 'Fyrreskov nord' })).toBeVisible();
|
||||
await palette.locator('li button', { hasText: 'Fyrreskov nord' }).click();
|
||||
|
||||
// Noten er åben …
|
||||
await expect(page).toHaveURL(/\/noter\?note=[^&]+&q=Fyrreskov/);
|
||||
await expect(page.getByLabel('Notens tekst')).toContainText('Fyrreskov nord');
|
||||
|
||||
// … og listen ved siden af den er de andre træffere, ikke den normale liste.
|
||||
await expect(page.getByPlaceholder('Søg i noter')).toHaveValue('Fyrreskov');
|
||||
const list = page.locator(".list");
|
||||
await expect(list.getByText('Fyrreskov syd')).toBeVisible();
|
||||
await expect(list.getByText('Egeskov')).toHaveCount(0);
|
||||
|
||||
// Og adressen lyver ikke, når feltet tømmes: uden det ville et genbesøg hente
|
||||
// en søgning tilbage, man selv havde ryddet væk.
|
||||
await page.getByPlaceholder('Søg i noter').fill('');
|
||||
await expect(page).not.toHaveURL(/[?&]q=/);
|
||||
await expect(list.getByText('Egeskov')).toBeVisible();
|
||||
|
||||
expect(trouble).toEqual([]);
|
||||
});
|
||||
|
||||
Reference in New Issue
Block a user