Patch platformdirs where it's used

* For reference: https://docs.python.org/dev/library/unittest.mock.html#where-to-patch https://bbguimaraes.com/blog/python-mocking.html Signed-off-by: João Gonçalves <[email protected]>
brightway-lca · Jun 13, 2024 · c426a1b · c426a1b
1 parent 14f216c
commit c426a1b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/tests/conftest.py b/tests/conftest.py
@@ -15,7 +15,5 @@ def fixtures_dir():
 
 @pytest.fixture(autouse=True)
 def temporary_logs_dir(monkeypatch, tmp_path):
-    def temp_user_log_dir(*args, **kwargs):
-        return tmp_path
-
-    monkeypatch.setattr(platformdirs, "user_log_dir", temp_user_log_dir)
+    monkeypatch.setattr("bw_simapro_csv.main.user_log_dir", lambda *args, **kwargs: tmp_path)
+    yield tmp_path
diff --git a/tests/test_header.py b/tests/test_header.py
@@ -1,9 +1,15 @@
 from datetime import datetime
+from pathlib import Path
 
 from bw_simapro_csv import SimaProCSV
 from bw_simapro_csv.header import SimaProCSVType
 
 
+def test_log_file_patching(fixtures_dir: Path, temporary_logs_dir: Path):
+    obj = SimaProCSV(fixtures_dir / "allocation.csv")
+    assert str(temporary_logs_dir) in str(obj.logs_dir)
+
+
 def test_basic_header_extraction(fixtures_dir):
     obj = SimaProCSV(fixtures_dir / "allocation.csv")
     assert obj.header == {