Skip to content

Add tests for RateLimiter and BlockchainSecurity utilities - #952

Merged
nanaf6203-bit merged 1 commit into
MettaChain:mainfrom
YaronZaki:fix/issue-845-security-utilities-tests
Aug 26, 2026
Merged

nanaf6203-bit merged 1 commit into
MettaChain:mainfrom
YaronZaki:fix/issue-845-security-utilities-tests

Conversation

@YaronZaki

@YaronZaki YaronZaki commented Aug 26, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Added test coverage for security utilities that were only partially tested.

Changes

  • Tests for RateLimiter singleton, sliding window, cleanup, and different configs
  • Tests for BlockchainSecurity address risk checks, transaction validation, caching, and value normalization

Closes #845
Closes #846
Closes #847
Closes #848

@drips-wave

drips-wave Bot commented Aug 26, 2026

Copy link
Copy Markdown

@YaronZaki Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@nanaf6203-bit nanaf6203-bit left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice work, thanks for getting this over the line!

@nanaf6203-bit
nanaf6203-bit merged commit 32674bd into MettaChain:main Aug 26, 2026
2 of 5 checks passed
@github-actions

Copy link
Copy Markdown

🔒 Preview Environment Destroyed

The preview environment for this PR has been torn down.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants