rename parent_directory_path field to parent_path

Signed-off-by: Aayush Kumar <aayush214.kumar@gmail.com>

Author: aayushkdev

scanpipe/migrations/0073_codebaseresource_parent_directory_path_and_more.py renamed to scanpipe/migrations/0073_codebaseresource_parent_path_and_more.py

@@ -1,4 +1,4 @@
-# Generated by Django 5.1.9 on 2025-06-14 10:11
+# Generated by Django 5.1.9 on 2025-06-16 17:42
 
 from django.db import migrations, models
 
@@ -12,11 +12,11 @@ class Migration(migrations.Migration):
     operations = [
         migrations.AddField(
             model_name='codebaseresource',
-            name='parent_directory_path',
+            name='parent_path',
             field=models.CharField(blank=True, help_text='Path of the immediate parent directory of a resource. For top level resources the value is set to None', max_length=2000, null=True),
         ),
         migrations.AddIndex(
             model_name='codebaseresource',
-            index=models.Index(fields=['project', 'parent_directory_path'], name='scanpipe_co_project_f4a24b_idx'),
+            index=models.Index(fields=['project', 'parent_path'], name='scanpipe_co_project_008448_idx'),
         ),
     ]

scanpipe/models.py

@@ -2696,7 +2696,7 @@ class CodebaseResource(
         ),
     )
 
-    parent_directory_path = models.CharField(
+    parent_path = models.CharField(
         max_length=2000,
         null=True,
         blank=True,
@@ -2799,7 +2799,7 @@ class Meta:
             models.Index(fields=["compliance_alert"]),
             models.Index(fields=["is_binary"]),
             models.Index(fields=["is_text"]),
-            models.Index(fields=["project", "parent_directory_path"]),
+            models.Index(fields=["project", "parent_path"]),
         ]
         constraints = [
             models.UniqueConstraint(
@@ -2813,8 +2813,8 @@ def __str__(self):
         return self.path
 
     def save(self, *args, **kwargs):
-        if self.path and not self.parent_directory_path:
-            self.parent_directory_path = parent_directory(
+        if self.path and not self.parent_path:
+            self.parent_path = parent_directory(
                 str(self.path), with_trail=False
             )
         super().save(*args, **kwargs)

scanpipe/pipes/__init__.py

@@ -96,7 +96,7 @@ def make_codebase_resource(project, location, save=True, **extra_fields):
     codebase_resource = CodebaseResource(
         project=project,
         path=relative_path,
-        parent_directory_path=parent_path,
+        parent_path=parent_path,
         **resource_data,
     )
 

scanpipe/tests/test_models.py

@@ -1645,15 +1645,15 @@ def test_scanpipe_can_compute_compliance_alert_for_license_exceptions(self):
         resource.update(detected_license_expression=license_expression)
         self.assertEqual("warning", resource.compute_compliance_alert())
 
-    def test_scanpipe_codebase_root_parent_directory_path(self):
+    def test_scanpipe_codebase_root_parent_path(self):
         resource1 = self.project1.codebaseresources.create(path="file")
 
-        self.assertEqual("", resource1.parent_directory_path)
+        self.assertEqual("", resource1.parent_path)
 
-    def test_scanpipe_codebase_regular_parent_directory_path(self):
+    def test_scanpipe_codebase_regular_parent_path(self):
         resource2 = self.project1.codebaseresources.create(path="dir1/dir2/file")
 
-        self.assertEqual("dir1/dir2", resource2.parent_directory_path)
+        self.assertEqual("dir1/dir2", resource2.parent_path)
 
     def test_scanpipe_scan_fields_model_mixin_methods(self):
         expected = [

scanpipe/tests/test_pipelines.py

@@ -880,13 +880,13 @@ def test_scanpipe_scan_codebase_creates_top_level_paths(self):
         expected_top_level_paths = ["is-npm-1.0.0.tgz", "is-npm-1.0.0.tgz-extract"]
 
         top_level_resources = project1.codebaseresources.filter(
-            parent_directory_path=None
+            parent_path=None
         )
         top_level_paths = [res.path for res in top_level_resources]
 
         self.assertListEqual(top_level_paths, expected_top_level_paths)
 
-    def test_scanpipe_scan_codebase_creates_parent_directory_path_field(self):
+    def test_scanpipe_scan_codebase_creates_parent_path_field(self):
         pipeline_name = "scan_codebase"
         project1 = make_project()
 
@@ -908,14 +908,14 @@ def test_scanpipe_scan_codebase_creates_parent_directory_path_field(self):
         ]
 
         top_level_resources = project1.codebaseresources.filter(
-            parent_directory_path=None
+            parent_path=None
         )
         top_level_paths = [res.path for res in top_level_resources]
 
         self.assertListEqual(top_level_paths, expected_top_level_paths)
 
         nested_resources = project1.codebaseresources.filter(
-            parent_directory_path="is-npm-1.0.0.tgz-extract/package"
+            parent_path="is-npm-1.0.0.tgz-extract/package"
         )
         nested_paths = [res.path for res in nested_resources]