Bugfix/warningschecker twice#4620
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4620 +/- ##
==========================================
- Coverage 95.75% 95.75% -0.01%
==========================================
Files 111 111
Lines 24678 24683 +5
Branches 2446 2446
==========================================
+ Hits 23630 23634 +4
+ Misses 740 739 -1
- Partials 308 310 +2
Continue to review full report at Codecov.
|
RonnyPfannschmidt
left a comment
There was a problem hiding this comment.
approving based on the discussion we had in the issue
|
Thank you! |
| result.stdout.fnmatch_lines(["*1 failed in*"]) | ||
|
|
||
|
|
||
| def test_warningschecker_twice(testdir): |
There was a problem hiding this comment.
I would go for a simple unittest here:
def test_warningschecker_twice():
expectation = pytest.warns(UserWarning)
with expectation:
warnings.warn("Message A", UserWarning)
with expectation:
warnings.warn("Message B", UserWarning)There was a problem hiding this comment.
@nicoddemus
Makes sense, sorry for merging it too early.
Will you create a followup?
There was a problem hiding this comment.
Yep, way simpler! But now it should be another PR?
There was a problem hiding this comment.
Yep, we can't change this one anymore. Can you do it @Sup3rGeo? If you don't have the time, I will get to it later. 👍
Fixes #4617
changelogfolder, with a name like<ISSUE NUMBER>.<TYPE>.rst. See changelog/README.rst for details.masterbranch for bug fixes, documentation updates and trivial changes.