Skip to content

Verify HTTP metric counts and simplify probe coverage - #446

Merged
lesnik512 merged 1 commit into
mainfrom
test/metrics-and-health-contracts
Nov 6, 2026
Merged

lesnik512 merged 1 commit into
mainfrom
test/metrics-and-health-contracts

Conversation

@lesnik512

Copy link
Copy Markdown
Member

Changes

  • Share HTTP metrics cases across FastAPI, Litestar, and FastMCP and assert exact request counts for OpenTelemetry and Prometheus.
  • Test probe failures and warnings on the health-check instrument while keeping HTTP 200 and 503 coverage for each framework.
  • Synchronize the concurrency test with events so it does not depend on elapsed time.

Verification

  • just test-ci passed with 658 tests and 100% line coverage. Ruff and ty check passed.
  • The HTTP metrics matrix passed with both legacy and stable OpenTelemetry metric names active.
  • A temporary second request made the exact-count assertions fail for both backends.

@lesnik512
lesnik512 merged commit 822d8c5 into main Nov 6, 2026
35 checks passed
@lesnik512
lesnik512 deleted the test/metrics-and-health-contracts branch November 6, 2026 09:16
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