Changeset 12766 for main/waeup.ikoba

Timestamp:

15 Mar 2015, 09:37:37 (10 years ago)

Author:

Henrik Bettermann

Message:

Add browser tests and set up logger properly.

Location:

main/waeup.ikoba/trunk/src/waeup/ikoba

Files:

• documents/tests/test_browser.py (modified) (1 diff)
• payments/browser.py (modified) (3 diffs, 1 prop)
• payments/container.py (modified) (1 diff)
• payments/payment.py (modified) (2 diffs)
• payments/tests/test_browser.py (added)

main/waeup.ikoba/trunk/src/waeup/ikoba/documents/tests/test_browser.py

@@ -17 +17 @@
 ##
 """
-Test the customer-related UI components.
+Test the document-related UI components.
 """
 import shutil

main/waeup.ikoba/trunk/src/waeup/ikoba/payments/browser.py

@@ -1 @@
-## $Id: browser.py 12760 2015-03-14 05:46:24Z henrik $
+## $Id$
 ##
 ## Copyright (C) 2015 Uli Fouquet & Henrik Bettermann
@@ -147 +147 @@
     pnav = 5
     label = _('Manage payment section')
-    search_button = _('Find payments(s)')
+    search_button = _('Find payment(s)')
     remove_button = _('Remove selected')
 
@@ -192 +192 @@
             self.flash(_('Successfully removed: ${a}',
                 mapping={'a': ','.join(deleted)}))
+            ob_class = self.__implemented__.__name__.replace('waeup.ikoba.','')
+            self.context.logger.info(
+                '%s - removed: %s' % (ob_class, ','.join(deleted)))
         return

main/waeup.ikoba/trunk/src/waeup/ikoba/payments/container.py

@@ -23 +23 @@
 from waeup.ikoba.payments.interfaces import IPaymentsContainer
 from waeup.ikoba.utils.helpers import attrs_to_fields
+from waeup.ikoba.utils.logger import Logger
 
 
-class PaymentsContainer(grok.Container):
+class PaymentsContainer(grok.Container, Logger):
     """This is a container for all kind of payments.
     """
     grok.implements(IPaymentsContainer)
     grok.provides(IPaymentsContainer)
+
+    logger_name = 'waeup.ikoba.${sitename}.payments'
+    logger_filename = 'payments.log'
 
     def __init__(self):

main/waeup.ikoba/trunk/src/waeup/ikoba/payments/payment.py

@@ -28 +28 @@
 from waeup.ikoba.interfaces import MessageFactory as _
 from waeup.ikoba.utils.helpers import attrs_to_fields
-from waeup.ikoba.utils.logger import Logger
 from waeup.ikoba.payments.interfaces import (
     IPayment, STATE_UNPAID, STATE_FAILED, STATE_PAID,
@@ -167 +166 @@
 
 @attrs_to_fields
-class Payment(grok.Model, Logger):
+class Payment(grok.Model):
     """This is a payment.
     """
     grok.implements(IPayment)
     grok.provides(IPayment)
-
-    logger_name = 'waeup.ikoba.${sitename}.payments'
-    logger_filename = 'payments.log'
-    logger_format_str = '"%(asctime)s","%(user)s",%(message)s'
 
     def __init__(self, payer, payable, payee=None):