Skip to content
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

Build and test with JDK 17 & 21 #607

Merged
merged 8 commits into from
Sep 11, 2023
Merged

Build and test with JDK 17 & 21 #607

merged 8 commits into from
Sep 11, 2023

Conversation

martin-g
Copy link
Member

Update Maven in CI to 3.9.4 (latest stable)

Update Maven in CI to 3.9.4 (latest stable)

Signed-off-by: Martin Tzvetanov Grigorov <[email protected]>
Signed-off-by: Martin Tzvetanov Grigorov <[email protected]>
It fails on JdK 21 with:
```
[ERROR]   FilesTest.dontWaitTooMuchIfCantDelete:96 Mockito
Mockito cannot mock this class: class java.io.File.

If you're not sure why you're getting this error, please open an issue on GitHub.

Java               : 21
JVM vendor name    : Oracle Corporation
JVM vendor version : 21+35-2513
JVM name           : OpenJDK 64-Bit Server VM
JVM version        : 21+35-2513
JVM info           : mixed mode, sharing
OS name            : Linux
OS version         : 6.2.0-10018-tuxedo

You are seeing this disclaimer because Mockito is configured to create inlined mocks.
You can learn about inline mocks and their limitations under item #39 of the Mockito class javadoc.

Underlying exception : org.mockito.exceptions.base.MockitoException: Could not modify all classes [interface java.io.Serializable, class java.lang.Object, class java.io.File, interface java.lang.Comparable]
```

Signed-off-by: Martin Tzvetanov Grigorov <[email protected]>
The problem is explained at https://bugs.openjdk.org/browse/JDK-8304925

Update Byte-Buddy to fix failing tests due to Mockito not able to create
mocks

Signed-off-by: Martin Tzvetanov Grigorov <[email protected]>
Signed-off-by: Martin Tzvetanov Grigorov <[email protected]>
Test against latest stable - JDK 20

Signed-off-by: Martin Tzvetanov Grigorov <[email protected]>
Fix the version of EA
Update maven-javadoc-plugin

Signed-off-by: Martin Tzvetanov Grigorov <[email protected]>
@martin-g martin-g merged commit 5d6c882 into master Sep 11, 2023
3 checks passed
@martin-g martin-g deleted the test-with-java-21 branch September 11, 2023 08:27
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