Skip to content

test(x11): cover INCR clipboard boundaries - #530

Merged
xxx merged 1 commit into
masterfrom
mpd/test-x11-incr-clipboard
Aug 25, 2026
Merged

test(x11): cover INCR clipboard boundaries#530
xxx merged 1 commit into
masterfrom
mpd/test-x11-incr-clipboard

Conversation

@xxx

@xxx xxx commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Closes #375.

Adds X11 integration coverage for both unsupported incremental-transfer boundaries:

  • a foreign owner returning an INCR property is rejected explicitly rather than read as truncated text
  • text larger than Xvfb's 16 MiB BIG-REQUESTS ceiling is refused rather than leaving the requestor unanswered or returning partial data

The investigation also corrected the issue's approximate 256 KiB boundary: Xvfb accepts a 1 MiB one-shot property write through BIG-REQUESTS, so the outbound test uses 20 MiB to cross the actual server limit.

Verification:

  • both new ignored X11 integration tests directly
  • cargo test --workspace
  • cargo clippy --workspace --all-targets -- -D warnings

@xxx
xxx marked this pull request as ready for review August 25, 2026 22:27
@xxx
xxx merged commit 5c3f572 into master Aug 25, 2026
17 checks passed
@xxx
xxx deleted the mpd/test-x11-incr-clipboard branch August 25, 2026 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

glass-x11: app_exited_during_discovery, INCR and from_env have no test in either suite

1 participant