Fix cwd which differs between the tests & real running

Author: ArBridgeman

exasol/toolbox/nox/_artifacts.py

@@ -7,7 +7,10 @@
 import sys
 from collections.abc import Iterable
 from pathlib import Path
-from typing import Optional
+from typing import (
+    Optional,
+    Union,
+)
 
 import nox
 from nox import Session
@@ -184,7 +187,9 @@ def _copy_artifacts(source: Path, dest: Path, files: Iterable[str]):
             print(f"File not found {path}", file=sys.stderr)
 
 
-def _prepare_coverage_xml(session: Session, source: Path) -> None:
+def _prepare_coverage_xml(
+    session: Session, source: Path, cwd: Union[Path | None] = None
+) -> None:
     """
     Prepare the coverage XML for input into Sonar
 
@@ -208,7 +213,7 @@ def _prepare_coverage_xml(session: Session, source: Path) -> None:
         f"{source}/*",
         "--fail-under=0",
     ]
-    output = subprocess.run(command, capture_output=True, text=True, cwd=source)
+    output = subprocess.run(command, capture_output=True, text=True, cwd=cwd)  # nosec
     if output.returncode != 0:
         if output.stdout.strip() == "No data to report.":
             # Assuming that previous steps passed in the CI, this indicates — as

test/unit/nox/_artifacts_test.py

@@ -361,7 +361,7 @@ def test_no_coverage_file_silently_passes(monkeypatch, tmp_path, nox_session):
         coverage_db = tmp_path / COVERAGE_DB
         monkeypatch.setattr(_artifacts, "COVERAGE_XML", coverage_xml)
 
-        _prepare_coverage_xml(nox_session, tmp_path)
+        _prepare_coverage_xml(nox_session, tmp_path, cwd=tmp_path)
 
         assert not Path(coverage_db).exists()
         assert not Path(coverage_xml).exists()
@@ -378,7 +378,7 @@ def test_that_bad_coverage_file_still_raises_error(
         with pytest.raises(
             _SessionQuit, match="doesn't seem to be a coverage data file"
         ):
-            _prepare_coverage_xml(nox_session, tmp_path)
+            _prepare_coverage_xml(nox_session, tmp_path, cwd=tmp_path)
 
         assert coverage_db.exists()
         assert not coverage_xml.exists()