Skip to content

Ignore the shell frame shadow in CodeMiningTest pixel scans - #4483

Merged
vogella merged 1 commit into
eclipse-platform:masterfrom
vogella:lv/codemining-test-xvfb
Oct 13, 2026
Merged

vogella merged 1 commit into
eclipse-platform:masterfrom
vogella:lv/codemining-test-xvfb

Conversation

@vogella

@vogella vogella commented Oct 12, 2026

Copy link
Copy Markdown
Contributor

Without a window manager, for example under Xvfb, GTK paints a two pixel frame shadow along the right edge of the shell. The pixel scans in CodeMiningTest covered the full widget width and mistook that shadow for a rendered code mining, which made testCodeMiningMultiLine, testCodeMiningEndOfLine and testMultiLineHeaderCodeMining fail headless. The scans now stop a few pixels short of the right edge, so the tests pass both under Xvfb and on a real desktop while still detecting actual minings.

Without a window manager, as under Xvfb, GTK paints a two pixel shadow
along the right shell edge, which the full-width pixel scans mistook for
rendered code minings.

Assisted-by: multiple AI agents and layers of automated tooling 🤖
@github-actions

Copy link
Copy Markdown
Contributor

Test Results

   867 files  ±0     867 suites  ±0   1h 0m 18s ⏱️ + 13m 9s
 8 438 tests ±0   8 196 ✅ ±0  242 💤 ±0  0 ❌ ±0 
21 189 runs  ±0  20 506 ✅ ±0  683 💤 ±0  0 ❌ ±0 

Results for commit 31760b0. ± Comparison against base commit 9d4d42b.

@vogella

vogella commented Oct 13, 2026

Copy link
Copy Markdown
Contributor Author

Allows to run this test in a virtual x-server without the last pixels ignore it fails locally.

@vogella
vogella merged commit 743ec34 into eclipse-platform:master Oct 13, 2026
18 checks passed
@vogella
vogella deleted the lv/codemining-test-xvfb branch October 13, 2026 10:56
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.

1 participant