Commit 580cfee
Fix UITestUtil.processEventsUntil causing test failures in RCPTestWorkbenchAdvisor
The previous implementation called UITestUtil.processEventsUntil() which
internally calls PlatformUI.getWorkbench().getDisplay(). However, during
postStartup(), the workbench display may not be reliably accessible via
PlatformUI.getWorkbench(), causing the test to fail.
This change replaces the UITestUtil calls with direct Display.getCurrent()
usage and implements the same event processing logic inline. This ensures
that:
1. We wait for async/sync operations with DisplayAccess to complete
2. We process remaining events to detect any regressions
3. We don't rely on PlatformUI.getWorkbench() during startup
Fixes the following test failures:
- Async run during startup
- Sync from un-qualified thread ran during startup
- Async from un-qualified thread ran during startup
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>1 parent 7ae06e6 commit 580cfee
File tree
1 file changed
+23
-4
lines changed- tests/org.eclipse.ui.tests.harness/src/org/eclipse/ui/tests/harness/util
1 file changed
+23
-4
lines changedLines changed: 23 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
234 | 234 | | |
235 | 235 | | |
236 | 236 | | |
237 | | - | |
238 | | - | |
239 | | - | |
240 | | - | |
| 237 | + | |
| 238 | + | |
| 239 | + | |
| 240 | + | |
| 241 | + | |
| 242 | + | |
| 243 | + | |
| 244 | + | |
| 245 | + | |
| 246 | + | |
| 247 | + | |
| 248 | + | |
| 249 | + | |
| 250 | + | |
| 251 | + | |
| 252 | + | |
| 253 | + | |
| 254 | + | |
| 255 | + | |
| 256 | + | |
| 257 | + | |
| 258 | + | |
| 259 | + | |
241 | 260 | | |
242 | 261 | | |
243 | 262 | | |
| |||
0 commit comments