Fix multiple path bug in windows

Signed-off-by: Ayan Sinha Mahapatra <asmahapatra@aboutcode.org>

Author: AyanSinhaMahapatra

configure.bat

@@ -163,7 +163,7 @@ if %ERRORLEVEL% neq 0 (
     %FLOT_REQUIREMENTS%
 
 "%CFG_BIN_DIR%\flot" --pyproject pyproject-commoncode.toml
-"%CFG_BIN_DIR%\pip" install ./dist/commoncode*.whl
+"%CFG_BIN_DIR%\pip" install --force-reinstall ./dist/commoncode-32.5.2-py3-none-any.whl
 
 "%CFG_BIN_DIR%\pip" install ^
     --upgrade ^

src/commoncode/resource.py

@@ -49,6 +49,7 @@
 from commoncode.fileutils import file_name
 from commoncode.fileutils import parent_directory
 from commoncode.fileutils import splitext_name
+from commoncode.system import to_os_native_path
 
 """
 This module provides Codebase and Resource objects as an abstraction for files
@@ -64,7 +65,7 @@
 
 # Tracing flags
 TRACE = False
-TRACE_DEEP = False
+TRACE_DEEP = True
 
 
 def logger_debug(*args):
@@ -615,6 +616,9 @@ def err(_error):
                 if not created.is_file:
                     parents_by_loc[created.location] = created
 
+        if TRACE_DEEP:
+            logger_debug(f"parents_by_loc: {parents_by_loc}")
+
         # we start walking through all the input locations
         for included_location in includes:
             # Walk over the directory and build the resource tree
@@ -624,7 +628,12 @@ def err(_error):
                 max_depth=self.max_depth,
                 error_handler=err,
             ):
-                parent = parents_by_loc.pop(top)
+                if TRACE_DEEP:
+                    logger_debug(f"parents_by_loc: {parents_by_loc}")
+                try:
+                    parent = parents_by_loc.pop(top)
+                except KeyError:
+                    raise Exception(parents_by_loc, includes, root.location, )
                 for created in self._create_resources(
                     parent=parent,
                     top=top,
@@ -669,7 +678,7 @@ def _create_resources_common_prefix_to_inputs(self, root, includes):
             _, _, extra_dir_path = included_path.rpartition(root.location)
             extra_dirs = extra_dir_path.strip("/").split("/")
             if TRACE_DEEP:
-                    logger_debug(f"_create_resources_common_prefix_to_inputs: root:{root.location}, includes: {includes}")
+                logger_debug(f"_create_resources_common_prefix_to_inputs: included_path:{included_path}, extra_dirs: {extra_dirs}")
 
             dir_resource = root
             for dir_segment in extra_dirs:
@@ -678,8 +687,9 @@ def _create_resources_common_prefix_to_inputs(self, root, includes):
                     parent=dir_resource,
                     is_file=False,
                 )
-                if TRACE:
-                    logger_debug("Codebase.create_resources:", dir_resource)
+                if TRACE_DEEP:
+                    logger_debug(f"_create_resources_common_prefix_to_inputs: dir_resource:{dir_resource}, extra_dirs: {extra_dirs}")
+
                 yield dir_resource
 
     def _create_root_resource(self):

src/commoncode/system.py

@@ -13,6 +13,15 @@
 from commoncode.distro import parse_os_release
 
 
+def to_os_native_path(path):
+    """
+    Normalize a path to use the native OS path separator.
+    """
+    OS_PATH_SEP = "\\" if on_windows else "/"
+
+    return path.replace("/", OS_PATH_SEP).replace("\\", OS_PATH_SEP).rstrip(OS_PATH_SEP)
+
+
 def os_arch():
     """
     Return a tuple for the current the OS and architecture.

src/commoncode/testcase.py

@@ -29,7 +29,7 @@
 from commoncode.archive import extract_zip_raw
 from commoncode.archive import tar_can_extract  # NOQA
 from commoncode.system import on_posix
-from commoncode.system import on_windows
+from commoncode.system import to_os_native_path
 
 # a base test dir specific to a given test run
 # to ensure that multiple tests run can be launched in parallel
@@ -39,15 +39,6 @@
 timing_threshold = sys.maxsize
 
 
-def to_os_native_path(path):
-    """
-    Normalize a path to use the native OS path separator.
-    """
-    OS_PATH_SEP = "\\" if on_windows else "/"
-
-    return path.replace("/", OS_PATH_SEP).replace("\\", OS_PATH_SEP).rstrip(OS_PATH_SEP)
-
-
 def get_test_loc(
     test_path,
     test_data_dir,