Use OrderedDict when dumping ABOUT files

This ensure that we have a stable order on all OSes

Signed-off-by: Philippe Ombredanne <[email protected]>

Author: pombredanne

src/attributecode/model.py

@@ -1081,7 +1081,7 @@ def dumps(self, use_mapping=False, mapping_file=False, with_absent=False, with_e
         If with_absent, include absent (not present) fields.
         If with_empty, include empty fields.
         """
-        about_data = {}
+        about_data = OrderedDict()
         # Group the same license information (name, url, file) together
         license_key = []
         license_name = []
@@ -1110,7 +1110,7 @@ def dumps(self, use_mapping=False, mapping_file=False, with_absent=False, with_e
         # Group the same license information in a list
         license_group = list(zip_longest(license_key, license_name, license_file, license_url))
         for lic_group in license_group:
-            lic_dict = {}
+            lic_dict = OrderedDict()
             if lic_group[0]:
                 lic_dict['key'] = lic_group[0]
             if lic_group[1]:

tests/test_model.py

@@ -725,8 +725,8 @@ def test_About_dumps(self):
   An ABOUT file is a file.
 homepage_url: http://dejacode.org
 licenses:
-  - file: apache-2.0.LICENSE
-    key: apache-2.0
+  - key: apache-2.0
+    file: apache-2.0.LICENSE
 notice_file: NOTICE
 owner: nexB Inc.
 vcs_repository: https://github.com/dejacode/about-code-tool.git