Test on all OSs

This adds Windows and MacOS to our CI test matrix.

Author: Swatinem

.github/workflows/ci.yml

@@ -70,16 +70,12 @@ jobs:
           codecovcli create-report -t ${{ secrets.CODECOV_TOKEN }} --git-service github
 
   build-test-upload:
-    runs-on: ubuntu-latest
     strategy:
       fail-fast: false
       matrix:
-        include:
-          - python-version: "3.13"
-          - python-version: "3.12"
-          - python-version: "3.11"
-          - python-version: "3.10"
-          - python-version: "3.9"
+        os: [ubuntu-latest, windows-latest, macos-latest]
+        python-version: ["3.13", "3.12", "3.11", "3.10", "3.9"]
+    runs-on: ${{matrix.os}}
     steps:
       - uses: actions/checkout@v4
         with:
@@ -97,7 +93,7 @@ jobs:
           pip install -r tests/requirements.txt
       - name: Test with pytest
         run: |
-          pytest --cov --junitxml=${{matrix.python-version}}junit.xml
+          pytest --cov --junitxml=${{matrix.os}}-${{matrix.python-version}}junit.xml
         env:
           CODECOV_ENV: test
       - name: Dogfooding codecov-cli
@@ -109,8 +105,8 @@ jobs:
         if: ${{ !cancelled() }}
         uses: actions/upload-artifact@v4
         with:
-          name: ${{matrix.python-version}}junit.xml
-          path: ${{matrix.python-version}}junit.xml
+          name: ${{matrix.os}}-${{matrix.python-version}}junit.xml
+          path: ${{matrix.os}}-${{matrix.python-version}}junit.xml
 
   process-test-results:
     if: ${{ always() }}

codecov_cli/helpers/folder_searcher.py

@@ -17,7 +17,7 @@ def _is_included(
 ):
     return filename_include_regex.match(path.name) and (
         multipart_include_regex is None
-        or multipart_include_regex.match(str(path.resolve()))
+        or multipart_include_regex.match(path.resolve().as_posix())
     )
 
 
