mirrors/cpython
1
0
Fork 0
mirror of https://github.com/python/cpython.git synced 2025-08-30 05:35:08 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 8

Close #12383: Fix subprocess module with env={}: don't copy the environment

Browse source 
variables, start with an empty environment.
This commit is contained in:
Victor Stinner 2011-06-21 17:18:38 +02:00
parent b7149cad04
commit f1512a2967
3 changed files with 20 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

23
Lib/test/test_subprocess.py
View file

@ -331,13 +331,22 @@ class ProcessTestCase(BaseTestCase):
def test_env(self):
newenv = os.environ.copy()
newenv["FRUIT"] = "orange"
p = subprocess.Popen([sys.executable, "-c",
'import sys,os;'
'sys.stdout.write(os.getenv("FRUIT"))'],
stdout=subprocess.PIPE,
env=newenv)
self.addCleanup(p.stdout.close)
self.assertEqual(p.stdout.read(), b"orange")
with subprocess.Popen([sys.executable, "-c",
'import sys,os;'
'sys.stdout.write(os.getenv("FRUIT"))'],
stdout=subprocess.PIPE,
env=newenv) as p:
stdout, stderr = p.communicate()
self.assertEqual(stdout, b"orange")
def test_empty_env(self):
with subprocess.Popen([sys.executable, "-c",
'import os; '
'print(len(os.environ))'],
stdout=subprocess.PIPE,
env={}) as p:
stdout, stderr = p.communicate()
self.assertEqual(stdout.strip(), b"0")
def test_communicate_stdin(self):
p = subprocess.Popen([sys.executable, "-c",