Skip to content

feat(wabe): add rate limit tests#293

Merged
coratgerl merged 2 commits intomainfrom
rate-limit-tests
Feb 18, 2026
Merged

feat(wabe): add rate limit tests#293
coratgerl merged 2 commits intomainfrom
rate-limit-tests

Conversation

@coratgerl
Copy link
Collaborator

@coratgerl coratgerl commented Feb 18, 2026

Summary by CodeRabbit

  • Documentation

    • Expanded rate limiting documentation with comprehensive configuration details, behavior explanations (per-client enforcement, HTTP 429 responses), practical examples, and best practices for production and public APIs.
  • Tests

    • Added test coverage verifying GraphQL rate limiting behavior with and without rate limit configuration.

@coratgerl coratgerl self-assigned this Feb 18, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 18, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

This pull request expands rate limiting documentation with detailed guidance on configuration, behavior, and best practices, while adding corresponding test coverage for GraphQL endpoints and extending test infrastructure to support rate limiting configuration.

Changes

Cohort / File(s) Summary
Documentation
packages/wabe-documentation/docs/documentation/security/index.md
Expanded Rate limiting section with new subsections: Configuration, Options, Behavior, Example, and Best practices. Updated descriptions of interval and numberOfRequests properties; added per-client enforcement and HTTP 429 details.
Tests
packages/wabe/src/security.test.ts, packages/wabe/src/utils/testHelper.ts
Added two GraphQL rate-limiting tests (with/without configuration) and extended setupTests to accept optional rateLimit parameter for test configuration.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 hops with glee
Rate limits documented, from start to the end,
Tests now validate what the docs comprehend!
Config and behavior, all crystal and clear,
GraphQL flows safely—no abuse to fear! 🚀

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main change: adding rate limit tests to the wabe package, which is the primary focus of the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch rate-limit-tests

Comment @coderabbitai help to get the list of available commands and usage tips.

@coratgerl coratgerl merged commit 6196b7c into main Feb 18, 2026
4 checks passed
@coratgerl coratgerl deleted the rate-limit-tests branch February 18, 2026 19:03
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.

1 participant