From 957b85dead87ee5e1a90092d76b103bf7e4e42b1 Mon Sep 17 00:00:00 2001
From: David Miller <david@deadpansincerity.com>
Date: Fri, 21 Apr 2017 14:24:50 +0100
Subject: [PATCH] Test the ordering of bulk update explicitly in it's own test

---
 opal/tests/test_models.py | 20 +++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/opal/tests/test_models.py b/opal/tests/test_models.py
index 10eb5a783..6bf6bee3a 100644
--- a/opal/tests/test_models.py
+++ b/opal/tests/test_models.py
@@ -74,9 +74,27 @@ def test_bulk_update_patient_subrecords(self):
 
         colours = patient.patientcolour_set.all()
         self.assertEqual(len(colours), 2)
+        self.assertTrue(patient.episode_set.exists())
+
+    def test_bulk_update_patient_subrecords_respects_order(self):
+        patient = models.Patient()
+
+        d = {
+            "demographics": [{
+                "first_name": "Samantha",
+                "surname": "Sun",
+                "hospital_number": "123312"
+            }],
+            "patient_colour": [
+                {"name": "green"},
+                {"name": "purple"},
+            ]
+        }
+        patient.bulk_update(d, self.user)
+        colours = patient.patientcolour_set.all()
         self.assertEqual(colours[0].name, "green")
         self.assertEqual(colours[1].name, "purple")
-        self.assertTrue(patient.episode_set.exists())
+
 
     def test_bulk_update_with_existing_patient_episode(self):
         original_patient = models.Patient()