Sort holidays by 'date', 'description', and then 'region'

Before, order seemed too dependent on call order Signed-off-by: Thomas Lauf <thomas.lauf@tngtech.com>
GothenburgBitFactory · Nov 21, 2023 · 243207f · 243207f
1 parent 5537a6d
commit 243207f
Show file tree

Hide file tree

Showing 88 changed files with 1,073 additions and 1,073 deletions.
diff --git a/src/holidata/emitters.py b/src/holidata/emitters.py
@@ -25,7 +25,7 @@ class JsonEmitter(Emitter):
 
     def output(self, holidays):
         export_data = [h.as_dict() for h in holidays]
-        export_data.sort(key=lambda x: x["date"])
+        export_data.sort(key=lambda x: (x["date"], x["description"], x["region"]))
         return "\n".join([json.dumps(h, ensure_ascii=False, sort_keys=False, indent=None, separators=(",", ":")) for h in export_data]) + "\n"
 
 
@@ -34,7 +34,7 @@ class CsvEmitter(Emitter):
 
     def output(self, holidays):
         export_data = [h.as_dict() for h in holidays]
-        export_data.sort(key=lambda x: x["date"])
+        export_data.sort(key=lambda x: (x["date"], x["description"], x["region"]))
         result = io.StringIO()
 
         writer = csv.DictWriter(result,
@@ -52,7 +52,7 @@ class YamlEmitter(Emitter):
 
     def output(self, holidays):
         export_data = [h.as_dict() for h in holidays]
-        export_data.sort(key=lambda x: x["date"])
+        export_data.sort(key=lambda x: (x["date"], x["description"], x["region"]))
 
         output = "%YAML 1.1\n"
         output += "---\n"
@@ -76,7 +76,7 @@ class XmlEmitter(Emitter):
 
     def output(self, holidays):
         export_data = [h.as_dict() for h in holidays]
-        export_data.sort(key=lambda x: x["date"])
+        export_data.sort(key=lambda x: (x["date"], x["description"], x["region"]))
 
         output = "<?xml version=\"1.0\" encoding=\"UTF-8\" ?>\n"
         output += "<holidays>\n"

diff --git a/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2011] 1.py b/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2011] 1.py
@@ -1,15 +1,15 @@
 [
     {
         'date': '2011-01-01',
-        'description': 'Nový rok',
+        'description': 'Den obnovy samostatného českého státu',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',
         'type': 'NF'
     },
     {
         'date': '2011-01-01',
-        'description': 'Den obnovy samostatného českého státu',
+        'description': 'Nový rok',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',

diff --git a/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2012] 1.py b/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2012] 1.py
@@ -1,15 +1,15 @@
 [
     {
         'date': '2012-01-01',
-        'description': 'Nový rok',
+        'description': 'Den obnovy samostatného českého státu',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',
         'type': 'NF'
     },
     {
         'date': '2012-01-01',
-        'description': 'Den obnovy samostatného českého státu',
+        'description': 'Nový rok',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',

diff --git a/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2013] 1.py b/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2013] 1.py
@@ -1,15 +1,15 @@
 [
     {
         'date': '2013-01-01',
-        'description': 'Nový rok',
+        'description': 'Den obnovy samostatného českého státu',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',
         'type': 'NF'
     },
     {
         'date': '2013-01-01',
-        'description': 'Den obnovy samostatného českého státu',
+        'description': 'Nový rok',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',

diff --git a/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2014] 1.py b/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2014] 1.py
@@ -1,15 +1,15 @@
 [
     {
         'date': '2014-01-01',
-        'description': 'Nový rok',
+        'description': 'Den obnovy samostatného českého státu',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',
         'type': 'NF'
     },
     {
         'date': '2014-01-01',
-        'description': 'Den obnovy samostatného českého státu',
+        'description': 'Nový rok',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',

diff --git a/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2015] 1.py b/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2015] 1.py
@@ -1,15 +1,15 @@
 [
     {
         'date': '2015-01-01',
-        'description': 'Nový rok',
+        'description': 'Den obnovy samostatného českého státu',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',
         'type': 'NF'
     },
     {
         'date': '2015-01-01',
-        'description': 'Den obnovy samostatného českého státu',
+        'description': 'Nový rok',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',

diff --git a/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2016] 1.py b/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2016] 1.py
@@ -1,15 +1,15 @@
 [
     {
         'date': '2016-01-01',
-        'description': 'Nový rok',
+        'description': 'Den obnovy samostatného českého státu',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',
         'type': 'NF'
     },
     {
         'date': '2016-01-01',
-        'description': 'Den obnovy samostatného českého státu',
+        'description': 'Nový rok',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',

diff --git a/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2017] 1.py b/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2017] 1.py
@@ -1,15 +1,15 @@
 [
     {
         'date': '2017-01-01',
-        'description': 'Nový rok',
+        'description': 'Den obnovy samostatného českého státu',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',
         'type': 'NF'
     },
     {
         'date': '2017-01-01',
-        'description': 'Den obnovy samostatného českého státu',
+        'description': 'Nový rok',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',

diff --git a/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2018] 1.py b/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2018] 1.py
@@ -1,15 +1,15 @@
 [
     {
         'date': '2018-01-01',
-        'description': 'Nový rok',
+        'description': 'Den obnovy samostatného českého státu',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',
         'type': 'NF'
     },
     {
         'date': '2018-01-01',
-        'description': 'Den obnovy samostatného českého státu',
+        'description': 'Nový rok',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',

diff --git a/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2019] 1.py b/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2019] 1.py
@@ -1,15 +1,15 @@
 [
     {
         'date': '2019-01-01',
-        'description': 'Nový rok',
+        'description': 'Den obnovy samostatného českého státu',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',
         'type': 'NF'
     },
     {
         'date': '2019-01-01',
-        'description': 'Den obnovy samostatného českého státu',
+        'description': 'Nový rok',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',

diff --git a/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2020] 1.py b/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2020] 1.py
@@ -1,15 +1,15 @@
 [
     {
         'date': '2020-01-01',
-        'description': 'Nový rok',
+        'description': 'Den obnovy samostatného českého státu',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',
         'type': 'NF'
     },
     {
         'date': '2020-01-01',
-        'description': 'Den obnovy samostatného českého státu',
+        'description': 'Nový rok',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',

diff --git a/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2021] 1.py b/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2021] 1.py
@@ -1,15 +1,15 @@
 [
     {
         'date': '2021-01-01',
-        'description': 'Nový rok',
+        'description': 'Den obnovy samostatného českého státu',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',
         'type': 'NF'
     },
     {
         'date': '2021-01-01',
-        'description': 'Den obnovy samostatného českého státu',
+        'description': 'Nový rok',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',

diff --git a/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2022] 1.py b/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2022] 1.py
@@ -1,15 +1,15 @@
 [
     {
         'date': '2022-01-01',
-        'description': 'Nový rok',
+        'description': 'Den obnovy samostatného českého státu',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',
         'type': 'NF'
     },
     {
         'date': '2022-01-01',
-        'description': 'Den obnovy samostatného českého státu',
+        'description': 'Nový rok',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',

diff --git a/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2023] 1.py b/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2023] 1.py
@@ -1,15 +1,15 @@
 [
     {
         'date': '2023-01-01',
-        'description': 'Nový rok',
+        'description': 'Den obnovy samostatného českého státu',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',
         'type': 'NF'
     },
     {
         'date': '2023-01-01',
-        'description': 'Den obnovy samostatného českého státu',
+        'description': 'Nový rok',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',

diff --git a/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2024] 1.py b/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[cs-CZ-2024] 1.py
@@ -1,15 +1,15 @@
 [
     {
         'date': '2024-01-01',
-        'description': 'Nový rok',
+        'description': 'Den obnovy samostatného českého státu',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',
         'type': 'NF'
     },
     {
         'date': '2024-01-01',
-        'description': 'Den obnovy samostatného českého státu',
+        'description': 'Nový rok',
         'locale': 'cs-CZ',
         'notes': '',
         'region': '',

diff --git a/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[da-DK-2017] 1.py b/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[da-DK-2017] 1.py
@@ -65,19 +65,19 @@
     },
     {
         'date': '2017-06-05',
-        'description': 'Grundlovsdag',
+        'description': 'Anden pinsedag',
         'locale': 'da-DK',
         'notes': '',
         'region': '',
-        'type': 'NF'
+        'type': 'NRV'
     },
     {
         'date': '2017-06-05',
-        'description': 'Anden pinsedag',
+        'description': 'Grundlovsdag',
         'locale': 'da-DK',
         'notes': '',
         'region': '',
-        'type': 'NRV'
+        'type': 'NF'
     },
     {
         'date': '2017-12-25',

diff --git a/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[de-AT-2011] 1.py b/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[de-AT-2011] 1.py
@@ -172,15 +172,15 @@
         'description': 'Leopold',
         'locale': 'de-AT',
         'notes': '',
-        'region': '9',
+        'region': '3',
         'type': 'F'
     },
     {
         'date': '2011-11-15',
         'description': 'Leopold',
         'locale': 'de-AT',
         'notes': '',
-        'region': '3',
+        'region': '9',
         'type': 'F'
     },
     {

diff --git a/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[de-AT-2012] 1.py b/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[de-AT-2012] 1.py
@@ -172,15 +172,15 @@
         'description': 'Leopold',
         'locale': 'de-AT',
         'notes': '',
-        'region': '9',
+        'region': '3',
         'type': 'F'
     },
     {
         'date': '2012-11-15',
         'description': 'Leopold',
         'locale': 'de-AT',
         'notes': '',
-        'region': '3',
+        'region': '9',
         'type': 'F'
     },
     {

diff --git a/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[de-AT-2013] 1.py b/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[de-AT-2013] 1.py
@@ -172,15 +172,15 @@
         'description': 'Leopold',
         'locale': 'de-AT',
         'notes': '',
-        'region': '9',
+        'region': '3',
         'type': 'F'
     },
     {
         'date': '2013-11-15',
         'description': 'Leopold',
         'locale': 'de-AT',
         'notes': '',
-        'region': '3',
+        'region': '9',
         'type': 'F'
     },
     {

diff --git a/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[de-AT-2014] 1.py b/...s/snap_test_holidata/test_holidata_produces_holidays_for_locale_and_year[de-AT-2014] 1.py
@@ -172,15 +172,15 @@
         'description': 'Leopold',
         'locale': 'de-AT',
         'notes': '',
-        'region': '9',
+        'region': '3',
         'type': 'F'
     },
     {
         'date': '2014-11-15',
         'description': 'Leopold',
         'locale': 'de-AT',
         'notes': '',
-        'region': '3',
+        'region': '9',
         'type': 'F'
     },
     {