update parent_path to display root files on empty string instead of None to align with the code format

Signed-off-by: Aayush Kumar <[email protected]>

Author: aayushkdev

scanpipe/migrations/0074_codebaseresource_parent_path_and_more.py

@@ -13,7 +13,7 @@ class Migration(migrations.Migration):
         migrations.AddField(
             model_name='codebaseresource',
             name='parent_path',
-            field=models.CharField(blank=True, help_text='The path of the resource\'s parent directory. Set to None for top-level (root) resources. Used to efficiently retrieve a directory\'s contents.', max_length=2000, null=True),
+            field=models.CharField(blank=True, help_text='The path of the resource\'s parent directory. Set to None for top-level (root) resources. Used to efficiently retrieve a directory\'s contents.', max_length=2000),
         ),
         migrations.AddIndex(
             model_name='codebaseresource',

scanpipe/models.py

@@ -2698,7 +2698,6 @@ class CodebaseResource(
 
     parent_path = models.CharField(
         max_length=2000,
-        null=True,
         blank=True,
         help_text=_(
             "The path of the resource's parent directory. "
@@ -2815,7 +2814,7 @@ def __str__(self):
 
     def save(self, *args, **kwargs):
         if self.path and not self.parent_path:
-            self.parent_path = self.parent_directory()
+            self.parent_path = self.parent_directory() or ""
         super().save(*args, **kwargs)
 
     def get_absolute_url(self):

scanpipe/pipes/__init__.py

@@ -74,7 +74,7 @@ def make_codebase_resource(project, location, save=True, **extra_fields):
     parent_path = str(relative_path.parent)
 
     if parent_path == ".":
-        parent_path = None
+        parent_path = ""
 
     try:
         resource_data = scancode.get_resource_info(location=str(location))

scanpipe/tests/test_models.py

@@ -1648,7 +1648,7 @@ def test_scanpipe_can_compute_compliance_alert_for_license_exceptions(self):
     def test_scanpipe_codebase_root_parent_path(self):
         resource1 = self.project1.codebaseresources.create(path="file")
 
-        self.assertIsNone(resource1.parent_path)
+        self.assertEqual("", resource1.parent_path)
 
     def test_scanpipe_codebase_regular_parent_path(self):
         resource2 = self.project1.codebaseresources.create(path="dir1/dir2/file")

scanpipe/tests/test_pipelines.py

@@ -879,7 +879,7 @@ 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_path=None)
+        top_level_resources = project1.codebaseresources.filter(parent_path="")
         top_level_paths = [resource.path for resource in top_level_resources]
 
         self.assertListEqual(top_level_paths, expected_top_level_paths)
@@ -905,7 +905,7 @@ def test_scanpipe_scan_codebase_creates_parent_path_field(self):
             "is-npm-1.0.0.tgz-extract/package/readme.md",
         ]
 
-        top_level_resources = project1.codebaseresources.filter(parent_path=None)
+        top_level_resources = project1.codebaseresources.filter(parent_path="")
         top_level_paths = [resource.path for resource in top_level_resources]
 
         self.assertListEqual(top_level_paths, expected_top_level_paths)