Skip to content

[test] improve HMAC test coverage#342

Open
meilu-git wants to merge 2 commits intomainfrom
meilu/hmac_tests
Open

[test] improve HMAC test coverage#342
meilu-git wants to merge 2 commits intomainfrom
meilu/hmac_tests

Conversation

@meilu-git
Copy link
Copy Markdown
Collaborator

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds additional integration tests in api/tests to exercise more HMAC edge-cases and cross-check single-shot vs streaming behavior, improving coverage of the HMAC API surface.

Changes:

  • Add tests for empty-message HMAC sign/verify (single-shot and streaming) and determinism.
  • Add tests around size limits (exact 1024 bytes, and rejection of >1024 bytes).
  • Add tests for various negative/mismatch scenarios (different keys/curves, tag length, chunking patterns, update behavior).

Comment thread api/tests/src/algo/hmac/hmac_tests.rs Outdated
Comment thread api/tests/src/algo/hmac/hmac_tests.rs Outdated
Comment thread api/tests/src/algo/hmac/hmac_tests.rs Outdated
Comment thread api/tests/src/algo/hmac/hmac_tests.rs Outdated
Comment thread api/tests/src/algo/hmac/hmac_tests.rs Outdated
Comment thread api/tests/src/algo/hmac/hmac_tests.rs Outdated
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.

2 participants