From 96abb4808622f770b69e712e47670327fc5bcf20 Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Thu, 6 Aug 2026 21:20:18 -0400
Subject: [PATCH] =?UTF-8?q?test(tuning):=20query=20the=20window/last-week?=
 =?UTF-8?q?=20headers=20as=20column=20headers=20=E2=80=94=20#2495?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

getByText(/Plays/) matched my own new caption as well as the header, since
the caption explains which columns cover the window. Query by columnheader
role instead, which is what the assertion actually means.

Also reordered the caption: prepending the clarification turned it into a
run-on that opened mid-explanation before saying what the chart was.
---
 web/src/routes/admin/tuning/+page.svelte   | 7 +++----
 web/src/routes/admin/tuning/tuning.test.ts | 7 +++++--
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/web/src/routes/admin/tuning/+page.svelte b/web/src/routes/admin/tuning/+page.svelte
index f1b8419b..bececa1f 100644
--- a/web/src/routes/admin/tuning/+page.svelte
+++ b/web/src/routes/admin/tuning/+page.svelte
@@ -428,13 +428,12 @@
     <div>
       <h2 class="text-lg font-semibold">Weekly trends</h2>
       <p class="text-xs text-text-secondary">
-        <span class="font-medium">Skip and completion are the most recent week only</span> —
-        the number after the skip rate is that week's play count, so a rate off a
-        handful of plays reads as what it is. Plays and Taste hit cover the whole
-        window.
         Skip rate per surface over the last {trends?.weeks ?? 12} weeks (lower is better; all
         users aggregated, rates only). Dashed ticks mark tuning changes. Taste hit is the share
         of plays whose artist fits the current taste profile.
+        <span class="font-medium">The skip and completion columns show the most recent week
+        alone</span>, not the whole window — the figure after the skip rate is that week's
+        play count, so a rate drawn from a handful of listens reads as what it is.
       </p>
     </div>
     {#if trendsFailed}
diff --git a/web/src/routes/admin/tuning/tuning.test.ts b/web/src/routes/admin/tuning/tuning.test.ts
index d0e2f63c..00abfa72 100644
--- a/web/src/routes/admin/tuning/tuning.test.ts
+++ b/web/src/routes/admin/tuning/tuning.test.ts
@@ -210,8 +210,11 @@ describe('Admin tuning page', () => {
     // The window/last-week distinction has to be visible in the headers, or the
     // Plays total reads as the denominator of the skip rate. That misreading is
     // what #2495 was filed over.
-    expect(screen.getByText(/Plays/)).toHaveTextContent('(window)');
-    expect(screen.getByText(/^Skip/)).toHaveTextContent('(last wk)');
+    //
+    // Queried as column headers rather than by text: the caption below also
+    // mentions "Plays", and matching on the word finds the prose too.
+    expect(screen.getByRole('columnheader', { name: /Plays/ })).toHaveTextContent('(window)');
+    expect(screen.getByRole('columnheader', { name: /^Skip/ })).toHaveTextContent('(last wk)');
     // The knob turn is listed under the chart AND tooltipped on each
     // sparkline's marker tick, hence getAllBy.
     expect(