Skip to content

fix test with warnings#493

Merged
markstory merged 1 commit intogetsentry:masterfrom
beliaev-maksim:mbeliaev/warnings
Feb 7, 2022
Merged

fix test with warnings#493
markstory merged 1 commit intogetsentry:masterfrom
beliaev-maksim:mbeliaev/warnings

Conversation

@beliaev-maksim
Copy link
Copy Markdown
Collaborator

pipe on master failed: https://github.com/getsentry/responses/runs/5068593004?check_suite_focus=true

new version of pytest was released, now we cannot check warns with None, there is a new method:
https://docs.pytest.org/en/7.0.x/how-to/capture-warnings.html#additional-use-cases-of-warnings-in-tests

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 7, 2022

Codecov Report

Merging #493 (e5708f2) into master (2a9924e) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #493   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            6         6           
  Lines         2160      2161    +1     
=========================================
+ Hits          2160      2161    +1     
Impacted Files Coverage Δ
responses/test_responses.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2a9924e...e5708f2. Read the comment docs.

@markstory markstory merged commit 930b784 into getsentry:master Feb 7, 2022
@beliaev-maksim beliaev-maksim deleted the mbeliaev/warnings branch February 7, 2022 15:07
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