TST: Add test for the multi-file adapter.

Author: DWesl

src/check_python_h_first/get_submodule_paths.py

@@ -13,8 +13,21 @@ def get_submodule_paths():
     Get paths to submodules so that we can exclude them from things like
     check_test_name.py, check_unicode.py, etc.
     """
-    root_directory = os.path.dirname(os.path.dirname(__file__))
+    # Find the git repo root
+    root_directory = os.getcwd()
+    git_dir = os.path.join(root_directory, ".git")
+    while not os.path.exists(git_dir):
+        next_root = os.path.abspath(os.path.join(root_directory, ".."))
+        if next_root == root_directory:
+            break
+        else:
+            root_directory = next_root
+        git_dir = os.path.join(root_directory, ".git")
+
+    # Check for submodules
     gitmodule_file = os.path.join(root_directory, ".gitmodules")
+    if not os.path.exists(gitmodule_file):
+        return []
     with open(gitmodule_file) as gitmodules:
         data = gitmodules.read().split("\n")
         submodule_paths = [

src/check_python_h_first/wrapper.py

@@ -12,7 +12,7 @@
 import subprocess
 import sys
 
-from . import get_submodule_paths
+from .get_submodule_paths import get_submodule_paths
 from .single_file import check_python_h_included_first
 
 C_CPP_EXTENSIONS = (".c", ".h", ".cpp", ".hpp", ".cc", ".hh", ".cxx", ".hxx")

tests/test_wrapper.py

@@ -3,7 +3,7 @@
 import glob
 import os.path
 
-from check_python_h_first.wrapper import find_c_cpp_files, sort_order
+from check_python_h_first.wrapper import find_c_cpp_files, process_files, sort_order
 
 THIS_DIR = os.path.dirname(__file__)
 
@@ -21,3 +21,15 @@ def test_find_c_cpp_files():
     """Test that the function can find all the files."""
     result = find_c_cpp_files(THIS_DIR)
     assert set(result) == set(HEADER_LIST + SOURCE_LIST)
+
+
+def test_process_files():
+    """Test that process files detects the number of failures.
+
+    Will break down if one file has multiple failures.
+    """
+    result = process_files(HEADER_LIST + SOURCE_LIST)
+    assert result == sum(
+        os.path.basename(name).startswith("system")
+        for name in HEADER_LIST + SOURCE_LIST
+    )