From ab7c7c1f647a897b0083e302cdf2327ac48161af Mon Sep 17 00:00:00 2001 From: sudacode Date: Tue, 7 Jul 2026 00:55:14 -0700 Subject: [PATCH] test(launcher): drop bogus X11 DISPLAY from youtube playback test The youtube playback test forced DISPLAY=:99 and XDG_SESSION_TYPE=x11, which on headless Linux CI made the overlay's X11 window tracker block on a nonexistent X server until the launcher timed out. The test only asserts the URL reaches mpv, so the X override is unnecessary; x11 backend selection is already covered by launcher/smoke.e2e.test.ts. --- launcher/main.test.ts | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/launcher/main.test.ts b/launcher/main.test.ts index 1a014722..63debff6 100644 --- a/launcher/main.test.ts +++ b/launcher/main.test.ts @@ -595,8 +595,10 @@ ${bunBinary} -e "const net=require('node:net'); const fs=require('node:fs'); con ...makeTestEnv(homeDir, xdgConfigHome), PATH: `${binDir}${path.delimiter}${process.env.Path || process.env.PATH || ''}`, Path: `${binDir}${path.delimiter}${process.env.Path || process.env.PATH || ''}`, - DISPLAY: ':99', - XDG_SESSION_TYPE: 'x11', + // Don't force an X11 session with a bogus DISPLAY: headless CI has no X + // server at :99, so the overlay's X11 window tracker would block until the + // launcher times out. This test only checks the URL reaches mpv; the x11 + // backend path is covered by launcher/smoke.e2e.test.ts via `--backend x11`. SUBMINER_APPIMAGE_PATH: appPath, SUBMINER_TEST_MPV_ARGS: mpvArgsPath, SUBMINER_TEST_CAPTURE: path.join(root, 'captured-args.txt'),