diff options
author | Gregory P. Smith <greg@krypto.org> | 2015-01-20 17:19:47 -0800 |
---|---|---|
committer | Gregory P. Smith <greg@krypto.org> | 2015-01-20 17:19:47 -0800 |
commit | b5684c48e1e37ff8fb1cf6cc42cae31bd2da37d8 (patch) | |
tree | 037b8c1c2016ce27dacc02f6d7a67643f90d9fea /Lib/test/script_helper.py | |
parent | b176d40398f4a6f15fc3f63ef55fb064eca13ee3 (diff) | |
download | cpython-git-b5684c48e1e37ff8fb1cf6cc42cae31bd2da37d8.tar.gz |
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 a7bb0d51b2..87a781e31a 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): |