Fixed #433

 * problem was caused by the different multi-lic file format  between
json and CSV (CSV with '\n' line break)

Author: chinyeungli

docs/CHANGELOG.rst

@@ -4,7 +4,8 @@
 	* Enhance the `transform` to also work with JSON file
     * Update transform code (See #427 and #428)
     * Fixed #431 - Error handling for empty "_file" fields
-    * Fixed #432 - Handled UTF-8 variant invented by Microsoft 
+    * Fixed #432 - Handled UTF-8 variant invented by Microsoft
+    * Fixed #433 - problem was caused by the different multi-lic file format  between json and CSV (CSV with '\n' line break)
 
 2020-05-05
     Release 4.0.2

src/attributecode/model.py

@@ -990,7 +990,6 @@ def load_dict(self, fields_dict, base_dir, running_inventory=False, reference_di
                 # 'Field licenses is a custom field.'
                 licenses_field = (key, value)
                 fields.remove(licenses_field)
-
         errors = self.process(
             fields=fields,
             about_file_path=self.about_file_path,
@@ -1033,7 +1032,12 @@ def dumps(self):
                 # Restore the original_value as it was parsed for
                 # validation purpose
                 if field.original_value:
-                    license_file = field.original_value.split('\n')
+                    # This line break is for the components that have multiple license
+                    # values in CSV format.
+                    if '\n' in field.original_value:
+                        license_file = field.original_value.split('\n')
+                    else:
+                        license_file = field.value.keys()
                 else:
                     license_file = field.value.keys()
             elif field.name == 'license_url' and field.value:

tests/test_model.py

@@ -676,6 +676,37 @@ def test_get_field_names_does_not_return_duplicates_custom_fields(self):
         result = model.get_field_names(abouts)
         assert expected == result
 
+    def test_load_dict_issue_433(self):
+        package_data = {
+            'about_resource': 'package1.zip',
+            'name': 'package',
+            'version': '1.0',
+            'copyright': 'copyright on package',
+            'license_expression': 'license1 AND license2',
+            'notice_file': 'package1.zip.NOTICE',
+            'licenses': [
+                {'key': 'license1', 'name': 'License1', 'file': 'license1.LICENSE'},
+                {'key': 'license2', 'name': 'License2', 'file': 'license2.LICENSE'},
+            ],
+        }
+        about = model.About()
+        about.load_dict(package_data, base_dir='')
+        as_dict = about.as_dict()
+        expected = '''about_resource: package1.zip
+name: package
+version: '1.0'
+license_expression: license1 AND license2
+copyright: copyright on package
+notice_file: package1.zip.NOTICE
+licenses:
+  - key: license1
+    name: License1
+    file: license1.LICENSE
+  - key: license2
+    name: License2
+    file: license2.LICENSE
+'''
+        assert about.dumps() == expected
 
 class SerializationTest(unittest.TestCase):
     def test_About_dumps(self):