Fixed colcon ignore detection

Author: Joschi3

package_xml_validation/helpers/workspace.py

@@ -182,8 +182,8 @@ def get_pkgs_in_wrs(path: Path) -> list[str]:
 def _is_ignored_dir(path: Path) -> bool:
     """
     Return True if `path` or any ancestor contains an ignore marker:
-    - 'coclon_ignore' (as requested)
-    - 'COLCON_IGNORE' (colcon's standard)
+    - 'colcon_ignore'
+    - 'COLCON_IGNORE'
 
     Args:
         path: Directory path to inspect.
@@ -194,7 +194,7 @@ def _is_ignored_dir(path: Path) -> bool:
     """
     path = path.resolve()
     for parent in (path, *path.parents):
-        if (parent / "coclon_ignore").exists() or (parent / "COLCON_IGNORE").exists():
+        if (parent / "colcon_ignore").exists() or (parent / "COLCON_IGNORE").exists():
             return True
     return False
 

tests/test_workspace.py

@@ -167,9 +167,7 @@ def test_find_workspace_root_raises_outside_ws(self):
 
     # --- pkg_iterator -------------------------------------------------------
     def test_pkg_iterator_lists_pkgs_and_respects_colcon_ignore(self):
-        self.t.print_tree()  # Debug: print the temp tree structure
         pkgs = SUT.pkg_iterator(self.ws / "src")
-        print(pkgs.keys())
         self.assertEqual(set(pkgs.keys()), {"inner_pkg", "pkg1", "pkg_nameless"})
         self.assertNotIn("pkg_ignored", pkgs)
         self.assertNotIn("third_pkg", pkgs)