codecov-cli 10.1.1__tar.gz → 10.2.1__tar.gz

{codecov_cli-10.1.1/codecov_cli.egg-info → codecov_cli-10.2.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.2
 Name: codecov-cli
-Version: 10.1.1
+Version: 10.2.1
 Summary: Codecov Command Line Interface
 Author-email: Tom Hu <thomas.hu@sentry.io>
 Maintainer-email: Codecov Support <support@codecov.io>
@@ -216,7 +216,7 @@ Requires-Dist: pyyaml==6.*
 Requires-Dist: regex
 Requires-Dist: responses==0.21.*
 Requires-Dist: sentry-sdk>=2.20.0
-Requires-Dist: test-results-parser==0.5.*
+Requires-Dist: test-results-parser==0.5.4
 Requires-Dist: tree-sitter==0.20.*
 Requires-Dist: wrapt>=1.17.2
 
@@ -392,6 +392,7 @@ Codecov-cli supports user input. These inputs, along with their descriptions and
 |-s, --dir, --coverage-files-search-root-folder | Folder where to search for coverage files default: (Current Working Directory) | Optional
 |--exclude, --coverage-files-search-exclude-folder | Folders to exclude from search | Optional
 |-f, --file, --coverage-files-search-direct-file | Explicit files to upload | Optional
+|--recurse-submodules | Whether to enumerate files inside of submodules for path-fixing purposes. Off by default. | Optional
 |--disable-search | Disable search for coverage files. This is helpful when specifying what files you want to upload with the --file option.| Optional
 |-b, --build, --build-code | Specify the build number manually | Optional
 |--build-url | The URL of the build where this is running | Optional

{codecov_cli-10.1.1 → codecov_cli-10.2.1}/README.md

@@ -170,6 +170,7 @@ Codecov-cli supports user input. These inputs, along with their descriptions and
 |-s, --dir, --coverage-files-search-root-folder | Folder where to search for coverage files default: (Current Working Directory) | Optional
 |--exclude, --coverage-files-search-exclude-folder | Folders to exclude from search | Optional
 |-f, --file, --coverage-files-search-direct-file | Explicit files to upload | Optional
+|--recurse-submodules | Whether to enumerate files inside of submodules for path-fixing purposes. Off by default. | Optional
 |--disable-search | Disable search for coverage files. This is helpful when specifying what files you want to upload with the --file option.| Optional
 |-b, --build, --build-code | Specify the build number manually | Optional
 |--build-url | The URL of the build where this is running | Optional

{codecov_cli-10.1.1 → codecov_cli-10.2.1}/codecov_cli/commands/commit.py

@@ -24,7 +24,7 @@ logger = logging.getLogger("codecovcli")
     "--pr",
     "--pull-request-number",
     "pull_request_number",
-    help="Specify the pull request number mannually. Used to override pre-existing CI environment variables",
+    help="Specify the pull request number manually. Used to override pre-existing CI environment variables",
     cls=CodecovOption,
     fallback_field=FallbackFieldEnum.pull_request_number,
 )

{codecov_cli-10.1.1 → codecov_cli-10.2.1}/codecov_cli/commands/empty_upload.py

@@ -6,8 +6,8 @@ import sentry_sdk
 
 from codecov_cli.fallbacks import CodecovOption, FallbackFieldEnum
 from codecov_cli.helpers.args import get_cli_args
-from codecov_cli.helpers.git import GitService
 from codecov_cli.helpers.options import global_options
+from codecov_cli.services.commit import create_commit_logic
 from codecov_cli.services.empty_upload import empty_upload_logic
 from codecov_cli.types import CommandContext
 
@@ -16,6 +16,26 @@ logger = logging.getLogger("codecovcli")
 
 @click.command()
 @click.option("--force", is_flag=True, default=False)
