Do not assign None to codebase attributes #48

Signed-off-by: Jono Yang <jyang@nexb.com>

Author: JonoYang

src/commoncode/resource.py

@@ -1840,6 +1840,8 @@ def _populate(self, scan_data):
         ##########################################################
         for attr_name in self.codebase_attributes:
             value = scan_data.get(attr_name)
+            if value == None:
+                continue
             setattr(self.attributes, attr_name, value)
 
         ##########################################################

tests/test_resource.py

@@ -12,6 +12,8 @@
 from os.path import exists
 from os.path import join
 
+import attr
+
 from commoncode.fileutils import parent_directory
 from commoncode.resource import Codebase
 from commoncode.resource import Resource
@@ -661,6 +663,17 @@ def test_VirtualCodebase_with_paths_works(self):
         )
         check_against_expected_json_file(results, expected_file, regen=False)
 
+    def test_VirtualCodebase_codebase_attributes_assignment(self):
+        test_codebase = self.get_test_loc('resource/with_path/virtual-codebase.json')
+        vc = VirtualCodebase(
+            location=test_codebase,
+            codebase_attributes=dict(
+                packages=attr.ib(default=attr.Factory(list))
+            ),
+        )
+        self.assertNotEqual(vc.attributes.packages, None)
+        self.assertEqual(vc.attributes.packages, [])
+
 
 class TestCodebaseCache(FileBasedTesting):
     test_data_dir = join(dirname(__file__), 'data')