Changeset 9125 for main/waeup.kofa/trunk/src/waeup/kofa

Timestamp:

30 Aug 2012, 07:26:27 (12 years ago)

Author:

Henrik Bettermann

Message:

Add removeMessage to ObjectHistory? class which removes a single history message.

Location:

main/waeup.kofa/trunk/src/waeup/kofa

Files:

• objecthistory.py (modified) (1 diff)
• tests/test_objecthistory.py (modified) (1 diff)

main/waeup.kofa/trunk/src/waeup/kofa/objecthistory.py

@@ -92 +92 @@
         self._annotations[self.history_key] = new_msgs
         return
+
+    def removeMessage(self, number):
+        """Removes a single history message.
+
+        """
+        msgs = self._getMessages()
+        msgs.pop(number)
+        self._annotations[self.history_key] = msgs
+        return

main/waeup.kofa/trunk/src/waeup/kofa/tests/test_objecthistory.py

@@ -101 +101 @@
         self.assertMatches('<YYYY-MM-DD hh:mm:ss> UTC - blow by Bob', result)
 
+    def test_remove_message(self):
+        principal = Principal('bob')
+        principal.title = 'Bob'
+        newInteraction(Participation(principal)) # set current user
+        hist = IObjectHistory(self.obj)
+        hist.addMessage('blah')
+        hist.addMessage('blow')
+        self.assertEqual(len(hist._getMessages()),2)
+        result = ' '.join(hist.messages)
+        self.assertTrue('blah by Bob' in result)
+        self.assertTrue('blow by Bob' in result)
+        hist.removeMessage(1)
+        self.assertEqual(len(hist._getMessages()),1)
+        result = ' '.join(hist.messages)
+        self.assertFalse('blow by Bob' in result)
+
     def test_messages(self):
         # we get messages as a persistent list of strings