Changes the order of MRNs returned by get_mrn_and_date_from_merge_com…

…ment We now return them with the most recent first
openhealthcare · Dec 7, 2022 · 820d795 · 820d795
1 parent 93227f4
commit 820d795
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/intrahospital_api/test/test_update_demographics.py b/intrahospital_api/test/test_update_demographics.py
@@ -455,18 +455,18 @@ def test_multiple_results(self):
         )
         self.assertEqual(len(result), 2)
         mrn, merge_dt = result[0]
-        self.assertEqual(mrn, "123456")
+        self.assertEqual(mrn, "789")
         self.assertEqual(
             merge_dt,
             timezone.make_aware(
-                datetime.datetime(2016, 12, 5, 15, 49)
+                datetime.datetime(2022, 3, 2, 14, 56)
             )
         )
         mrn, merge_dt = result[1]
-        self.assertEqual(mrn, "789")
+        self.assertEqual(mrn, "123456")
         self.assertEqual(
             merge_dt,
             timezone.make_aware(
-                datetime.datetime(2022, 3, 2, 14, 56)
+                datetime.datetime(2016, 12, 5, 15, 49)
             )
         )
diff --git a/intrahospital_api/update_demographics.py b/intrahospital_api/update_demographics.py
@@ -209,7 +209,7 @@ def get_mrn_and_date_from_merge_comment(merge_comment):
         merge_dt = datetime.datetime.strptime(date_str, "%d %b %Y %I:%M%p")
         result.append((mrn, timezone.make_aware(merge_dt),))
     # return by merged date
-    return sorted(result, key=lambda x: x[1])
+    return sorted(result, key=lambda x: x[1], reverse=True)
 
 def upstream_merged_mrn(mrn):
     """