Skip to content

Commit

Permalink
test: remove unnecessary returns after pytest.skip
Browse files Browse the repository at this point in the history
pytest.skip raises.
  • Loading branch information
scop committed Jun 24, 2019
1 parent 07eada3 commit d5fa7e3
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions test/t/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,6 @@ def bash(request) -> pexpect.spawn:
else:
bash.close()
pytest.skip(skipif)
return
xfail = marker.kwargs.get("xfail")
if xfail:
try:
Expand Down Expand Up @@ -190,7 +189,6 @@ def is_testable(bash: pexpect.spawn, cmd: str) -> bool:
return False
if not load_completion_for(bash, cmd):
pytest.skip("No completion for command %s" % cmd)
return False
return True


Expand Down Expand Up @@ -377,7 +375,6 @@ def assert_complete(
pass
else:
pytest.skip(skipif)
return CompletionResult("", [])
xfail = kwargs.get("xfail")
if xfail:
try:
Expand Down

0 comments on commit d5fa7e3

Please sign in to comment.