Skip to content

test(stdlib): Stop mocking HTTPSConnection.send in trace header tests#6156

Merged
alexander-alderman-webb merged 2 commits intomasterfrom
webb/stdlib/tests
Apr 29, 2026
Merged

test(stdlib): Stop mocking HTTPSConnection.send in trace header tests#6156
alexander-alderman-webb merged 2 commits intomasterfrom
webb/stdlib/tests

Conversation

@alexander-alderman-webb
Copy link
Copy Markdown
Contributor

@alexander-alderman-webb alexander-alderman-webb commented Apr 28, 2026

Description

Emulate basic HTTP 1.1 response in HTTPSConnection subclass instead of inspecting mock.Mock arguments.

Issues

Reminders

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 28, 2026

Codecov Results 📊

13 passed | Total: 13 | Pass Rate: 100% | Execution Time: 7.74s

📊 Comparison with Base Branch

Metric Change
Total Tests
Passed Tests
Failed Tests
Skipped Tests

✨ No test changes detected

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 14791 uncovered lines.
✅ Project coverage is 31.38%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    27.38%    31.38%       +4%
==========================================
  Files          190       190         —
  Lines        21543     21554       +11
  Branches      7182      7182         —
==========================================
+ Hits          5898      6763      +865
- Misses       15645     14791      -854
- Partials       477       571       +94

Generated by Codecov Action

@alexander-alderman-webb alexander-alderman-webb marked this pull request as ready for review April 29, 2026 05:59
@alexander-alderman-webb alexander-alderman-webb requested a review from a team as a code owner April 29, 2026 05:59
@alexander-alderman-webb alexander-alderman-webb changed the title test(stdlib): Remove mocks in outgoing trace header tests test(stdlib): Stop mocking HTTPSConnection.send in trace header tests Apr 29, 2026
@alexander-alderman-webb alexander-alderman-webb merged commit a4c16d7 into master Apr 29, 2026
156 checks passed
@alexander-alderman-webb alexander-alderman-webb deleted the webb/stdlib/tests branch April 29, 2026 10:12
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.

2 participants