commit c8e1092180492a7bf6f73d700929f9636653f52a
parent 83b3c508e4d9c517b8f6c726122ee53168a167d5
Author: I Mean I'm Just Saying <imeanimjustsaying@kiwifarms.st>
Date: Fri, 28 Aug 2026 03:26:24 -0400
editor: slice 4 e2e fix — channels-sort's header list names columns that exist
`channels-sort.spec.ts` asserted `aria-sort="none"` on nine headers, two of
which — "Downloads" and "Transcripts" — stopped existing when the per-operation
pipeline strip landed (`1184ba9`): those columns are labelled with each
operation's `shortLabel` ("Download", "Transcribe", "Digest", …) and are derived
from the operations enabled in settings. The locator has matched nothing since,
and the spec was already red at `626a0d0` — this is not slice 4's break, but
slice 4's suite run is where it surfaced.
The loop now covers the eight FIXED columns, which includes the new "Report",
and says why the pipeline columns are not in a literal list.
Nothing under transcripts/ was read or written for this commit.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Diffstat:
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/editor/e2e/channels-sort.spec.ts b/editor/e2e/channels-sort.spec.ts
@@ -13,6 +13,14 @@ test("clicking column headers sorts the channels table and indicates direction",
await page.goto("/channels");
// Default: no header carries an active aria-sort.
+ //
+ // THE FIXED COLUMNS ONLY. The pipeline columns to the right of Report are
+ // derived from the operations enabled in settings and are labelled with each
+ // operation's `shortLabel` ("Download", "Transcribe", "Digest", …), so a
+ // literal list of them here is a list of whatever the fixture's settings
+ // happened to enable. (This loop used to name "Downloads" and "Transcripts",
+ // which were the header labels before the per-operation strip landed; they
+ // have matched nothing since.)
for (const header of [
"Slug",
"Name",
@@ -21,8 +29,7 @@ test("clicking column headers sorts the channels table and indicates direction",
"Sync",
"Playlist",
"Last sync",
- "Downloads",
- "Transcripts",
+ "Report",
]) {
const th = page
.getByRole("columnheader")