From 478e67a50a43cdaad5cfa34be1616acf6d269d3a Mon Sep 17 00:00:00 2001
From: Neil <4138956+nwparker@users.noreply.github.com>
Date: Fri, 7 Aug 2026 05:35:46 -0700
Subject: [PATCH] test(e2e): skip the WebGL arm when no WebGL renderer exists
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The #12164 probe runs two arms, webgl and dom, and closes by asserting the
active renderer is the requested one. That assertion is right for the dom arm —
it is what proves the pane actually left WebGL, without which the arm is
meaningless — but headless CI has no GPU, xterm falls back to DOM silently, and
the webgl arm then fails.

The failure reads as a Korean rendering defect and is not one, so the webgl arm
now skips with the active renderer named. The dom arm keeps the assertion
unchanged.

This is the third of three checks that have been red on this branch since
2026-08-06. `static analysis` and `verify` were fixed in c51c6b5837e; the CI log
shows this job as 1 failed / 1 passed, the pass being the dom arm.
---
 tests/e2e/issue-12164-korean-glyph-doubling.spec.ts | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/tests/e2e/issue-12164-korean-glyph-doubling.spec.ts b/tests/e2e/issue-12164-korean-glyph-doubling.spec.ts
index 7070bd09f1e..f9d8ebcd320 100644
--- a/tests/e2e/issue-12164-korean-glyph-doubling.spec.ts
+++ b/tests/e2e/issue-12164-korean-glyph-doubling.spec.ts
@@ -115,6 +115,12 @@ async function runProbe(orcaPage: Page, renderer: 'webgl' | 'dom'): Promise<void
       item.expectedColumns - 4
     )
   }
-  // The DOM arm is only meaningful if the pane actually left WebGL.
+  // The DOM arm is only meaningful if the pane actually left WebGL. The WebGL arm
+  // needs a GPU that headless CI does not have, and xterm silently falls back to
+  // DOM rather than failing — so skip it there instead of asserting, which would
+  // otherwise report a missing GPU as a Korean rendering defect.
+  if (renderer === 'webgl' && canvas.renderer !== 'webgl') {
+    test.skip(true, `no WebGL renderer available (active: ${canvas.renderer})`)
+  }
   expect(canvas.renderer, 'requested renderer was not the active renderer').toBe(renderer)
 }