Skip to content

Fix iteration counter reset for AESCCM parameterized tests - #1715

Merged
jasonkatonica merged 1 commit into
IBM:java27from
Mohit-Rajbhar100698:backport/java27/fix/reset-aes-iteration-counter
Aug 20, 2026
Merged

Fix iteration counter reset for AESCCM parameterized tests#1715
jasonkatonica merged 1 commit into
IBM:java27from
Mohit-Rajbhar100698:backport/java27/fix/reset-aes-iteration-counter

Conversation

@Mohit-Rajbhar100698

Copy link
Copy Markdown
Collaborator

The iteration counter was shared across parameterized test runs, causing subsequent provider executions to skip the test loop. Reset the counter before each parameterized execution and update debug logs.

Changes:

  • Reset the iteration counter in TestAESCCM.java and TestAESCCM2.java during test setup for each parameterized execution.
  • Update debug logs.

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

The iteration counter was shared across parameterized test runs,
causing subsequent provider executions to skip the test loop.
Reset the counter before each parameterized execution and update
debug logs.

Changes:
- Reset the iteration counter in TestAESCCM.java and
  TestAESCCM2.java during test setup for each parameterized
  execution.
- Update debug logs.

Signed-off-by: Mohit Rajbhar <mohit.rajbhar@ibm.com>
@johnpeck-us-ibm
johnpeck-us-ibm self-requested a review August 19, 2026 14:33
Comment thread src/test/java/ibm/jceplus/junit/tests/TestAESCCM.java
Comment thread src/test/java/ibm/jceplus/junit/tests/TestAESCCM.java
Comment thread src/test/java/ibm/jceplus/junit/tests/TestAESCCM.java

@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. Be sure to open an issue for the use of statics in this repostiroy and link the issue to this PR.

@jasonkatonica
jasonkatonica merged commit dbf937f into IBM:java27 Aug 20, 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.

3 participants