External/buzz
1
0
Fork 0
mirror of https://github.com/block/buzz.git synced 2026-08-18 06:50:31 +02:00
Code Issues Packages Projects Releases Wiki Activity

test(e2e): align permission outcome assertion with label-based rendering

Browse Source 
Test 03 asserted /Approved.*allow_once/ — the old format that rendered
the raw ACP kind string. describePermissionOutcome now returns the
harness-provided label (or verb-only fallback); the legacy non-ask path
has no label map, so the rendered text is 'Approved'.

Production code is untouched; this test file is the only change.

Co-authored-by: Will Pfleger <pfleger.will@gmail.com>
Signed-off-by: Will Pfleger <pfleger.will@gmail.com>
This commit is contained in:
Hayt
2026-08-08 17:59:53 -04:00
co-authored by Will Pfleger
parent f858cabb26
commit 60c247e20e
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
desktop/tests/e2e/observer-feed-screenshots.spec.ts
+4 -2
View File
@@ -275,8 +275,10 @@ test.describe("observer feed screenshots", () => {
},
]);
// The permission row should show the "Approved (allow_once)" outcome.
await expect(feedPanel.getByText(/Approved.*allow_once/)).toBeVisible({
// The permission row shows the harness-provided option label ("Allow once"),
// not the raw ACP kind. The legacy non-ask path has no label map, so it
// falls back to the verb-only form: "Approved".
await expect(feedPanel.getByText("Approved")).toBeVisible({
timeout: 5_000,
});
await settleAnimations(feedPanel);