Skip to content

Commit

Permalink
Add EnvironmentVarGuard for test_pdb.py
Browse files Browse the repository at this point in the history
  • Loading branch information
WolframAlph committed Jan 5, 2025
1 parent 2228e92 commit a9022ff
Showing 1 changed file with 9 additions and 16 deletions.
25 changes: 9 additions & 16 deletions Lib/test/test_pdb.py
Original file line number Diff line number Diff line change
Expand Up @@ -3152,17 +3152,13 @@ def run_pdb_script(self, script, commands,
self.addCleanup(os_helper.unlink, '.pdbrc')
self.addCleanup(os_helper.unlink, filename)

homesave = None
if remove_home:
homesave = os.environ.pop('HOME', None)
try:
with os_helper.EnvironmentVarGuard() as env:
if remove_home:
env.unset('HOME')
if script_args is None:
script_args = []
stdout, stderr = self._run_pdb([filename] + script_args, commands, expected_returncode, extra_env)
finally:
if homesave is not None:
os.environ['HOME'] = homesave
return stdout, stderr
return stdout, stderr

def run_pdb_module(self, script, commands):
"""Runs the script code as part of a module"""
Expand Down Expand Up @@ -3585,17 +3581,14 @@ def test_readrc_kwarg(self):
self.assertIn("NameError: name 'invalid' is not defined", stdout)

def test_readrc_homedir(self):
save_home = os.environ.pop("HOME", None)
with os_helper.temp_dir() as temp_dir, patch("os.path.expanduser"):
rc_path = os.path.join(temp_dir, ".pdbrc")
os.path.expanduser.return_value = rc_path
try:
with os_helper.EnvironmentVarGuard() as env:
env.unset("HOME")
with os_helper.temp_dir() as temp_dir, patch("os.path.expanduser"):
rc_path = os.path.join(temp_dir, ".pdbrc")
os.path.expanduser.return_value = rc_path
with open(rc_path, "w") as f:
f.write("invalid")
self.assertEqual(pdb.Pdb().rcLines[0], "invalid")
finally:
if save_home is not None:
os.environ["HOME"] = save_home

def test_header(self):
stdout = StringIO()
Expand Down

0 comments on commit a9022ff

Please sign in to comment.