Skip to content

test(mcp): Remove vacuous test_fastmcp_prompt_async() - #7420

Open
alexander-alderman-webb wants to merge 1 commit into
masterfrom
webb/mcp/remove_test_fastmcp_prompt_async
Open

test(mcp): Remove vacuous test_fastmcp_prompt_async()#7420
alexander-alderman-webb wants to merge 1 commit into
masterfrom
webb/mcp/remove_test_fastmcp_prompt_async

Conversation

@alexander-alderman-webb

Copy link
Copy Markdown
Contributor

Description

The only assertion is trivially true because an HTTP server transaction is created.

Issues

Reminders

@alexander-alderman-webb
alexander-alderman-webb requested a review from a team as a code owner September 7, 2026 14:29
@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

130162 passed | ⏭️ 7118 skipped | Total: 137280 | Pass Rate: 94.81% | Execution Time: 481m 15s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -426
Passed Tests 📉 -422
Failed Tests
Skipped Tests 📉 -4

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2504 uncovered lines.
❌ Project coverage is 90.27%. Comparing base (36ff01e) to head (7692241).

Coverage diff
@@            Coverage Diff             @@
##        master       #PR       +/-##
==========================================
- Coverage    90.28%    90.27%    -0.01%
==========================================
  Files          193       193         —
  Lines        25729     25729         —
  Branches      9496      9496         —
==========================================
+ Hits         23228     23225        -3
- Misses        2501      2504        +3
- Partials      1432      1432         —

Generated by Codecov Action

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