Skip to content

Rewrite "New Wizard" tests to no longer require the SWTBot runner#1330

Merged
ptziegler merged 1 commit intoeclipse-windowbuilder:masterfrom
ptziegler:rewrite-swtbot-test
Jan 25, 2026
Merged

Rewrite "New Wizard" tests to no longer require the SWTBot runner#1330
ptziegler merged 1 commit intoeclipse-windowbuilder:masterfrom
ptziegler:rewrite-swtbot-test

Conversation

@ptziegler
Copy link
Contributor

Newer Eclipse versions no longer launch a workspace when running a test outside the UI thread. Because this is a requirement for all of our SWTBot tests, we can no longer run them locally.

The tests were originally written like that because they open a modal dialog and would thus lead to a deadlock if the test is also executed within the UI thread. But thanks to the changes to the UiContext, we have a way to make this work regardless.

@ptziegler ptziegler added this to the 1.23.0 milestone Dec 21, 2025
@github-actions
Copy link

github-actions bot commented Dec 21, 2025

Test Results

 2 720 files   - 8   2 720 suites   - 8   10h 28m 6s ⏱️ - 23m 50s
 7 018 tests ±0   6 960 ✅ ±0   58 💤 ±0  0 ❌ ±0 
28 072 runs  ±0  27 830 ✅ ±0  242 💤 ±0  0 ❌ ±0 

Results for commit 5be1ad1. ± Comparison against base commit d2c7a71.

♻️ This comment has been updated with latest results.

@ptziegler ptziegler force-pushed the rewrite-swtbot-test branch 16 times, most recently from e9feebe to 25aa318 Compare December 24, 2025 00:46
@ptziegler ptziegler force-pushed the rewrite-swtbot-test branch 5 times, most recently from 2a038ac to fddbde8 Compare January 10, 2026 13:45
Newer Eclipse versions no longer launch a workspace when running a test
outside the UI thread. Because this is a requirement for all of our
SWTBot tests, we can no longer run them locally.

The tests were originally written like that because they open a modal
dialog and would thus lead to a deadlock if the test is also executed
within the UI thread. But thanks to the changes to the `UiContext`, we
have a way to make this work regardless.
@ptziegler ptziegler force-pushed the rewrite-swtbot-test branch from fddbde8 to 5be1ad1 Compare January 25, 2026 08:45
@ptziegler ptziegler merged commit 941c788 into eclipse-windowbuilder:master Jan 25, 2026
12 of 13 checks passed
@ptziegler ptziegler deleted the rewrite-swtbot-test branch January 25, 2026 10:38
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