Fix tests

Author: ewjoachim

tests/unit/test_coverage.py

@@ -28,11 +28,11 @@ def test_get_coverage_info(mocker, coverage_json, coverage_obj):
         "coverage_comment.subprocess.run", return_value=json.dumps(coverage_json)
     )
 
-    result = coverage.get_coverage_info(merge=True, coverage_path=".")
+    result = coverage.get_coverage_info(merge=True, coverage_path=pathlib.Path("."))
 
     assert run.call_args_list == [
-        mocker.call("coverage", "combine", path="."),
-        mocker.call("coverage", "json", "-o", "-", path="."),
+        mocker.call("coverage", "combine", path=pathlib.Path(".")),
+        mocker.call("coverage", "json", "-o", "-", path=pathlib.Path(".")),
     ]
 
     assert result == coverage_obj
@@ -43,9 +43,12 @@ def test_get_coverage_info__no_merge(mocker, coverage_json):
         "coverage_comment.subprocess.run", return_value=json.dumps(coverage_json)
     )
 
-    coverage.get_coverage_info(merge=False, coverage_path=".")
+    coverage.get_coverage_info(merge=False, coverage_path=pathlib.Path("."))
 
-    assert mocker.call("coverage", "combine", path=".") not in run.call_args_list
+    assert (
+        mocker.call("coverage", "combine", path=pathlib.Path("."))
+        not in run.call_args_list
+    )
 
 
 def test_get_coverage_info__error_base(mocker, get_logs):
@@ -54,7 +57,7 @@ def test_get_coverage_info__error_base(mocker, get_logs):
     )
 
     with pytest.raises(subprocess.SubProcessError):
-        coverage.get_coverage_info(merge=False, coverage_path=".")
+        coverage.get_coverage_info(merge=False, coverage_path=pathlib.Path("."))
 
     assert not get_logs("ERROR")
 
@@ -66,7 +69,7 @@ def test_get_coverage_info__error_no_source(mocker, get_logs):
     )
 
     with pytest.raises(subprocess.SubProcessError):
-        coverage.get_coverage_info(merge=False, coverage_path=".")
+        coverage.get_coverage_info(merge=False, coverage_path=pathlib.Path("."))
 
     assert get_logs("ERROR", "Cannot read")
 
@@ -77,24 +80,33 @@ def test_generate_coverage_html_files(mocker):
     )
 
     coverage.generate_coverage_html_files(
-        destination=pathlib.Path("/tmp/foo"), coverage_path="."
+        destination=pathlib.Path("/tmp/foo"), coverage_path=pathlib.Path(".")
     )
 
     assert run.call_args_list == [
         mocker.call(
-            "coverage", "html", "--skip-empty", "--directory", "/tmp/foo", path="."
+            "coverage",
+            "html",
+            "--skip-empty",
+            "--directory",
+            "/tmp/foo",
+            path=pathlib.Path("."),
         ),
     ]
 
 
 def test_generate_coverage_markdown(mocker):
     run = mocker.patch("coverage_comment.subprocess.run", return_value="foo")
 
-    result = coverage.generate_coverage_markdown(coverage_path=".")
+    result = coverage.generate_coverage_markdown(coverage_path=pathlib.Path("."))
 
     assert run.call_args_list == [
         mocker.call(
-            "coverage", "report", "--format=markdown", "--show-missing", path="."
+            "coverage",
+            "report",
+            "--format=markdown",
+            "--show-missing",
+            path=pathlib.Path("."),
         ),
     ]
 

tests/unit/test_template.py

@@ -1,5 +1,6 @@
 import datetime
 import decimal
+import pathlib
 
 import pytest
 
@@ -86,8 +87,8 @@ def test_template_full():
             missing_branches=0,
         ),
         files={
-            "codebase/code.py": coverage.FileCoverage(
-                path="codebase/code.py",
+            pathlib.Path("codebase/code.py"): coverage.FileCoverage(
+                path=pathlib.Path("codebase/code.py"),
                 executed_lines=[1, 2, 5, 6, 9],
                 missing_lines=[],
                 excluded_lines=[],
@@ -103,8 +104,8 @@ def test_template_full():
                     missing_branches=0,
                 ),
             ),
-            "codebase/other.py": coverage.FileCoverage(
-                path="codebase/other.py",
+            pathlib.Path("codebase/other.py"): coverage.FileCoverage(
+                path=pathlib.Path("codebase/other.py"),
                 executed_lines=[1, 2, 3],
                 missing_lines=[],
                 excluded_lines=[],
@@ -129,13 +130,13 @@ def test_template_full():
         total_percent_covered=decimal.Decimal("1"),
         num_changed_lines=39,
         files={
-            "codebase/code.py": coverage.FileDiffCoverage(
-                path="codebase/code.py",
+            pathlib.Path("codebase/code.py"): coverage.FileDiffCoverage(
+                path=pathlib.Path("codebase/code.py"),
                 percent_covered=decimal.Decimal("0.5"),
                 violation_lines=[5],
             ),
-            "codebase/other.py": coverage.FileDiffCoverage(
-                path="codebase/other.py",
+            pathlib.Path("codebase/other.py"): coverage.FileDiffCoverage(
+                path=pathlib.Path("codebase/other.py"),
                 percent_covered=decimal.Decimal("1"),
                 violation_lines=[],
             ),