Changeset 9326

Timestamp:

10 Oct 2012, 21:14:45 (12 years ago)

Author:

Henrik Bettermann

Message:

Exports must be logged.

Location:

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

Files:

• pages.py (modified) (2 diffs)
• tests/test_browser.py (modified) (1 diff)

main/waeup.kofa/trunk/src/waeup/kofa/browser/pages.py

@@ -1535 +1535 @@
     grok.require('waeup.manageDataCenter')
 
-    def render(self, exporter=None, job_id=None):
+    def render(self, job_id=None):
 
         manager = getUtility(IJobManager)
@@ -1558 +1558 @@
         self.context.delete_export_entry(
             self.context.entry_from_job_id(job_id))
+        ob_class = self.__implemented__.__name__.replace('waeup.kofa.','')
+        self.context.logger.info('%s - exported: %s' % (ob_class, filename))
         return result
 

main/waeup.kofa/trunk/src/waeup/kofa/browser/tests/test_browser.py

@@ -225 +225 @@
         self.assertEqual(result, None)
         self.assertEqual(self.stored_in_datacenter(job_id), False)
-        #logfile = os.path.join(
-        #    self.app['datacenter'].storage, 'logs', 'datacenter.log')
-        #logcontent = open(logfile).read()
-        #self.assertTrue('zope.mgr - browser.pages.ExportCSVView - '
-        #                'exported: faculties' in logcontent)
+        logfile = os.path.join(
+            self.app['datacenter'].storage, 'logs', 'datacenter.log')
+        logcontent = open(logfile).read()
+        self.assertTrue('zope.mgr - browser.pages.ExportCSVView - '
+                        'exported: WAeUP.Kofa_faculties.csv' in logcontent)
         return