Skip to content

Commit

Permalink
remove some defaulted PYTHON_EXE_ENV
Browse files Browse the repository at this point in the history
Signed-off-by: mayeut <[email protected]>
  • Loading branch information
mayeut committed Mar 25, 2023
1 parent 1ebdaae commit 249dadd
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 12 deletions.
1 change: 0 additions & 1 deletion psutil/tests/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -478,7 +478,6 @@ def pyrun(src, **kwds):
"""
kwds.setdefault("stdout", None)
kwds.setdefault("stderr", None)
kwds.setdefault("env", PYTHON_EXE_ENV)
srcfile = get_testfn()
try:
with open(srcfile, 'wt') as f:
Expand Down
4 changes: 1 addition & 3 deletions psutil/tests/test_posix.py
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@
from psutil.tests import CI_TESTING
from psutil.tests import HAS_NET_IO_COUNTERS
from psutil.tests import PYTHON_EXE
from psutil.tests import PYTHON_EXE_ENV
from psutil.tests import PsutilTestCase
from psutil.tests import mock
from psutil.tests import retry_on_failure
Expand Down Expand Up @@ -139,8 +138,7 @@ class TestProcess(PsutilTestCase):
@classmethod
def setUpClass(cls):
cls.pid = spawn_testproc([PYTHON_EXE, "-E", "-O"],
stdin=subprocess.PIPE,
env=PYTHON_EXE_ENV).pid
stdin=subprocess.PIPE).pid

@classmethod
def tearDownClass(cls):
Expand Down
16 changes: 8 additions & 8 deletions psutil/tests/test_process.py
Original file line number Diff line number Diff line change
Expand Up @@ -136,25 +136,25 @@ def test_wait_exited(self):
# Test waitpid() + WIFEXITED -> WEXITSTATUS.
# normal return, same as exit(0)
cmd = [PYTHON_EXE, "-c", "pass"]
p = self.spawn_psproc(cmd, env=PYTHON_EXE_ENV)
p = self.spawn_psproc(cmd)
code = p.wait()
self.assertEqual(code, 0)
self.assertProcessGone(p)
# exit(1), implicit in case of error
cmd = [PYTHON_EXE, "-c", "1 / 0"]
p = self.spawn_psproc(cmd, stderr=subprocess.PIPE, env=PYTHON_EXE_ENV)
p = self.spawn_psproc(cmd, stderr=subprocess.PIPE)
code = p.wait()
self.assertEqual(code, 1)
self.assertProcessGone(p)
# via sys.exit()
cmd = [PYTHON_EXE, "-c", "import sys; sys.exit(5);"]
p = self.spawn_psproc(cmd, env=PYTHON_EXE_ENV)
p = self.spawn_psproc(cmd)
code = p.wait()
self.assertEqual(code, 5)
self.assertProcessGone(p)
# via os._exit()
cmd = [PYTHON_EXE, "-c", "import os; os._exit(5);"]
p = self.spawn_psproc(cmd, env=PYTHON_EXE_ENV)
p = self.spawn_psproc(cmd)
code = p.wait()
self.assertEqual(code, 5)
self.assertProcessGone(p)
Expand Down Expand Up @@ -710,7 +710,7 @@ def test_exe(self):

def test_cmdline(self):
cmdline = [PYTHON_EXE, "-c", "import time; time.sleep(60)"]
p = self.spawn_psproc(cmdline, env=PYTHON_EXE_ENV)
p = self.spawn_psproc(cmdline)
# XXX - most of the times the underlying sysctl() call on Net
# and Open BSD returns a truncated string.
# Also /proc/pid/cmdline behaves the same so it looks
Expand All @@ -736,7 +736,7 @@ def test_long_cmdline(self):
self.assertEqual(p.cmdline(), cmdline)

def test_name(self):
p = self.spawn_psproc(PYTHON_EXE, env=PYTHON_EXE_ENV)
p = self.spawn_psproc(PYTHON_EXE)
name = p.name().lower()
pyexe = os.path.basename(os.path.realpath(sys.executable)).lower()
assert pyexe.startswith(name), (pyexe, name)
Expand Down Expand Up @@ -877,7 +877,7 @@ def test_cwd(self):
def test_cwd_2(self):
cmd = [PYTHON_EXE, "-c",
"import os, time; os.chdir('..'); time.sleep(60)"]
p = self.spawn_psproc(cmd, env=PYTHON_EXE_ENV)
p = self.spawn_psproc(cmd)
call_until(p.cwd, "ret == os.path.dirname(os.getcwd())")

@unittest.skipIf(not HAS_CPU_AFFINITY, 'not supported')
Expand Down Expand Up @@ -974,7 +974,7 @@ def test_open_files(self):

# another process
cmdline = "import time; f = open(r'%s', 'r'); time.sleep(60);" % testfn
p = self.spawn_psproc([PYTHON_EXE, "-c", cmdline], env=PYTHON_EXE_ENV)
p = self.spawn_psproc([PYTHON_EXE, "-c", cmdline])

for x in range(100):
filenames = [os.path.normcase(x.path) for x in p.open_files()]
Expand Down

0 comments on commit 249dadd

Please sign in to comment.