eclipse-modisco / org.eclipse.modisco

Eclipse Public License 2.0
0 stars 0 forks source link

[tests] Remove redundant tests #1073

Open eclipse-modisco-bot opened 3 days ago

eclipse-modisco-bot commented 3 days ago

| --- | --- | | Bugzilla Link | 582735 | | Status | NEW | | Importance | P3 normal | | Reported | Dec 13, 2023 13:08 EDT | | Modified | Dec 13, 2023 13:10 EDT | | See also | 582717 | | Reporter | Ed Willink |

Description

The test pom.xml have two executions:

\ default-test ... <...\ **/AllTestsNotInUIThread.class \ AllUITests ...\ **/AllTestsInUIThread.class But no test plugin has both, so one is alwatys redundant. Indeed only org.eclipse.modisco.infra.browser.tests has the AllTestsNotInUIThread and I doubt that is what it says. Rationalize (and support maven-sure-fire for genuinely non-UI thread).