[CalendarServer-changes] [11683] CalendarServer/trunk/twext/python/test/test_log.py

[source_changes at macosforge.org]

Revision: 11683
          http://trac.calendarserver.org//changeset/11683
Author:   wsanchez at apple.com
Date:     2013-09-12 17:07:32 -0700 (Thu, 12 Sep 2013)
Log Message:
-----------
Add test_unfilteredObserver()

Modified Paths:
--------------
    CalendarServer/trunk/twext/python/test/test_log.py

Modified: CalendarServer/trunk/twext/python/test/test_log.py
===================================================================
--- CalendarServer/trunk/twext/python/test/test_log.py	2013-09-12 23:34:19 UTC (rev 11682)
+++ CalendarServer/trunk/twext/python/test/test_log.py	2013-09-13 00:07:32 UTC (rev 11683)
@@ -591,7 +591,7 @@
             self.fail("Observer raised an exception and the exception was not logged.")
 
 
-    def test_observerRaiseAndLoggerHatesMe(self):
+    def test_observerRaisesAndLoggerHatesMe(self):
         nonTestEvents = []
         Logger.publisher.addObserver(lambda e: nonTestEvents.append(e))
 
@@ -650,12 +650,25 @@
         self.assertNotIn(event_none, events)
 
 
+    def test_unfilteredObserver(self):
+        namespace = __name__
 
-#    def test_addObserver_unfiltered(self):
-#        pass
+        event_debug = dict(log_namespace=namespace, log_level=LogLevel.debug, log_format="")
+        event_error = dict(log_namespace=namespace, log_level=LogLevel.error, log_format="")
+        events = []
 
+        observer = lambda e: events.append(e)
 
+        publisher = DefaultLogPublisher()
 
+        publisher.addObserver(observer, filtered=False)
+        publisher(event_debug)
+        publisher(event_error)
+        self.assertIn(event_debug, events)
+        self.assertIn(event_error, events)
+
+
+
 class FilteringLogObserverTests(SetUpTearDown, unittest.TestCase):
     """
     Tests for L{FilteringLogObserver}.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/calendarserver-changes/attachments/20130912/0d86bd38/attachment.html>