New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Skip test_system_ssh_version if no ssh found + split parsing into separate test #7422
Conversation
…t exist We have it only documented in the docstring and also used in few of other tests but never explicitly tested for the runner. To rely on it, I decided to add a dedicated test right there for the runner
…ing into separate test
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## maint #7422 +/- ##
==========================================
+ Coverage 88.10% 90.65% +2.55%
==========================================
Files 327 327
Lines 44692 44707 +15
Branches 0 5947 +5947
==========================================
+ Hits 39374 40528 +1154
+ Misses 5318 4164 -1154
- Partials 0 15 +15
☔ View full report in Codecov by Sentry. |
try: | ||
WitlessRunner().run(['ssh', '-V'], protocol=StdOutErrCapture) | ||
except FileNotFoundError as exc: | ||
raise SkipTest(f"no ssh binary available: {exc}") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it would be preferrable to use pytest.skip()
instead.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, done in ce3f788
PR released in |
There is some auxiliary commits as well which would have more information.
Part to address 3rd failing test on conda #7421