Changeset 17550

Timestamp:

18 Aug 2023, 04:24:57 (13 months ago)

Author:

Henrik Bettermann

Message:

Add logging messages before after creating students from applicants.

Location:

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

Files:

• browser.py (modified) (4 diffs)
• tests/test_applicantcopier.py (modified) (2 diffs)

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

@@ -755 +755 @@
 
     def update(self):
+        ob_class = self.__implemented__.__name__.replace('waeup.kofa.','')
+        applicants_root = grok.getSite()['applicants']
         grok.getSite()['configuration'].maintmode_enabled_by = u'admin'
+        applicants_root.logger.info('%s - started in %s' % (
+            ob_class, self.context.__name__))
         transaction.commit()
         # Wait 10 seconds for all transactions to be finished.
@@ -778 +782 @@
                 failed.append(
                     (result.applicant_id, self.url(result, 'manage'), msg))
-                ob_class = self.__implemented__.__name__.replace(
-                    'waeup.kofa.','')
         grok.getSite()['configuration'].maintmode_enabled_by = None
         self.successful = ', '.join(created)
         self.failed = failed
+        applicants_root.logger.info('%s - created in %s: %s' % (
+            ob_class, self.context.__name__, self.successful))
         return
 
@@ -1403 +1407 @@
         site = grok.getSite()
         kofa_utils = getUtility(IKofaUtils)
-        ob_class = self.__implemented__.__name__.replace('waeup.kofa.','')
         failed = ''
         emails_sent = 0
@@ -1890 +1893 @@
         site = grok.getSite()
         kofa_utils = getUtility(IKofaUtils)
-        ob_class = self.__implemented__.__name__.replace('waeup.kofa.','')
         failed = ''
         emails_sent = 0

main/waeup.kofa/trunk/src/waeup/kofa/applicants/tests/test_applicantcopier.py

@@ -304 +304 @@
         self.assertFalse('Failed records' in self.browser.contents)
         self.assertTrue('Successfully created records' in self.browser.contents)
+        logcontent = open(logfile).read()
+        self.assertTrue('applicants.browser.CreateAllStudentsPage - started in app2021'
+        in logcontent)
+        self.assertTrue('%s - Student record created (K1000000)' % self.applicant.applicant_id
+        in logcontent)
+        self.assertTrue('applicants.browser.CreateAllStudentsPage - created in app2021: %s' % self.applicant.applicant_id
+        in logcontent)
 
     def test_hidden_batch_copying_all(self):
@@ -321 +328 @@
         self.browser.open(self.root_path + '/createallstudents')
         self.assertTrue('Successfully created records:' in self.browser.contents)
+        logcontent = open(logfile).read()
+        self.assertTrue('applicants.browser.CreateAllStudentsPage - started in applicants'
+        in logcontent)
+        self.assertTrue('%s - Student record created (K1000000)' % self.applicant.applicant_id
+        in logcontent)
+        self.assertTrue('applicants.browser.CreateAllStudentsPage - created in applicants: %s' % self.applicant.applicant_id
+        in logcontent)
 
     def test_copier_wo_passport(self):