X Tutup
Skip to content

Testing Java LTS versions#536

Merged
poovamraj merged 5 commits intov4-devfrom
testing-lts-versions
Mar 7, 2022
Merged

Testing Java LTS versions#536
poovamraj merged 5 commits intov4-devfrom
testing-lts-versions

Conversation

@poovamraj
Copy link
Copy Markdown
Contributor

Changes

Please describe both what is changing and why this is important. Include:

  • We have added 2 new tasks to run the Unit tests in different JVM LTS versions.
  • Test class compilation will target Java 8 since we run unit tests in Java 8 (Previously this was compiled for Java 11)
  • Jacoco toolVersion is explicitly mentioned since Java 17 is provided only above the specified version
  • Had to ignore Tests for secp256k1 curve since it is disabled for Java 15+. We have to decide on whether we are going to support this and then decide on running these tests

References

Jacoco Release note mentioning Java 17 support - https://www.jacoco.org/jacoco/trunk/doc/changes.html
Java dropping support for secp256k1 curve - https://www.oracle.com/java/technologies/javase/15-relnote-issues.html#JDK-8237219

Testing

  • Ensured the mentioned tasks are running in respective Java LTS versions
  • Ensured the added tasks are run as part of the check task

@poovamraj poovamraj requested a review from a team as a code owner March 4, 2022 11:02
@poovamraj
Copy link
Copy Markdown
Contributor Author

CI Failure for Java 17 Unit Test task when testing unsupported curve for ECDSA

Screenshot 2022-03-04 at 21 53 45

Copy link
Copy Markdown
Contributor

@jimmyjames jimmyjames left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just requested that you remove the unused Ignore import since we're excluding for java 17 only now.

@poovamraj poovamraj merged commit ed91dc6 into v4-dev Mar 7, 2022
@poovamraj poovamraj added this to the v4-Beta milestone Mar 8, 2022
@poovamraj poovamraj modified the milestones: v4-Beta, 4.0.0-beta.0 May 6, 2022
@evansims evansims deleted the testing-lts-versions branch July 5, 2022 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

X Tutup