+@click.option(
+    "--parent-sha",
+    help="SHA (with 40 chars) of what should be the parent of this commit",
+)
+@click.option(
+    "-P",
+    "--pr",
+    "--pull-request-number",
+    "pull_request_number",
+    help="Specify the pull request number manually. Used to override pre-existing CI environment variables",
+    cls=CodecovOption,
+    fallback_field=FallbackFieldEnum.pull_request_number,
+)
+@click.option(
+    "-B",
+    "--branch",
+    help="Branch to which this commit belongs to",
+    cls=CodecovOption,
+    fallback_field=FallbackFieldEnum.branch,
+)
 @global_options
 @click.pass_context
 def empty_upload(
@@ -26,11 +46,29 @@ def empty_upload(
     token: typing.Optional[str],
     git_service: typing.Optional[str],
     fail_on_error: typing.Optional[bool],
+    parent_sha: typing.Optional[str],
+    pull_request_number: typing.Optional[int],
+    branch: typing.Optional[str],
 ):
     with sentry_sdk.start_transaction(op="task", name="Empty Upload"):
         with sentry_sdk.start_span(name="empty_upload"):
             enterprise_url = ctx.obj.get("enterprise_url")
             args = get_cli_args(ctx)
+
+            logger.debug("Attempting to Create Commit before doing an empty upload.")
+            create_commit_logic(
+                commit_sha,
+                parent_sha,
+                pull_request_number,
+                branch,
+                slug,
+                token,
+                git_service,
+                enterprise_url,
+                fail_on_error,
+                args,
+            )
+
             logger.debug(
                 "Starting empty upload process",
                 extra=dict(

{codecov_cli-10.1.1 → codecov_cli-10.2.1}/codecov_cli/commands/report.py

@@ -21,7 +21,7 @@ logger = logging.getLogger("codecovcli")
     "--pr",
     "--pull-request-number",
     "pull_request_number",
-    help="Specify the pull request number mannually. Used to override pre-existing CI environment variables",
+    help="Specify the pull request number manually. Used to override pre-existing CI environment variables",
     cls=CodecovOption,
     fallback_field=FallbackFieldEnum.pull_request_number,
 )

{codecov_cli-10.1.1 → codecov_cli-10.2.1}/codecov_cli/commands/upload.py

@@ -66,6 +66,12 @@ _global_upload_options = [
         multiple=True,
         default=[],
     ),
+    click.option(
+        "--recurse-submodules",
+        help="Whether to enumerate files inside of submodules for path-fixing purposes. Off by default.",
+        is_flag=True,
+        default=False,
+    ),
     click.option(
         "--disable-search",
         help="Disable search for coverage files. This is helpful when specifying what files you want to upload with the --file option.",
@@ -118,7 +124,7 @@ _global_upload_options = [
         "--pr",
         "--pull-request-number",
         "pull_request_number",
-        help="Specify the pull request number mannually. Used to override pre-existing CI environment variables",
+        help="Specify the pull request number manually. Used to override pre-existing CI environment variables",
         cls=CodecovOption,
         fallback_field=FallbackFieldEnum.pull_request_number,
     ),
@@ -242,6 +248,7 @@ def do_upload(
     network_root_folder: pathlib.Path,
     plugin_names: typing.List[str],
     pull_request_number: typing.Optional[str],
+    recurse_submodules: bool,
     report_type_str: str,
     slug: typing.Optional[str],
     swift_project: typing.Optional[str],
@@ -297,6 +304,7 @@ def do_upload(
                 network_root_folder=network_root_folder,
                 plugin_names=plugin_names,
                 pull_request_number=pull_request_number,
+                recurse_submodules=recurse_submodules,
                 report_code=report_code,
                 slug=slug,
                 swift_project=swift_project,

{codecov_cli-10.1.1 → codecov_cli-10.2.1}/codecov_cli/commands/upload_coverage.py

@@ -56,6 +56,7 @@ def upload_coverage(
     parent_sha: typing.Optional[str],
     plugin_names: typing.List[str],
     pull_request_number: typing.Optional[str],
+    recurse_submodules: bool,
     report_code: str,
     report_type_str: str,
     slug: typing.Optional[str],
@@ -115,6 +116,7 @@ def upload_coverage(
                     parent_sha=parent_sha,
                     plugin_names=plugin_names,
                     pull_request_number=pull_request_number,
+                    recurse_submodules=recurse_submodules,
                     report_code=report_code,
                     slug=slug,
                     swift_project=swift_project,
@@ -173,6 +175,7 @@ def upload_coverage(
                     network_root_folder=network_root_folder,
                     plugin_names=plugin_names,
                     pull_request_number=pull_request_number,
+                    recurse_submodules=recurse_submodules,
                     report_code=report_code,
                     report_type_str=report_type_str,
                     slug=slug,

{codecov_cli-10.1.1 → codecov_cli-10.2.1}/codecov_cli/commands/upload_process.py

@@ -54,6 +54,7 @@ def upload_process(
     parent_sha: typing.Optional[str],
     plugin_names: typing.List[str],
     pull_request_number: typing.Optional[str],
+    recurse_submodules: bool,
     report_code: str,
     report_type_str: str,
     slug: typing.Optional[str],
@@ -122,6 +123,7 @@ def upload_process(
                 network_root_folder=network_root_folder,
                 plugin_names=plugin_names,
                 pull_request_number=pull_request_number,
+                recurse_submodules=recurse_submodules,
                 report_code=report_code,
                 report_type_str=report_type_str,
                 slug=slug,

{codecov_cli-10.1.1 → codecov_cli-10.2.1}/codecov_cli/helpers/versioning_systems.py

@@ -25,7 +25,7 @@ class VersioningSystemInterface(ABC):
 
     @abstractmethod
     def list_relevant_files(
-        self, directory: t.Optional[Path] = None
+        self, directory: t.Optional[Path] = None, recurse_submodules: bool = False
     ) -> t.Optional[t.List[str]]:
         pass
 
@@ -125,14 +125,17 @@ class GitVersioningSystem(VersioningSystemInterface):
             return Path(p.stdout.decode().rstrip())
         return None
 
-    def list_relevant_files(self, directory: t.Optional[Path] = None) -> t.List[str]:
+    def list_relevant_files(
+        self, directory: t.Optional[Path] = None, recurse_submodules: bool = False
+    ) -> t.List[str]:
         dir_to_use = directory or self.get_network_root()
         if dir_to_use is None:
             raise ValueError("Can't determine root folder")
 
-        res = subprocess.run(
-            ["git", "-C", str(dir_to_use), "ls-files"], capture_output=True
-        )
+        cmd = ["git", "-C", str(dir_to_use), "ls-files"]
+        if recurse_submodules:
+            cmd.append("--recurse-submodules")
+        res = subprocess.run(cmd, capture_output=True)
 
         return [
             (
@@ -155,5 +158,7 @@ class NoVersioningSystem(VersioningSystemInterface):
     def get_fallback_value(self, fallback_field: FallbackFieldEnum):
         return None
 
-    def list_relevant_files(self, directory: t.Optional[Path] = None) -> t.List[str]:
+    def list_relevant_files(
+        self, directory: t.Optional[Path] = None, recurse_submodules: bool = False
+    ) -> t.List[str]:
         return []

{codecov_cli-10.1.1 → codecov_cli-10.2.1}/codecov_cli/services/upload/__init__.py

@@ -56,6 +56,7 @@ def do_upload_logic(
     parent_sha: typing.Optional[str] = None,
     plugin_names: typing.List[str],
     pull_request_number: typing.Optional[str],
+    recurse_submodules: bool = False,
     report_code: str,
     slug: typing.Optional[str],
     swift_project: typing.Optional[str],
@@ -87,6 +88,7 @@ def do_upload_logic(
     )
     network_finder = select_network_finder(
         versioning_system,
+        recurse_submodules=recurse_submodules,
         network_filter=network_filter,
         network_prefix=network_prefix,
         network_root_folder=network_root_folder,

{codecov_cli-10.1.1 → codecov_cli-10.2.1}/codecov_cli/services/upload/network_finder.py

@@ -8,17 +8,21 @@ class NetworkFinder(object):
     def __init__(
         self,
         versioning_system: VersioningSystemInterface,
+        recurse_submodules: bool,
         network_filter: typing.Optional[str],
         network_prefix: typing.Optional[str],
         network_root_folder: pathlib.Path,
     ):
         self.versioning_system = versioning_system
+        self.recurse_submodules = recurse_submodules
         self.network_filter = network_filter
         self.network_prefix = network_prefix
         self.network_root_folder = network_root_folder
 
     def find_files(self, ignore_filters=False) -> typing.List[str]:
-        files = self.versioning_system.list_relevant_files(self.network_root_folder)
+        files = self.versioning_system.list_relevant_files(
+            self.network_root_folder, self.recurse_submodules
+        )
 
         if files and not ignore_filters:
             if self.network_filter:
@@ -31,12 +35,14 @@ class NetworkFinder(object):
 
 def select_network_finder(
     versioning_system: VersioningSystemInterface,
+    recurse_submodules: bool,
     network_filter: typing.Optional[str],
     network_prefix: typing.Optional[str],
     network_root_folder: pathlib.Path,
 ):
     return NetworkFinder(
         versioning_system,
+        recurse_submodules,
         network_filter,
         network_prefix,
         network_root_folder,

{codecov_cli-10.1.1 → codecov_cli-10.2.1}/codecov_cli/services/upload_coverage/__init__.py

@@ -16,6 +16,7 @@ def upload_coverage_logic(
     build_code: typing.Optional[str],
     build_url: typing.Optional[str],
     commit_sha: str,
+    recurse_submodules: bool,
     disable_file_fixes: bool,
     disable_search: bool,
     dry_run: bool,
@@ -58,6 +59,7 @@ def upload_coverage_logic(
         build_code=build_code,
         build_url=build_url,
         commit_sha=commit_sha,
+        recurse_submodules=recurse_submodules,
         disable_file_fixes=disable_file_fixes,
         disable_search=disable_search,
         dry_run=dry_run,

{codecov_cli-10.1.1 → codecov_cli-10.2.1/codecov_cli.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.2
 Name: codecov-cli
-Version: 10.1.1
+Version: 10.2.1
 Summary: Codecov Command Line Interface
 Author-email: Tom Hu <thomas.hu@sentry.io>
 Maintainer-email: Codecov Support <support@codecov.io>
@@ -216,7 +216,7 @@ Requires-Dist: pyyaml==6.*
 Requires-Dist: regex
 Requires-Dist: responses==0.21.*
 Requires-Dist: sentry-sdk>=2.20.0
-Requires-Dist: test-results-parser==0.5.*
+Requires-Dist: test-results-parser==0.5.4
 Requires-Dist: tree-sitter==0.20.*
 Requires-Dist: wrapt>=1.17.2
 
@@ -392,6 +392,7 @@ Codecov-cli supports user input. These inputs, along with their descriptions and
 |-s, --dir, --coverage-files-search-root-folder | Folder where to search for coverage files default: (Current Working Directory) | Optional
 |--exclude, --coverage-files-search-exclude-folder | Folders to exclude from search | Optional
 |-f, --file, --coverage-files-search-direct-file | Explicit files to upload | Optional
+|--recurse-submodules | Whether to enumerate files inside of submodules for path-fixing purposes. Off by default. | Optional
 |--disable-search | Disable search for coverage files. This is helpful when specifying what files you want to upload with the --file option.| Optional
 |-b, --build, --build-code | Specify the build number manually | Optional
 |--build-url | The URL of the build where this is running | Optional

{codecov_cli-10.1.1 → codecov_cli-10.2.1}/codecov_cli.egg-info/SOURCES.txt

@@ -156,6 +156,7 @@ tests/ci_adapters/test_teamcity.py
 tests/ci_adapters/test_travis_ci.py
 tests/ci_adapters/test_woodpeckerci.py
 tests/commands/__init__.py
+tests/commands/test_invoke_empty_upload.py
 tests/commands/test_invoke_labelanalysis.py
 tests/commands/test_invoke_upload.py
 tests/commands/test_invoke_upload_coverage.py

{codecov_cli-10.1.1 → codecov_cli-10.2.1}/codecov_cli.egg-info/requires.txt

@@ -5,6 +5,6 @@ pyyaml==6.*
 regex
 responses==0.21.*
 sentry-sdk>=2.20.0
-test-results-parser==0.5.*
+test-results-parser==0.5.4
 tree-sitter==0.20.*
 wrapt>=1.17.2

{codecov_cli-10.1.1 → codecov_cli-10.2.1}/pyproject.toml

@@ -18,7 +18,7 @@ dependencies = [
     "regex",
     "responses==0.21.*",
     "sentry-sdk>=2.20.0",
-    "test-results-parser==0.5.*",
+    "test-results-parser==0.5.4",
     "tree-sitter==0.20.*",
     "wrapt>=1.17.2",
 ]
@@ -26,7 +26,7 @@ license = {file = "LICENSE"}
 name = "codecov-cli"
 readme = "README.md"
 requires-python = ">= 3.9"
-version = "10.1.1"
+version = "10.2.1"
 
 [project.scripts]
 codecov = "codecov_cli.main:run"

{codecov_cli-10.1.1 → codecov_cli-10.2.1}/requirements.txt

@@ -47,7 +47,7 @@ sniffio==1.3.1
     # via
     #   anyio
     #   httpx
-test-results-parser==0.5.1
+test-results-parser==0.5.4
     # via codecov-cli (pyproject.toml)
 tree-sitter==0.20.4
     # via codecov-cli (pyproject.toml)

codecov_cli-10.2.1/tests/commands/test_invoke_empty_upload.py

@@ -0,0 +1,25 @@
+from click.testing import CliRunner
+
+from codecov_cli.fallbacks import FallbackFieldEnum
+from codecov_cli.main import cli
+from tests.factory import FakeProvider, FakeVersioningSystem
+
+def test_invoke_empty_upload_with_create_commit(mocker):
+    create_commit_mock = mocker.patch("codecov_cli.commands.empty_upload.create_commit_logic")
+    empty_upload_mock = mocker.patch("codecov_cli.commands.empty_upload.empty_upload_logic")
+
+    fake_ci_provider = FakeProvider({FallbackFieldEnum.commit_sha: None})
+    mocker.patch("codecov_cli.main.get_ci_adapter", return_value=fake_ci_provider)
+
+    runner = CliRunner()
+    result = runner.invoke(cli, ["empty-upload",
+                    "-C", "command-sha",
+                    "--slug", "owner/repo",
+                    "--parent-sha", "asdf",
+                    "--branch", "main",
+                    "--pr", 1234], obj={})
+    assert result.exit_code == 0
+
+    create_commit_mock.assert_called_once()
+    empty_upload_mock.assert_called_once()
+

{codecov_cli-10.1.1 → codecov_cli-10.2.1}/tests/commands/test_invoke_upload_coverage.py

@@ -91,6 +91,9 @@ def test_upload_coverage_options(mocker):
             "                                  wish to only upload the specified files,",
             "                                  please consider using --disable-search to",
             "                                  disable uploading other files.",
+            "  --recurse-submodules            Whether to enumerate files inside of",
+            "                                  submodules for path-fixing purposes. Off by",
+            "                                  default.",
             "  --disable-search                Disable search for coverage files. This is",
             "                                  helpful when specifying what files you want to",
             "                                  upload with the --file option.",
@@ -103,8 +106,8 @@ def test_upload_coverage_options(mocker):
             "                                  Codecov UI",
             "  -B, --branch TEXT               Branch to which this commit belongs to",
             "  -P, --pr, --pull-request-number TEXT",
-            "                                  Specify the pull request number mannually.",
-            "                                  Used to override pre-existing CI environment",
+            "                                  Specify the pull request number manually. Used",
+            "                                  to override pre-existing CI environment",
             "                                  variables",
             "  -e, --env, --env-var TEXT       Specify environment variables to be included",
             "                                  with this build.",

{codecov_cli-10.1.1 → codecov_cli-10.2.1}/tests/commands/test_invoke_upload_process.py

@@ -91,6 +91,9 @@ def test_upload_process_options(mocker):
             "                                  wish to only upload the specified files,",
             "                                  please consider using --disable-search to",
             "                                  disable uploading other files.",
+            "  --recurse-submodules            Whether to enumerate files inside of",
+            "                                  submodules for path-fixing purposes. Off by",
+            "                                  default.",
             "  --disable-search                Disable search for coverage files. This is",
             "                                  helpful when specifying what files you want to",
             "                                  upload with the --file option.",
@@ -103,8 +106,8 @@ def test_upload_process_options(mocker):
             "                                  Codecov UI",
             "  -B, --branch TEXT               Branch to which this commit belongs to",
             "  -P, --pr, --pull-request-number TEXT",
-            "                                  Specify the pull request number mannually.",
-            "                                  Used to override pre-existing CI environment",
+            "                                  Specify the pull request number manually. Used",
+            "                                  to override pre-existing CI environment",
             "                                  variables",
             "  -e, --env, --env-var TEXT       Specify environment variables to be included",
             "                                  with this build.",

{codecov_cli-10.1.1 → codecov_cli-10.2.1}/tests/helpers/test_network_finder.py

@@ -15,6 +15,7 @@ def test_find_files(mocker, tmp_path):
     assert (
         NetworkFinder(
             versioning_system=mocked_vs,
+            recurse_submodules=False,
             network_filter=None,
             network_prefix=None,
             network_root_folder=tmp_path,
@@ -24,6 +25,7 @@ def test_find_files(mocker, tmp_path):
     assert (
         NetworkFinder(
             versioning_system=mocked_vs,
+            recurse_submodules=False,
             network_filter="hello",
             network_prefix="bello",
             network_root_folder=tmp_path,
@@ -33,13 +35,14 @@ def test_find_files(mocker, tmp_path):
     assert (
         NetworkFinder(
             versioning_system=mocked_vs,
+            recurse_submodules=False,
             network_filter="hello",
             network_prefix="bello",
             network_root_folder=tmp_path,
         ).find_files(True)
         == filenames
     )
-    mocked_vs.list_relevant_files.assert_called_with(tmp_path)
+    mocked_vs.list_relevant_files.assert_called_with(tmp_path, False)
 
 
 def test_find_files_with_filter(mocker, tmp_path):
@@ -52,6 +55,7 @@ def test_find_files_with_filter(mocker, tmp_path):
     assert (
         NetworkFinder(
             versioning_system=mocked_vs,
+            recurse_submodules=False,
             network_filter="hello",
             network_prefix=None,
             network_root_folder=tmp_path,
@@ -61,13 +65,14 @@ def test_find_files_with_filter(mocker, tmp_path):
     assert (
         NetworkFinder(
             versioning_system=mocked_vs,
+            recurse_submodules=False,
             network_filter="hello",
             network_prefix="bello",
             network_root_folder=tmp_path,
         ).find_files(True)
         == filenames
     )
-    mocked_vs.list_relevant_files.assert_called_with(tmp_path)
+    mocked_vs.list_relevant_files.assert_called_with(tmp_path, False)
 
 
 def test_find_files_with_prefix(mocker, tmp_path):
@@ -80,6 +85,7 @@ def test_find_files_with_prefix(mocker, tmp_path):
     assert (
         NetworkFinder(
             versioning_system=mocked_vs,
+            recurse_submodules=False,
             network_filter=None,
             network_prefix="hello",
             network_root_folder=tmp_path,
@@ -89,13 +95,14 @@ def test_find_files_with_prefix(mocker, tmp_path):
     assert (
         NetworkFinder(
             versioning_system=mocked_vs,
+            recurse_submodules=False,
             network_filter="hello",
             network_prefix="bello",
             network_root_folder=tmp_path,
         ).find_files(True)
         == filenames
     )
-    mocked_vs.list_relevant_files.assert_called_with(tmp_path)
+    mocked_vs.list_relevant_files.assert_called_with(tmp_path, False)
 
 
 def test_find_files_with_filter_and_prefix(mocker, tmp_path):
@@ -108,6 +115,7 @@ def test_find_files_with_filter_and_prefix(mocker, tmp_path):
     assert (
         NetworkFinder(
             versioning_system=mocked_vs,
+            recurse_submodules=False,
             network_filter="hello",
             network_prefix="bello",
             network_root_folder=tmp_path,
@@ -117,10 +125,11 @@ def test_find_files_with_filter_and_prefix(mocker, tmp_path):
     assert (
         NetworkFinder(
             versioning_system=mocked_vs,
+            recurse_submodules=False,
             network_filter="hello",
             network_prefix="bello",
             network_root_folder=tmp_path,
         ).find_files(True)
         == filenames
     )
-    mocked_vs.list_relevant_files.assert_called_with(tmp_path)
+    mocked_vs.list_relevant_files.assert_called_with(tmp_path, False)

{codecov_cli-10.1.1 → codecov_cli-10.2.1}/tests/helpers/test_versioning_systems.py

@@ -130,3 +130,14 @@ class TestGitVersioningSystem(object):
         vs = GitVersioningSystem()
         with pytest.raises(ValueError):
             vs.list_relevant_files()
+
+    def test_list_relevant_files_recurse_submodules(self, mocker, tmp_path):
+        subproc_run = mocker.patch(
+            "codecov_cli.helpers.versioning_systems.subprocess.run"
+        )
+        vs = GitVersioningSystem()
+        _ = vs.list_relevant_files(tmp_path, recurse_submodules=True)
+        subproc_run.assert_called_with(
+            ["git", "-C", str(tmp_path), "ls-files", "--recurse-submodules"],
+            capture_output=True,
+        )

{codecov_cli-10.1.1 → codecov_cli-10.2.1}/tests/services/upload/test_upload_collector.py

@@ -167,7 +167,7 @@ def test_generate_upload_data(tmp_path):
 
     file_finder = FileFinder(tmp_path)
 
-    network_finder = NetworkFinder(GitVersioningSystem(), None, None, None)
+    network_finder = NetworkFinder(GitVersioningSystem(), False, None, None, None)
 
     collector = UploadCollector([], network_finder, file_finder, None)
 
@@ -187,7 +187,7 @@ def test_generate_upload_data_with_none_network(
     (tmp_path / "coverage.xml").touch()
 
     file_finder = FileFinder(tmp_path)
-    network_finder = NetworkFinder(NoVersioningSystem(), None, None, None)
+    network_finder = NetworkFinder(NoVersioningSystem(), False, None, None, None)
 
     collector = UploadCollector([], network_finder, file_finder, {})
 

{codecov_cli-10.1.1 → codecov_cli-10.2.1}/tests/services/upload/test_upload_service.py

@@ -104,6 +104,7 @@ def test_do_upload_logic_happy_path_legacy_uploader(mocker):
     )
     mock_select_network_finder.assert_called_with(
         versioning_system,
+        recurse_submodules=False,
         network_filter=None,
         network_prefix=None,
         network_root_folder=None,
@@ -218,6 +219,7 @@ def test_do_upload_logic_happy_path(mocker):
     )
     mock_select_network_finder.assert_called_with(
         versioning_system,
+        recurse_submodules=False,
         network_filter=None,
         network_prefix=None,
         network_root_folder=None,
@@ -309,6 +311,7 @@ def test_do_upload_logic_dry_run(mocker):
     )
     mock_select_network_finder.assert_called_with(
         versioning_system,
+        recurse_submodules=False,
         network_filter=None,
         network_prefix=None,
         network_root_folder=None,
@@ -501,6 +504,7 @@ def test_do_upload_no_cov_reports_found(mocker):
     )
     mock_select_network_finder.assert_called_with(
         versioning_system,
+        recurse_submodules=False,
         network_filter=None,
         network_prefix=None,
         network_root_folder=None,
@@ -597,6 +601,7 @@ def test_do_upload_rase_no_cov_reports_found_error(mocker):
     )
     mock_select_network_finder.assert_called_with(
         versioning_system,
+        recurse_submodules=False,
         network_filter=None,
         network_prefix=None,
         network_root_folder=None,
@@ -678,6 +683,7 @@ def test_do_upload_logic_happy_path_test_results(mocker):
     )
     mock_select_network_finder.assert_called_with(
         versioning_system,
+        recurse_submodules=False,
         network_filter="some_dir",
         network_prefix="hello/",
         network_root_folder="root/",