Testing updating of record

nuntium/models.py

@@ -91,9 +91,15 @@ def do_something_with_a_vanished_popit_api_instance(self, popit_api_instance):
 
     def _load_persons_from_a_popit_api(self, popit_api_instance):
         success_relating_people, error = self.relate_with_persons_from_popit_api_instance(popit_api_instance)
-        if success_relating_people:
-            record, created = WriteitInstancePopitInstanceRecord\
-                .objects.get_or_create(
+        record = WriteitInstancePopitInstanceRecord.objects.filter(
+            writeitinstance=self,
+            popitapiinstance=popit_api_instance
+        )
+        record.update = datetime.datetime.today()
+        record.save()
+        if success_relating_people and not record:
+            record = WriteitInstancePopitInstanceRecord\
+                .objects.create(
                     writeitinstance=self,
                     popitapiinstance=popit_api_instance)
 

nuntium/tests/popit_writeit_relation_tests.py

@@ -7,6 +7,7 @@
 from django.contrib.auth.models import User
 from django.conf import settings
 from nuntium.popit_api_instance import PopitApiInstance
+from datetime import timedelta, datetime
 
 
 class PopitWriteitRelationRecord(TestCase):
@@ -150,3 +151,31 @@ def test_it_should_be_able_to_update_twice(self):
             )
 
         self.assertNotEqual(record.created, record.updated)
+
+    def test_it_should_update_the_date_every_time_it_is_updated(self):
+        '''As described in http://github.com/ciudadanointeligente/write-it/issues/412 the updated date is not updated'''
+
+        popit_load_data()
+        # loading data into the popit-api
+        writeitinstance = WriteItInstance.objects.create(
+            name='instance 1',
+            slug='instance-1',
+            owner=self.owner,
+            )
+
+        writeitinstance.load_persons_from_a_popit_api(settings.TEST_POPIT_API_URL)
+        popit_instance = ApiInstance.objects.get(url=settings.TEST_POPIT_API_URL)
+        record = WriteitInstancePopitInstanceRecord.objects.get(
+            writeitinstance=writeitinstance,
+            popitapiinstance=popit_instance,
+            )
+        created_and_updated = datetime.today() - timedelta(days=2)
+
+        record.updated = created_and_updated
+        record.created = created_and_updated
+        record.save()
+
+        writeitinstance.load_persons_from_a_popit_api(settings.TEST_POPIT_API_URL)
+        record_again = WriteitInstancePopitInstanceRecord.objects.get(id=record.id)
+        self.assertNotEqual(record_again.updated, created_and_updated)
+        self.assertEquals(record_again.created, created_and_updated)