Skip to content

Add test for RequestQuotaReached exception#3094

Open
adamtheturtle wants to merge 2 commits into
mainfrom
adamtheturtle/issue-822-request-quota-test
Open

Add test for RequestQuotaReached exception#3094
adamtheturtle wants to merge 2 commits into
mainfrom
adamtheturtle/issue-822-request-quota-test

Conversation

@adamtheturtle

Copy link
Copy Markdown
Member

Summary

  • Implements the stub test_request_quota_reached so RequestQuotaReachedError is exercised end-to-end through VWS.list_targets.
  • Uses a custom transport because vws-python-mock still does not simulate this result code (mock#53).
  • Removes the pragma: no cover on RequestQuotaReachedError.

Closes #822.

Test plan

  • pytest tests/test_vws_exceptions.py::test_request_quota_reached
  • CI green on this PR

Made with Cursor

adamtheturtle and others added 2 commits July 17, 2026 07:51
The mock does not yet simulate this result code, so the test uses a
custom transport. Closes #822.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
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.

Add test which hits RequestQuotaReached

1 participant