Skip to content

Conversation

@jukzi
Copy link
Contributor

@jukzi jukzi commented Oct 9, 2024

Nightly org.eclipse.ui.tests.UiTestSuite.txt contains intentional RuntimeException that are hard to distinguish from unintentional exceptions.

@jukzi
Copy link
Contributor Author

jukzi commented Oct 9, 2024

see https://download.eclipse.org/eclipse/downloads/drops4/I20241008-1810/testresults/ep434I-unit-win32-java17_win32.win32.x86_64_17/org.eclipse.ui.tests.UiTestSuite.txt

for example

!ENTRY org.eclipse.ui.workbench 4 2 2024-10-09 09:19:17.245
!MESSAGE Problems occurred when invoking code from plug-in: "org.eclipse.ui.workbench".
!STACK 0
java.lang.RuntimeException
	at org.eclipse.ui.tests.api.IWorkingSetManagerTest.lambda$0(IWorkingSetManagerTest.java:422)
	at org.eclipse.ui.internal.AbstractWorkingSetManager$5.run(AbstractWorkingSetManager.java:378)
	at org.eclipse.core.runtime.SafeRunner.run(SafeRunner.java:47)
	at org.eclipse.ui.internal.AbstractWorkingSetManager.lambda$1(AbstractWorkingSetManager.java:386)
	at org.eclipse.ui.internal.AbstractWorkingSetManager.firePropertyChange(AbstractWorkingSetManager.java:391)
	at org.eclipse.ui.internal.AbstractWorkingSetManager.internalAddWorkingSet(AbstractWorkingSetManager.java:235)
	at org.eclipse.ui.internal.AbstractWorkingSetManager.addWorkingSet(AbstractWorkingSetManager.java:227)
	at org.eclipse.ui.internal.WorkingSetManager.addWorkingSet(WorkingSetManager.java:66)
	at org.eclipse.ui.tests.api.IWorkingSetManagerTest.testListenerSafety(IWorkingSetManagerTest.java:431)

Nightly org.eclipse.ui.tests.UiTestSuite.txt contains intentional
RuntimeException that are hard to distinguish from unintentional
exceptions.
@jukzi jukzi force-pushed the ui.TestException branch from 2fae71a to 6893a47 Compare October 9, 2024 15:27
@github-actions
Copy link
Contributor

github-actions bot commented Oct 9, 2024

Test Results

 1 815 files  ±0   1 815 suites  ±0   1h 33m 21s ⏱️ - 16m 3s
 7 704 tests ±0   7 475 ✅  - 1  228 💤 ±0  1 ❌ +1 
24 273 runs  ±0  23 525 ✅  - 1  747 💤 ±0  1 ❌ +1 

For more details on these failures, see this check.

Results for commit 6893a47. ± Comparison against base commit b945db1.

@jukzi
Copy link
Contributor Author

jukzi commented Oct 9, 2024

ignore random fail #294 (comment)

@jukzi jukzi merged commit d5815f3 into eclipse-platform:master Oct 9, 2024
13 of 15 checks passed
@jukzi jukzi deleted the ui.TestException branch October 9, 2024 16:43
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