diff options
| author | Gregory P. Smith <greg@krypto.org> | 2015-01-21 01:19:47 (GMT) |
|---|---|---|
| committer | Gregory P. Smith <greg@krypto.org> | 2015-01-21 01:19:47 (GMT) |
| commit | b5684c48e1e37ff8fb1cf6cc42cae31bd2da37d8 (patch) | |
| tree | 037b8c1c2016ce27dacc02f6d7a67643f90d9fea /Lib/test/script_helper.py | |
| parent | b176d40398f4a6f15fc3f63ef55fb064eca13ee3 (diff) | |
| download | cpython-b5684c48e1e37ff8fb1cf6cc42cae31bd2da37d8.zip cpython-b5684c48e1e37ff8fb1cf6cc42cae31bd2da37d8.tar.gz cpython-b5684c48e1e37ff8fb1cf6cc42cae31bd2da37d8.tar.bz2 | |
Add the command line to the AssertionError raised by test.script_helper's
Python subprocess failure assertion error messages for easier debugging.
Adds a unittest for test.script_helper to confirm that this code works as
it is otherwise uncovered by an already passing test suite that uses it. :)
Diffstat (limited to 'Lib/test/script_helper.py')
| -rw-r--r-- | Lib/test/script_helper.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/Lib/test/script_helper.py b/Lib/test/script_helper.py index a7bb0d5..87a781e 100644 --- a/Lib/test/script_helper.py +++ b/Lib/test/script_helper.py @@ -51,8 +51,9 @@ def _assert_python(expected_success, *args, **env_vars): err = strip_python_stderr(err) if (rc and expected_success) or (not rc and not expected_success): raise AssertionError( - "Process return code is %d, " - "stderr follows:\n%s" % (rc, err.decode('ascii', 'ignore'))) + "Process return code is %d, command line was: %r, " + "stderr follows:\n%s" % (rc, cmd_line, + err.decode('ascii', 'ignore'))) return rc, out, err def assert_python_ok(*args, **env_vars): |