fix: add tons of log statements

Author: thomasrockhu-codecov

.github/workflows/build_assets.yml

@@ -2,6 +2,12 @@
 name: Build Compiled Assets
 
 on:
+  workflow_dispatch:
+    inputs:
+      release:
+        type: boolean
+        default: false
+        description: "Attach artifacts to a release"
   workflow_call:
     inputs:
       release:

codecov_cli/helpers/folder_searcher.py

@@ -57,14 +57,42 @@ def search_files(
         search_for_directories (bool)
 
     """
+    logger.debug(
+        "search_files",
+        extra=dict(
+            extra_log_attributes=dict(
+                local_args=locals(),
+            ),
+        ),
+    )
     this_is_included = functools.partial(
         _is_included, filename_include_regex, multipart_include_regex
     )
     this_is_excluded = functools.partial(
         _is_excluded, filename_exclude_regex, multipart_exclude_regex
     )
     for dirpath, dirnames, filenames in os.walk(folder_to_search):
+        logger.debug(
+            "search_files.os_walk",
+            extra=dict(
+                extra_log_attributes=dict(
+                    folder_to_search=folder_to_search,
+                    dirpath=dirpath,
+                    dirnames=dirnames,
+                    filenames=filenames,
+                ),
+            ),
+        )
         dirs_to_remove = set(d for d in dirnames if d in folders_to_ignore)
+        logger.debug(
+            "search_files.dirs_to_remove",
+            extra=dict(
+                extra_log_attributes=dict(
+                    dirs_to_remove=dirs_to_remove,
+                    folders_to_ignore=folders_to_ignore,
+                ),
+            ),
+        )
 
         if multipart_exclude_regex is not None:
             dirs_to_remove.union(
@@ -80,6 +108,16 @@ def search_files(
             # This is the documented way of doing this on python docs
             dirnames.remove(directory)
 
+        logger.debug(
+            "search_files.post_dir_removal",
+            extra=dict(
+                extra_log_attributes=dict(
+                    dirnames=dirnames,
+                    search_for_directories=search_for_directories,
+                ),
+            ),
+        )
+
         if search_for_directories:
             for directory in dirnames:
                 dir_path = pathlib.Path(dirpath) / directory
@@ -88,6 +126,18 @@ def search_files(
         else:
             for single_filename in filenames:
                 file_path = pathlib.Path(dirpath) / single_filename
+                logger.debug(
+                    "search_files.search_for_filenames",
+                    extra=dict(
+                        extra_log_attributes=dict(
+                            single_filename=single_filename,
+                            file_path=file_path,
+                            is_excluded=this_is_excluded(file_path),
+                            is_included=this_is_included(file_path),
+                            yield_it=(not this_is_excluded(file_path) and this_is_included(file_path)),
+                        ),
+                    ),
+                )
                 if not this_is_excluded(file_path) and this_is_included(file_path):
                     yield file_path
 

codecov_cli/helpers/versioning_systems.py

@@ -191,11 +191,50 @@ def get_fallback_value(self, fallback_field: FallbackFieldEnum):
     def list_relevant_files(
         self, directory: t.Optional[Path] = None, recurse_submodules: bool = False
     ) -> t.List[str]:
+        logger.debug(
+            "NoVersioningSystem.list_relevant_files.start",
+            extra=dict(
+                extra_log_attributes=dict(
+                    directory=directory,
+                    recurse_submodules=recurse_submodules,
+                ),
+            ),
+        )
         dir_to_use = directory or self.get_network_root()
+        logger.debug(
+            "NoVersioningSystem.list_relevant_files.dir_to_use",
+            extra=dict(
+                extra_log_attributes=dict(
+                    dir_to_use=dir_to_use,
+                ),
+            )
+        )
         if dir_to_use is None:
             raise ValueError("Can't determine root folder")
 
         files = search_files(
             dir_to_use, folders_to_ignore=[], filename_include_regex=re.compile("")
         )
-        return [f.relative_to(dir_to_use).as_posix() for f in files]
+        logger.debug(
+            "NoVersioningSystem.list_relevant_files.files",
+            extra=dict(
+                extra_log_attributes=dict(
+                    dir_to_use=dir_to_use,
+                    files=files,
+                ),
+            ),
+        )
+
+        return_files = []
+        for f in files:
+            return_files.append(f.relative_to(dir_to_use).as_posix())
+        logger.debug(
+            "NoVersioningSystem.list_relevant_files.return_files",
+            extra=dict(
+                extra_log_attributes=dict(
+                    dir_to_use=dir_to_use,
+                    return_files=return_files,
+                ),
+            ),
+        )
+        return return_files

codecov_cli/services/upload/network_finder.py

@@ -1,8 +1,11 @@
+import logging
 import pathlib
 import typing
 
 from codecov_cli.helpers.versioning_systems import VersioningSystemInterface
 
+logger = logging.getLogger("codecovcli")
+
 
 class NetworkFinder(object):
     def __init__(
@@ -24,13 +27,25 @@ def find_files(self, ignore_filters=False) -> typing.List[str]:
             self.network_root_folder, self.recurse_submodules
         )
 
+        logger.debug(
+            "NetworkFinder.find_files",
+            extra=dict(
+                extra_log_attributes=dict(
+                    files=files,
+                    ignore_filters=ignore_filters,
+                    network_root_folder=self.network_root_folder,
+                    recurse_submodules=self.recurse_submodules,
+                ),
+            ),
+        )
+
         if files and not ignore_filters:
             if self.network_filter:
                 files = [file for file in files if file.startswith(self.network_filter)]
             if self.network_prefix:
                 files = [self.network_prefix + file for file in files]
 
-        return files
+        return files or []
 
 
 def select_network_finder(