use monkeypatch to avoid rouge env vars

Author: bisgaard-itis

clients/python/test/test_osparc/test_basic.py

@@ -1,9 +1,8 @@
 import json
-import os
 import subprocess
 from pathlib import Path
 from typing import Any, Dict, List, Set
-
+from faker import Faker
 import osparc
 import osparc._settings
 import pydantic
@@ -75,15 +74,17 @@ def test_dependencies(tmp_path: Path):
 
 
 @pytest.mark.parametrize("valid", [True, False])
-def test_parent_project_validation(faker, valid: bool):
+def test_parent_project_validation(
+    faker: Faker, monkeypatch: pytest.MonkeyPatch, valid: bool
+):
     if valid:
-        os.environ["OSPARC_STUDY_ID"] = f"{faker.uuid4()}"
-        os.environ["OSPARC_NODE_ID"] = f"{faker.uuid4()}"
+        monkeypatch.setenv("OSPARC_STUDY_ID", faker.uuid4())
+        monkeypatch.setenv("OSPARC_NODE_ID", faker.uuid4())
         parent_info = osparc._settings.ParentProjectInfo()
         assert parent_info.x_simcore_parent_project_uuid is not None
         assert parent_info.x_simcore_parent_node_id is not None
     else:
-        os.environ["OSPARC_STUDY_ID"] = f"{faker.text()}"
-        os.environ["OSPARC_NODE_ID"] = f"{faker.text()}"
+        monkeypatch.setenv("OSPARC_STUDY_ID", faker.text())
+        monkeypatch.setenv("OSPARC_NODE_ID", faker.text())
         with pytest.raises(pydantic.ValidationError):
             _ = osparc._settings.ParentProjectInfo()