Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update httpx and fix caplog assertions in tests #317

Merged
merged 1 commit into from
Dec 23, 2024

Conversation

douglatornell
Copy link
Member

Upgraded httpx, and pytest-httpx in requirements.txt to newer versions. Adjusted caplog index assertions in test cases to align with the updated logging behavior of httpx. This ensures compatibility and correct verification of log records across all affected tests.

This issue arose due to a change in the log output of httpx=0.28.0. It was found by the GitHub Actions pytest-with-coverage workflows when the tests were run with the updated version of httpx in response to an unrelated dependabot PR.

Upgraded `httpx`, and `pytest-httpx` in `requirements.txt` to newer versions.
Adjusted caplog index assertions in test cases to align with the updated
logging behavior of `httpx`. This ensures compatibility and correct
verification of log records across all affected tests.

This issue arose due to a change in the log output of `httpx=0.28.0`. It was
found by the GitHub Actions pytest-with-coverage workflows when the tests were
run with the updated version of `httpx` in response to an unrelated dependabot
PR.
@douglatornell douglatornell added the bug Something isn't working label Dec 23, 2024
@douglatornell douglatornell added this to the v24.2 milestone Dec 23, 2024
Copy link

codecov bot commented Dec 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.72%. Comparing base (b9e2423) to head (970645d).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #317   +/-   ##
=======================================
  Coverage   77.72%   77.72%           
=======================================
  Files         133      133           
  Lines       18778    18778           
  Branches      762      762           
=======================================
  Hits        14595    14595           
  Misses       4117     4117           
  Partials       66       66           
Flag Coverage Δ
unittests 77.72% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@douglatornell douglatornell deleted the update-httpx-tests branch December 23, 2024 17:51
@douglatornell douglatornell restored the update-httpx-tests branch December 23, 2024 17:51
@douglatornell douglatornell reopened this Dec 23, 2024
@douglatornell douglatornell merged commit 3bf0696 into main Dec 23, 2024
15 checks passed
@douglatornell douglatornell deleted the update-httpx-tests branch December 23, 2024 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant