Skip to content

Cover service HTTP status handling - #3227

Merged
adamtheturtle merged 1 commit into
mainfrom
cover-service-http-statuses
Sep 11, 2026
Merged

Cover service HTTP status handling#3227
adamtheturtle merged 1 commit into
mainfrom
cover-service-http-statuses

Conversation

@adamtheturtle

Copy link
Copy Markdown
Member

Summary

  • exercise bodyless rate-limit responses through the public VWS clients
  • exercise VuMark server errors through the public sync and async services
  • remove six coverage exclusions from the now-covered status branches

These responses are created at the HTTP transport boundary, so the tests cover the real public error behavior without reaching into private helpers.

Validation

  • all configured pre-commit hooks
  • all configured pre-push hooks, including Mypy, Pyright, Ty, and Pyrefly
  • full test suite with 100% branch coverage

@adamtheturtle
adamtheturtle force-pushed the cover-service-http-statuses branch from 335bdf7 to bc299e6 Compare September 10, 2026 23:14
@adamtheturtle
adamtheturtle merged commit b68340f into main Sep 11, 2026
13 checks passed
@adamtheturtle
adamtheturtle deleted the cover-service-http-statuses branch September 11, 2026 15:40
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