eclipse-jdt / eclipse.jdt.core

Eclipse Public License 2.0
162 stars 131 forks source link

Lack of test isolation between CompletionTests and CompletionTests2 #1204

Closed stephan-herrmann closed 1 year ago

stephan-herrmann commented 1 year ago

The following tweaks in RunCompletionModelTests will cause failures:

Now change CompletionTests.suite() from using BYTECODE_DECLARATION_ORDER to ALPHABETICAL_SORT:

stephan-herrmann commented 1 year ago

Note, that above observations were on top of other pending fixes:

jarthana commented 1 year ago

I find most, (if not all, but need to be confirmed) failures are due to the tests setting the properties with JavaCore.setOptions but not applying them on the test project. Will confirm and produce a PR for M2.