Make PipTestEnvironment stderr checks more assertive
See original GitHub issueWe can tighten the tests in the following way. Rename the PipTestEnvironment.run()
arguments of allow_stderr_warning
and allow_stderr_error
arguments to expect_stderr_warning
and expect_stderr_error
:
and make the test fail if the argument is passed and a warning or error isn’t present, respectively. The idea is that the argument should only be passed if it’s actually needed, unlike the current case where the test is passing if the argument is provided unnecessarily.
Also, to do this it might be necessary to do a check of some kind to know if the argument should be passed. For example, whether a deprecation warning is emitted can depend on what version is running.
Posted in its original form by @cjerdonek in #161 (comment)
Issue Analytics
- State:
- Created 4 years ago
- Reactions:1
- Comments:9 (9 by maintainers)
Top Results From Across the Web
How can I pipe stderr and still test if the command failed?
When I run the script without piping to logger, everything is fine. When I pipe everything to logger for review at a later...
Read more >turnt: Docs, Tutorials, Reviews | Openbase
Tiny Unified Runner N' Tester (Turnt). Turnt is a simple snapshot testing tool inspired by Cram and LLVM's lit. It's good for testing...
Read more >avocado Documentation - Read the Docs
Avocado is a set of tools and libraries to help with automated testing. One can call it a test framework with benefits. Native...
Read more >Galaxy Tool XML File
* aggressive : adds checks for non zero exit codes, and checks for Exception: , Error: in the standard error. Additionally checks for...
Read more >My Document - Adaptive Computing
and instructions for how to set up and use compute node checks. ... make and make install that works across all Linux distributions...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
Hey @ssurbhi560. Thanks for pinging me! Can’t really find a time to get this done. Would you like to pick it up? Please, feel free to 🙏
Hey @gutsytechster! I won’t be able to work on this right now, feel free to work on it. 😃