From c3eed03998ca1a6d80f3149231b0f380a61b7b60 Mon Sep 17 00:00:00 2001
From: Kat <00-kat@proton.me>
Date: Mon, 23 Mar 2026 00:44:29 +1100
Subject: [PATCH] cmd/pkgserver: aria-describe test node summary with state

The summary marker does not appear in the AOM, so setting its alt text
is fruitless.
---
 cmd/pkgserver/ui/static/test.ts | 13 ++++++++-----
 cmd/pkgserver/ui/test.html      |  4 ++++
 2 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/cmd/pkgserver/ui/static/test.ts b/cmd/pkgserver/ui/static/test.ts
index 634d0e48..a8394c5b 100644
--- a/cmd/pkgserver/ui/static/test.ts
+++ b/cmd/pkgserver/ui/static/test.ts
@@ -294,12 +294,14 @@ export class DOMReporter implements Reporter {
         let parent = document.getElementById("root");
         for (const node of path) {
             let child: HTMLDetailsElement;
+            let summary: HTMLElement;
             outer: for (const d of parent.children) {
                 if (!(d instanceof HTMLDetailsElement)) continue;
                 for (const s of d.children) {
                     if (!(s instanceof HTMLElement)) continue;
                     if (!(s.tagName === "SUMMARY" && s.innerText === node)) continue;
                     child = d;
+                    summary = s;
                     break outer;
                 }
             }
@@ -307,7 +309,7 @@ export class DOMReporter implements Reporter {
                 child = document.createElement("details");
                 child.className = "test-node";
                 child.ariaRoleDescription = "test";
-                const summary = document.createElement("summary");
+                summary = document.createElement("summary");
                 summary.appendChild(document.createTextNode(node));
                 summary.ariaRoleDescription = "test name";
                 child.appendChild(summary);
@@ -320,17 +322,18 @@ export class DOMReporter implements Reporter {
                 child.classList.add("failure");
                 child.classList.remove("skip");
                 child.classList.remove("success");
+                summary.setAttribute("aria-labelledby", "failure-description");
                 break;
             case "skip":
                 if (child.classList.contains("failure")) break;
                 child.classList.add("skip");
                 child.classList.remove("success");
+                summary.setAttribute("aria-labelledby", "skip-description");
                 break;
             case "success":
-                if (!(child.classList.contains("failure") ||
-                      child.classList.contains("skip"))) {
-                    child.classList.add("success");
-                }
+                if (child.classList.contains("failure") || child.classList.contains("skip")) break;
+                child.classList.add("success");
+                summary.setAttribute("aria-labelledby", "success-description");
                 break;
             }
 
diff --git a/cmd/pkgserver/ui/test.html b/cmd/pkgserver/ui/test.html
index ade2c20b..4e0f5399 100644
--- a/cmd/pkgserver/ui/test.html
+++ b/cmd/pkgserver/ui/test.html
@@ -15,6 +15,10 @@
     failed<span id="skip-counter-text" class="hidden">, <span id="skip-counter">0</span> skipped</span>.
 </p>
 
+<p hidden id="success-description">Successful test</p>
+<p hidden id="failure-description">Failed test</p>
+<p hidden id="skip-description">Partially or fully skipped test</p>
+
 <div id="root">
 </div>