Properly mock the environment.

canonical · Sep 22, 2024 · 9020d5d · 9020d5d
1 parent f012b5e
commit 9020d5d
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/test/test_model.py b/test/test_model.py
@@ -3242,7 +3242,7 @@ def test_relation_remote_app_name_env(self, monkeypatch: pytest.MonkeyPatch):
         monkeypatch.setenv('JUJU_RELATION_ID', 'x:5')
         monkeypatch.setenv('JUJU_REMOTE_APP', 'remoteapp1')
         assert self.backend.relation_remote_app_name(5) == 'remoteapp1'
-        os.environ['JUJU_RELATION_ID'] = '5'
+        monkeypatch.setenv('JUJU_RELATION_ID', '5')
         assert self.backend.relation_remote_app_name(5) == 'remoteapp1'
 
     def test_relation_remote_app_name_script_success(
@@ -3266,7 +3266,7 @@ def test_relation_remote_app_name_script_success(
 
         # JUJU_RELATION_ID unset but JUJU_REMOTE_APP set
         monkeypatch.delenv('JUJU_RELATION_ID')
-        os.environ['JUJU_REMOTE_APP'] = 'remoteapp1'
+        monkeypatch.setenv('JUJU_REMOTE_APP', 'remoteapp1')
         assert self.backend.relation_remote_app_name(5) == 'remoteapp2'
 
         # Both set, but JUJU_RELATION_ID a different relation