_get_env_run_update, update test_testdir_respects_monkeypatch

Author: blueyed

src/_pytest/pytester.py

@@ -855,7 +855,7 @@ def inline_run(self, *args, plugins=(), no_reraise_ctrlc=False):
         try:
             # Do not load user config (during runs only).
             mp_run = MonkeyPatch()
-            for k, v in self._env_run_update.items():
+            for k, v in self._get_env_run_update().items():
                 mp_run.setenv(k, v)
             finalizers.append(mp_run.undo)
 
@@ -1091,7 +1091,7 @@ def popen(
         env["PYTHONPATH"] = os.pathsep.join(
             filter(None, [os.getcwd(), env.get("PYTHONPATH", "")])
         )
-        env.update(self._env_run_update)
+        env.update(self._get_env_run_update())
         kw["env"] = env
 
         if stdin is Testdir.CLOSE_STDIN:
@@ -1263,13 +1263,17 @@ def spawn(self, cmd, expect_timeout=10.0):
 
         # Do not load user config.
         env = os.environ.copy()
-        env.update(self._env_run_update)
+        env.update(self._get_env_run_update())
 
         child = pexpect.spawn(cmd, logfile=logfile, env=env)
         self.request.addfinalizer(logfile.close)
         child.timeout = expect_timeout
         return child
 
+    def _get_env_run_update(self):
+        outer = {item[1] for item in self.monkeypatch._setitem if item[0] is os.environ}
+        return {k: v for k, v in self._env_run_update.items() if k not in outer}
+
 
 def getdecoded(out):
     try:

testing/test_pytester.py

@@ -551,8 +551,18 @@ def test_pytester_addopts_before_testdir(request, monkeypatch):
     assert os.environ.get("PYTEST_ADDOPTS") == orig
 
 
-def test_testdir_respects_monkeypatch(testdir, monkeypatch):
+@pytest.mark.parametrize("method", ("setenv", "delenv"))
+def test_testdir_respects_monkeypatch(method, testdir, monkeypatch):
     assert monkeypatch is testdir.monkeypatch
+    tmphome = str(testdir.tmpdir)
+    assert testdir._env_run_update["HOME"] == tmphome
+    assert testdir._get_env_run_update()["HOME"] == tmphome
+    if method == "setenv":
+        monkeypatch.setenv("HOME", "anotherhome")
+    else:
+        assert method == "delenv"
+        monkeypatch.delenv("HOME", raising=False)
+    assert "HOME" not in testdir._get_env_run_update()
 
 
 def test_run_stdin(testdir):