Skip to content

test: add negative-path crypto utility tests#267

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

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

Conversation

@vaibhavchopra-wq
Copy link
Copy Markdown

@vaibhavchopra-wq vaibhavchopra-wq commented May 11, 2026

Summary

Add comprehensive negative-path tests for signature verification utilities:

  • Empty signature rejection
  • Wrong length signature rejection
  • Wrong value signature rejection (not hex validation)
  • Tampered valid-hex signature rejection
  • Valid dynamic signature acceptance
  • Special characters in payload
  • Unicode characters in payload (multi-byte UTF-8)

Also includes CI and gemspec updates for independent mergeability:

  • Update GitHub Actions to v4
  • Update test matrix: 3.2, 3.3, 3.4, 4.0
  • Add ostruct gem for Ruby 4.0 compatibility
  • Update dev dependencies

Test plan

  • All new tests pass
  • CI passes on Ruby 3.2, 3.3, 3.4, 4.0
  • Coverage maintained or improved

🤖 Generated with Claude Code

@vaibhavchopra-wq vaibhavchopra-wq added the TestingNotRequired TestingNotRequired label for BVT label May 13, 2026
Add comprehensive tests for signature verification edge cases:
- Empty signature rejection
- Wrong length signature rejection
- Wrong value signature rejection (not hex validation)
- Tampered valid-hex signature rejection
- Valid dynamic signature acceptance
- Special characters in payload
- Unicode characters in payload

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@vaibhavchopra-wq vaibhavchopra-wq force-pushed the test/negative-path-crypto-tests branch from 0ed87e4 to 04317cd Compare May 13, 2026 06:44
- Update GitHub Actions to v4
- Update test matrix: 3.2, 3.3, 3.4, 4.0
- Add ostruct gem (removed from stdlib in Ruby 4.0)
- Update dev dependencies to latest versions

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
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