[Python-checkins] Fix the construction of subprocess.CalledProcessError in test_venv (GH-10400)

Miss Islington (bot) webhook-mailer at python.org
Wed Nov 7 17:46:14 EST 2018

commit: 0e3983689bc5c09ce9220c7976d94adf68ef118c
branch: 3.6
author: Miss Islington (bot) <31488909+miss-islington at users.noreply.github.com>
committer: GitHub <noreply at github.com>
date: 2018-11-07T14:46:10-08:00

Fix the construction of subprocess.CalledProcessError in test_venv (GH-10400)

The constructor of subprocess.CalledProcessError in the check_output function had an extra None in it.
(cherry picked from commit b93925047a025511c48a7bf3e6e6f0cfec79b8ed)

Co-authored-by: Pablo Galindo <Pablogsal at gmail.com>

M Lib/test/test_venv.py

diff --git a/Lib/test/test_venv.py b/Lib/test/test_venv.py
index d1ffb8132d81..842470fef08a 100644
--- a/Lib/test/test_venv.py
+++ b/Lib/test/test_venv.py
@@ -40,7 +40,7 @@ def check_output(cmd, encoding=None):
     out, err = p.communicate()
     if p.returncode:
         raise subprocess.CalledProcessError(
-            p.returncode, cmd, None, out, err)
+            p.returncode, cmd, out, err)
     return out, err
 class BaseTest(unittest.TestCase):

More information about the Python-checkins mailing list