Skip to content

test(pydantic-ai): Remove vacuous _set_agent_data() tests - #7450

Open
alexander-alderman-webb wants to merge 1 commit into
masterfrom
webb/pydantic-ai/remove-_set_agent_data-tests
Open

test(pydantic-ai): Remove vacuous _set_agent_data() tests#7450
alexander-alderman-webb wants to merge 1 commit into
masterfrom
webb/pydantic-ai/remove-_set_agent_data-tests

Conversation

@alexander-alderman-webb

Copy link
Copy Markdown
Contributor

Description

The assertions are trivial because a manual transaction is created, and we should not test private functions to begin with.

Issues

Reminders

@alexander-alderman-webb
alexander-alderman-webb requested a review from a team as a code owner September 9, 2026 17:21
@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

131875 passed | ⏭️ 7131 skipped | Total: 139006 | Pass Rate: 94.87% | Execution Time: 482m 27s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -30
Passed Tests 📉 -30
Failed Tests
Skipped Tests

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2527 uncovered lines.
✅ Project coverage is 90.2%. Comparing base (7e95b86) to head (00c3738).

Coverage diff
@@            Coverage Diff             @@
##        master       #PR       +/-##
==========================================
+ Coverage    90.20%    90.20%        —%
==========================================
  Files          193       193         —
  Lines        25788     25788         —
  Branches      9530      9530         —
==========================================
+ Hits         23261     23261         —
- Misses        2527      2527         —
- 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