Skip to content

Refactor JUnit 5 for BaseTestAESCCMInteropBC and BaseTestAESGCMUpdateInteropBC tests - #1750

Merged
jasonkatonica merged 1 commit into
IBM:java25from
Mohit-Rajbhar100698:backport/java25refactor/aes-interop-junit5
Sep 10, 2026
Merged

jasonkatonica merged 1 commit into
IBM:java25from
Mohit-Rajbhar100698:backport/java25refactor/aes-interop-junit5

Conversation

@Mohit-Rajbhar100698

Copy link
Copy Markdown
Collaborator

This change converts these tests to use JUnit 5 parameterized tests.

  • Convert BaseTestAESCCMInteropBC to a JUnit 5 parameterized test
  • Convert BaseTestAESGCMUpdateInteropBC to a JUnit 5 parameterized test
  • Remove the provider-specific wrapper test classes
  • Update TestAll.java to remove references to the deleted test classes
  • Add a new argument source method in TestArguments.java for the BC provider interoperability tests.

Back-ported from: #1659
Signed-off-by: Mohit Rajbhar mohit.rajbhar@ibm.com

…InteropBC tests

This change converts these tests to use JUnit 5 parameterized tests.
- Convert BaseTestAESCCMInteropBC to a JUnit 5 parameterized test
- Convert BaseTestAESGCMUpdateInteropBC to a JUnit 5 parameterized test
- Remove the provider-specific wrapper test classes
- Update TestAll.java to remove references to the deleted test classes
- Add a new argument source method in TestArguments.java for the BC provider interoperability tests.

Signed-off-by: Mohit Rajbhar <mohit.rajbhar@ibm.com>

@jasonkatonica jasonkatonica left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@KostasTsiounis KostasTsiounis left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@jasonkatonica
jasonkatonica merged commit ce683e8 into IBM:java25 Sep 10, 2026
3 checks passed
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.

4 participants