diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-15 16:55:18 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-15 16:55:18 -0500 |
| commit | 2579f738f77332861f47b1685074fa42381ef09c (patch) | |
| tree | 8cee4c443c8c8efa065366c77562880ac41ee835 /modules/diff-config.el | |
| parent | ce96fc954d2f7206b74ec55470c778fc9ff40206 (diff) | |
| download | dotemacs-2579f738f77332861f47b1685074fa42381ef09c.tar.gz dotemacs-2579f738f77332861f47b1685074fa42381ef09c.zip | |
test(recording): skip integration tests when screencast access fails
The three integration tests in test-video-audio-recording-process-cleanup spawn wf-recorder via cj/ffmpeg-record-video and assert on pgrep counts. They guard with executable-find and XDG_SESSION_TYPE checks, but neither catches the case where the subprocess can run wf-recorder yet lacks Wayland screencast permission. wf-recorder picks a region, retries "Failed to copy frame" 17 times, then exits with code 183 inside a second. The assertion fires against an empty pgrep.
I added test-cleanup--can-capture-frames, which calls cj/ffmpeg-record-video against a temp dir, waits 1s, and checks pgrep. If wf-recorder didn't survive, the three integration tests skip. The result is cached, so the ~2.5s cost is paid once per batch.
I added the same guard to test-integration-video-recording-multiple-start-stop-cycles. Its assertion is (= count initial-count), so it trivially passed in any environment where capture didn't work. Skipping is more honest than passing for the wrong reason.
Diffstat (limited to 'modules/diff-config.el')
0 files changed, 0 insertions, 0 deletions
