Skip to content

test: add negative-path crypto utility tests#330

Open
vaibhavchopra-wq wants to merge 2 commits into
masterfrom
test/negative-path-crypto-tests
Open

test: add negative-path crypto utility tests#330
vaibhavchopra-wq wants to merge 2 commits into
masterfrom
test/negative-path-crypto-tests

Conversation

@vaibhavchopra-wq
Copy link
Copy Markdown

Summary

  • Add comprehensive negative-path tests for signature verification utilities
  • Tests cover:
    • Empty signature rejection
    • Wrong length signature rejection
    • Non-hex signature rejection
    • Tampered valid-hex signature rejection
    • Valid dynamic signature acceptance
    • Special characters in payload
    • Unicode characters in payload (multi-byte UTF-8)

Test plan

  • All new tests pass
  • CI passes
  • Coverage maintained or improved

🤖 Generated with Claude Code

vaibhavchopra-wq and others added 2 commits May 10, 2026 20:34
Add comprehensive tests for signature verification edge cases:
- Empty signature rejection
- Wrong length signature rejection
- Non-hex signature rejection
- Tampered valid-length hex signature rejection
- Valid dynamic signature acceptance
- Special characters in payload
- Unicode in payload

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
setuptools is not pre-installed in Python 3.12+ GitHub runners.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@vaibhavchopra-wq vaibhavchopra-wq added the TestingNotRequired TestingNotRequired label for BVT label May 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

TestingNotRequired TestingNotRequired label for BVT

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant