Merge pull request #58 from nexB/fix-virtualcodebase-creation-bug

Fix virtualcodebase creation bug

Author: JonoYang

src/commoncode/resource.py

@@ -2034,12 +2034,13 @@ def _get_parent_directory(self, path_segments):
 
         current = self.root
         for segment in path_segments:
-            existing = resources_by_path.get(segment)
-            if not existing:
+            path = posixpath_join(current.path, segment)
+            existing = resources_by_path.get(path)
+            if not existing or existing == Codebase.CACHED_RESOURCE:
                 existing = self._get_or_create_resource(
                     name=segment,
                     # build the path based on parent
-                    path=posixpath_join(current.path, segment),
+                    path=path,
                     parent=current,
                     is_file=False,
                 )

tests/test_resource.py

@@ -1451,6 +1451,18 @@ def test_VirtualCodebase_can_be_created_without_RecursionError(self):
         test_file = self.get_test_loc('resource/virtual_codebase/zephyr-binary.json')
         VirtualCodebase(test_file)
 
+    def test_VirtualCodebase_can_be_created_with_repeated_root_directory(self):
+        paths = [
+            'to',
+            'to/to',
+            'to/to/to',
+            'to/to/to/to',
+        ]
+        resources = [{'path': path} for path in paths]
+        vc = VirtualCodebase(location={'files': resources})
+        walked_paths = [r.path for r in vc.walk()]
+        assert paths == walked_paths
+
 
 class TestResource(FileBasedTesting):
     test_data_dir = join(dirname(__file__), 'data')