sourmash
sourmash copied to clipboard
[MRG] use stderr for test output printing
Sets sys.stdout=sys.stderr
, per https://github.com/sourmash-bio/sourmash/pull/2138#issuecomment-1216779696. Suggested as a solution to pytest-xdist
not printing test output here: https://github.com/pytest-dev/pytest-xdist/issues/354#issuecomment-430502446
restores test output printing
@ctb is this a desired solution? Or did you find a better workaround?
Codecov Report
Merging #2217 (ce0f648) into latest (651ecea) will increase coverage by
7.37%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## latest #2217 +/- ##
==========================================
+ Coverage 84.66% 92.04% +7.37%
==========================================
Files 131 100 -31
Lines 15512 11236 -4276
Branches 2210 2210
==========================================
- Hits 13134 10342 -2792
+ Misses 2085 601 -1484
Partials 293 293
Flag | Coverage Δ | |
---|---|---|
python | 92.04% <ø> (ø) |
|
rust | ? |
Flags with carried forward coverage won't be shown. Click here to find out more.
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
I can't think of a specific situation where it will cause problems, and we don't do anything particularly tricky with stdout/stderr outside of the script runner (runtmp.sourmash(...)
) so ... let's give it a try and see!!