Skip to content

Fix iteration counter reset for AESCCM parameterized tests - #1712

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

Fix iteration counter reset for AESCCM parameterized tests#1712
jasonkatonica merged 1 commit into
IBM:java21from
Mohit-Rajbhar100698:backport/java21/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

@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.

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>
@Mohit-Rajbhar100698
Mohit-Rajbhar100698 force-pushed the backport/java21/fix/reset-aes-iteration-counter branch from 3260901 to c7e035b Compare August 20, 2026 08:49
@jasonkatonica
jasonkatonica merged commit c50f03b into IBM:java21 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