Patch to maybe show testing outputs if not in reference based on verbosity.#227
Patch to maybe show testing outputs if not in reference based on verbosity.#227emin63 wants to merge 1 commit intocomputationalmodelling:masterfrom
Conversation
|
I think something like this makes sense, though I wonder if we want to show it at a lower verbosity level than |
I don't have a view either way on those. Did you want some other changes to this PR or is there something preventing you from merging this and then making a decision on those issues later? I keep ending up copying this patch to multiple systems I am using. So getting this merged sooner would be my preference if possible. Thanks. |
|
Any feedback or issues with the current patch as it is? I run into this issue a lot and it would be great to have this as part of the main distribution instead of having to use my own local version with this patch. Thanks. |
This is a simple patch to make it so that if you run pytest with
-vvvand you have unexpected output instderrwhen testing the notebook, you will see the contents of stderr.Without this patch, you just get a test failure telling you:
but without showing you what is in stderr. This can be annoying if you are running an a CI environment like GitHub which has a different environment than your local system and is harder to debug minor warnings or other errors.