diff options
| -rw-r--r-- | setup.py | 1 | ||||
| -rw-r--r-- | testing/test_parseopt.py | 11 |
2 files changed, 7 insertions, 5 deletions
@@ -33,6 +33,7 @@ def main(): # fmt: off extras_require={ "testing": [ + "argcomplete", "hypothesis>=3.56", "nose", "requests", diff --git a/testing/test_parseopt.py b/testing/test_parseopt.py index e25705d00..91f9a7d2b 100644 --- a/testing/test_parseopt.py +++ b/testing/test_parseopt.py @@ -299,15 +299,12 @@ def test_argcomplete(testdir, monkeypatch): if not distutils.spawn.find_executable("bash"): pytest.skip("bash not available") script = str(testdir.tmpdir.join("test_argcomplete")) - pytest_bin = sys.argv[0] - if "pytest" not in os.path.basename(pytest_bin): - pytest.skip("need to be run with pytest executable, not {}".format(pytest_bin)) with open(str(script), "w") as fp: # redirect output from argcomplete to stdin and stderr is not trivial # http://stackoverflow.com/q/12589419/1307905 # so we use bash - fp.write('COMP_WORDBREAKS="$COMP_WORDBREAKS" %s 8>&1 9>&2' % pytest_bin) + fp.write('COMP_WORDBREAKS="$COMP_WORDBREAKS" python -m pytest 8>&1 9>&2') # alternative would be exteneded Testdir.{run(),_run(),popen()} to be able # to handle a keyword argument env that replaces os.environ in popen or # extends the copy, advantage: could not forget to restore @@ -323,7 +320,11 @@ def test_argcomplete(testdir, monkeypatch): # argcomplete not found pytest.skip("argcomplete not available") elif not result.stdout.str(): - pytest.skip("bash provided no output, argcomplete not available?") + pytest.skip( + "bash provided no output on stdout, argcomplete not available? (stderr={!r})".format( + result.stderr.str() + ) + ) else: result.stdout.fnmatch_lines(["--funcargs", "--fulltrace"]) os.mkdir("test_argcomplete.d") |