Depend on JUnit 6 instead of JUnit 5#3560
Depend on JUnit 6 instead of JUnit 5#3560trancexpress wants to merge 7 commits intoeclipse-xtext:mainfrom
Conversation
|
There is a surprising number of JUnit 5 mentions, including even a quick fix action for adding JUnit 5 to the classpath... Probably for This PR so far doesn't touch any of those. |
Test Results 8 076 files + 4 8 076 suites +4 3h 53m 12s ⏱️ - 15m 40s Results for commit a41f0b0. ± Comparison against base commit 088392b. This pull request removes 17 and adds 17 tests. Note that renamed tests count towards both.♻️ This comment has been updated with latest results. |
2c93f7a to
ad11eb8
Compare
|
Running Fully summary: At least in JDT/PDE, this was usually an indicator of mixing JUnit 5 and 6 at runtime. It might be something else here. Such as JUnit 4 tests... Trying this: Results in: |
|
maybe this is a way too old tycho? |
Should I try updating to |
|
see #3509 we had a brief discussion to look into that someone after the current release (building milestones and releases is actually already beyond my cappa) |
|
the problem with tycho 5 is
what happens if use selects java 17 in wizard? should we have a fallback to old version. this also affecs junit 5/6 what should we offer the user. |
|
can you also check Jenkinsfile |
|
With |
We can always delay the JUnit 6 update for when Advantest is not in a rush, since the workaround is trivial. |
|
Current errors: |
|
@cdietrich do we want also renames for bundles? If they have |
|
yes you can also rename them |
…esting.junit6.tests
|
can you check JUnitVersion enum and references @szarnekow @LorenzoBettini wdyt? |
org.eclipse.xtext.xtext.wizard/src/org/eclipse/xtext/xtext/wizard/TestProjectDescriptor.xtend
Outdated
Show resolved
Hide resolved
org.eclipse.xtext.xtext.wizard/src/org/eclipse/xtext/xtext/wizard/TargetPlatformProject.xtend
Outdated
Show resolved
Hide resolved
See my comments above. I seem to understand that the enum hasn't been changed. |
|
i assume no user uses it. https://github.com/search?q=%22import+org.eclipse.xtext.util.JUnitVersion%22&type=code maybe we should mark it somehow |
|
Let me know what you decide for the JUnit 5 constant that is currently removed. Also if I'm missing any changes, I'm not sure what else to grep for. |
|
then let remove the old constant, use the new one in the tests, wizard fragments etc |
See: #3552