History: add a creator field

* automatically add creator to new created item
* associated migrations
* associated tests
* tests

1 files changed, 24 insertions, 6 deletions

diff --git a/archaeological_files/tests.py b/archaeological_files/tests.py
index 58291bc04..b43967401 100644
--- a/archaeological_files/tests.py
+++ b/archaeological_files/tests.py
@@ -37,7 +37,7 @@ class FileTest(TestCase):
     def setUp(self):
         self.extra_models, self.model_list = {}, []
         self.user, created = User.objects.get_or_create(username='username')
-        o_user, created = User.objects.get_or_create(username='ousername')
+        self.o_user, created = User.objects.get_or_create(username='ousername')
         person_type = PersonType(label=u'Test person type',
                                  txt_idx='test_person', available=True)
         person_type.save()
@@ -45,7 +45,7 @@ class FileTest(TestCase):
         self.model_list.append(person_type)
 
         person = models.Person(title='Mr', surname='Surname', name='Name',
-                               history_modifier=o_user)
+                               history_modifier=self.o_user)
         person.save()
         self.extra_models['person'] = person
         self.model_list.append(person)
@@ -58,14 +58,15 @@ class FileTest(TestCase):
 
         dct = {'year':2010, 'numeric_reference':1000, 'file_type':file_type,
                'internal_reference':u'UNIT_testÉ ?', 'in_charge':person,
-               'history_modifier':o_user, 'total_surface':10000}
+               'history_modifier':self.o_user, 'total_surface':10000}
         self.item = self.model(**dct)
         self.item.save()
 
     def tearDown(self):
-        self.item.delete()
-        for item in reversed(self.model_list):
-            item.delete()
+        #self.item.delete()
+        #for item in reversed(self.model_list):
+        #    item.delete()
+        pass
 
     def testAddAndGetHistorized(self):
         """
@@ -80,6 +81,23 @@ class FileTest(TestCase):
         self.failUnlessEqual(self.item.history.count(), nb_hist+1)
         self.failUnlessEqual(self.item.history.all()[1].internal_reference,
                              base_label)
+        self.item.internal_reference = u"Unité_Testée"
+        self.item.history_modifier = self.user
+        self.item.skip_history_when_saving = True
+        self.item.save()
+        self.item.skip_history_when_saving = False
+        self.failUnlessEqual(self.item.history.count(), nb_hist+1)
+
+    def testCreatorHistorized(self):
+        """
+        Test creator association
+        """
+        self.failUnlessEqual(self.item.history_creator, self.o_user)
+        altuser, created = User.objects.get_or_create(username='altusername')
+        self.item.internal_reference = u"Unité_Test"
+        self.item.history_modifier = altuser
+        self.item.save()
+        self.failUnlessEqual(self.item.history_creator, self.o_user)
 
     def testIntelligentHistorisation(self):
         """