@@ -29,7 +29,8 @@ def _is_excluded(
     return (
         filename_exclude_regex is not None and filename_exclude_regex.match(path.name)
     ) or (
-        multipart_exclude_regex is not None and multipart_exclude_regex.match(str(path))
+        multipart_exclude_regex is not None
+        and multipart_exclude_regex.match(path.as_posix())
     )
 
 
@@ -69,7 +70,9 @@ def search_files(
             dirs_to_remove.union(
                 directory
                 for directory in dirnames
-                if multipart_exclude_regex.match(str(pathlib.Path(dirpath) / directory))
+                if multipart_exclude_regex.match(
+                    (pathlib.Path(dirpath) / directory).as_posix()
+                )
             )
 
         for directory in dirs_to_remove:

codecov_cli/helpers/versioning_systems.py

@@ -12,38 +12,38 @@
 logger = logging.getLogger("codecovcli")
 
 IGNORE_DIRS = [
-    '*.egg-info',
-    '.DS_Store',
-    '.circleci',
-    '.env',
-    '.envs',
-    '.git',
-    '.gitignore',
-    '.mypy_cache',
-    '.nvmrc',
-    '.nyc_output',
-    '.ruff_cache',
-    '.venv',
-    '.venvns',
-    '.virtualenv',
-    '.virtualenvs',
-    '__pycache__',
-    'bower_components',
-    'build/lib/',
-    'jspm_packages',
-    'node_modules',
-    'vendor',
-    'virtualenv',
-    'virtualenvs',
+    "*.egg-info",
+    ".DS_Store",
+    ".circleci",
+    ".env",
+    ".envs",
+    ".git",
+    ".gitignore",
+    ".mypy_cache",
+    ".nvmrc",
+    ".nyc_output",
+    ".ruff_cache",
+    ".venv",
+    ".venvns",
+    ".virtualenv",
+    ".virtualenvs",
+    "__pycache__",
+    "bower_components",
+    "build/lib/",
+    "jspm_packages",
+    "node_modules",
+    "vendor",
+    "virtualenv",
+    "virtualenvs",
 ]
 
 IGNORE_PATHS = [
-    '*.gif',
-    '*.jpeg',
-    '*.jpg',
-    '*.md',
-    '*.png',
-    'shunit2*',
+    "*.gif",
+    "*.jpeg",
+    "*.jpg",
+    "*.md",
+    "*.png",
+    "shunit2*",
 ]
 
 
@@ -203,12 +203,20 @@ def list_relevant_files(
 
         cmd = [
             "find",
-            dir_to_use,
-            *chain.from_iterable(["-name", block, "-prune", "-o"] for block in IGNORE_DIRS),
-            *chain.from_iterable(["-path", block, "-prune", "-o"] for block in IGNORE_PATHS),
+            str(dir_to_use),
+            *chain.from_iterable(
+                ["-name", block, "-prune", "-o"] for block in IGNORE_DIRS
+            ),
+            *chain.from_iterable(
+                ["-path", block, "-prune", "-o"] for block in IGNORE_PATHS
+            ),
             "-type",
             "f",
             "-print",
         ]
         res = subprocess.run(cmd, capture_output=True)
-        return [filename for filename in res.stdout.decode("unicode_escape").strip().split("\n") if filename]
+        return [
+            filename
+            for filename in res.stdout.decode("unicode_escape").strip().split("\n")
+            if filename
+        ]

codecov_cli/plugins/xcode.py

@@ -42,66 +42,70 @@ def run_preparation(self, collector) -> PreparationPluginReturn:
 
             filename_include_regex = globs_to_regex(["*.profdata"])
 
-            matched_paths = [
-                str(path)
-                for path in search_files(
+            matched_paths = list(
+                search_files(
                     folder_to_search=self.derived_data_folder,
                     folders_to_ignore=[],
                     filename_include_regex=filename_include_regex,
                 )
-            ]
+            )
+
             if not matched_paths:
                 logger.warning("No swift data found.")
                 return
 
             logger.info(
                 "Running swift coverage on the following list of files:",
-                extra=dict(extra_log_attributes=dict(matched_paths=matched_paths)),
+                extra=dict(
+                    extra_log_attributes=dict(
+                        matched_paths=[p.as_posix() for p in matched_paths]
+                    )
+                ),
             )
 
             for path in matched_paths:
                 self.swiftcov(path, self.app_name)
 
             return PreparationPluginReturn(success=True, messages="")
 
-    def swiftcov(self, path, app_name: str):
+    def swiftcov(self, path: pathlib.Path, app_name: str) -> None:
         directory = os.path.dirname(path)
         build_dir = pathlib.Path(re.sub("(Build).*", "Build", directory))
 
         for type in ["app", "framework", "xctest"]:
             filename_include_regex = re.compile(translate(f"*.{type}"))
-            matched_dir_paths = [
-                str(path)
-                for path in search_files(
-                    folder_to_search=pathlib.Path(build_dir),
-                    folders_to_ignore=[],
-                    filename_include_regex=filename_include_regex,
-                    search_for_directories=True,
-                )
-            ]
+            matched_dir_paths = search_files(
+                folder_to_search=build_dir,
+                folders_to_ignore=[],
+                filename_include_regex=filename_include_regex,
+                search_for_directories=True,
+            )
+
             for dir_path in matched_dir_paths:
                 # proj name without extension
-                proj = pathlib.Path(dir_path).stem
+                proj = dir_path.stem
                 if app_name == "" or (app_name.lower() in proj.lower()):
                     logger.info(f"+ Building reports for {proj} {type}")
-                    proj_path = pathlib.Path(pathlib.Path(dir_path) / proj)
+                    proj_path = dir_path / proj
                     dest = (
                         proj_path
                         if proj_path.is_file()
-                        else pathlib.Path(f"{dir_path}/Contents/MacOS/{proj}")
+                        else dir_path / "Contents/MacOS/{proj}"
                     )
                     output_file_name = f"{proj}.{type}.coverage.txt".replace(" ", "")
                     self.run_llvm_cov(output_file_name, path, dest)
 
-    def run_llvm_cov(self, output_file_name, path, dest):
+    def run_llvm_cov(
+        self, output_file_name: str, path: pathlib.Path, dest: pathlib.Path
+    ) -> None:
         with open(output_file_name, "w") as output_file:
             s = subprocess.run(
                 [
                     "xcrun",
                     "llvm-cov",
                     "show",
                     "-instr-profile",
-                    path,
+                    str(path),
                     str(dest),
                 ],
                 stdout=output_file,

codecov_cli/services/upload/upload_sender.py

@@ -176,7 +176,7 @@ def _get_file_fixers(
             total_fixed_lines = list(
                 file_fixer.fixed_lines_without_reason.union(fixed_lines_with_reason)
             )
-            file_fixers[str(file_fixer.path)] = {
+            file_fixers[file_fixer.path.as_posix()] = {
                 "eof": file_fixer.eof,
                 "lines": total_fixed_lines,
             }

tests/helpers/test_args.py

@@ -1,7 +1,8 @@
 import os
-from pathlib import PosixPath
+import sys
 
 import click
+import pytest
 
 from codecov_cli import __version__
 from codecov_cli.helpers.args import get_cli_args
@@ -28,7 +29,10 @@ def test_get_cli_args():
     assert get_cli_args(ctx) == expected
 
 
+@pytest.mark.skipif(sys.platform == "win32", reason="requires posix platform")
 def test_get_cli_args_with_posix():
+    from pathlib import PosixPath
+
     ctx = click.Context(click.Command("do-upload"))
     ctx.obj = {}
     ctx.obj["cli_args"] = {