-
Notifications
You must be signed in to change notification settings - Fork 51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes the issue that nativeTest cannot be executed using Junit 5.11.0-M2 #603
base: master
Are you sure you want to change the base?
Conversation
linghengqian
commented
Jun 13, 2024
•
edited
Loading
edited
- Fixes Unable to execute nativeTest under Junit 5.11.0-M2 #602 .
- Fixes the issue that nativeTest cannot be executed using Junit 5.11.0-M2.
- The project's gradle toolchain actively looks for a random jdk17 release, such as Microsoft OpenJDK 17. Setting up a GraalVM version of JDK11 can be very confusing.
43a5d98
to
581246b
Compare
219cf50
to
963e4c5
Compare
611dc68
to
fe745c6
Compare
419e583
to
2dcfe2f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- @dnestoro @fniephaus Is it possible to temporarily disable the execution of
./gradlew :native-gradle-plugin:functionalTest
on GraalVM CE Dev For JDK24 / https://github.com/graalvm/graalvm-ce-dev-builds/releases/tag/24.1.0-dev-20240612_2110 ? I cannot reproduce the CI results of Github Actions locally, refer to https://github.com/graalvm/native-build-tools/actions/runs/9517556127/job/26236415815?pr=603 .
sdk install java 17.0.11-graal
sdk use java 17.0.11-graal
git clone [email protected]:linghengqian/native-build-tools.git -b fix-junit
cd ./native-build-tools/
./gradlew :native-gradle-plugin:functionalTest
2ee0a26
to
10fb62e
Compare
.sdkmanrc
Outdated
@@ -1,3 +1,3 @@ | |||
# Enable auto-env through the sdkman_auto_env config | |||
# Add key=value pairs of SDKs to use below | |||
java=22.2.r11-grl | |||
java=17.0.11-graal |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did we need this change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- I initialized a new Ubuntu WSL distribution instance to test the current project, and it turns out that the project requires both GraalVM CE For JDK11 and GraalVM CE For JDK22 to be installed via SDKMAN! to run unit tests. I will roll back this change.
@@ -89,7 +89,7 @@ jobs: | |||
- name: "🔧 Install GraalVM (dev)" | |||
uses: graalvm/setup-graalvm@main | |||
with: | |||
java-version: 'dev' | |||
java-version: '22.0.1' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, why this change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- This is related to the issue I pointed out at Fixes the issue that nativeTest cannot be executed using Junit 5.11.0-M2 #603 (review) . Under GraalVM CE For JDK24 Dev, this CI fails as https://github.com/graalvm/native-build-tools/actions/runs/9517556127/job/26236415815?pr=603 . This CI works fine under GraalVM CE For JDK22, as shown in https://github.com/graalvm/native-build-tools/actions/runs/9520173469/job/26244879976?pr=603 . I don't know anything about the changes after JDK23.
- I have now rolled back the changes to show the CI.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
@linghengqian let me try to fix failing gate |