Changeset 16012 for main/waeup.kofa/trunk/src/waeup/kofa/students

Timestamp:

24 Feb 2020, 21:26:35 (5 years ago)

Author:

Henrik Bettermann

Message:

Implement BatchProcessor.checkCreateRequirements and
BatchProcessor.checkUpdateRequirements methods mainly
to protect course result lists of graduated student.

Location:

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

Files:

• batching.py (modified) (4 diffs)
• tests/test_batching.py (modified) (5 diffs)

main/waeup.kofa/trunk/src/waeup/kofa/students/batching.py

@@ -621 +621 @@
         return
 
+    def checkCreateRequirements(self, parent, row, site):
+        """
+        """
+        if parent.student.studycourse_locked:
+            return 'Studycourse is locked.'
+        return None
+
     def checkUpdateRequirements(self, obj, row, site):
         """
@@ -626 +633 @@
         if obj.student.studycourse_locked:
             return 'Studylevel is locked.'
+        return None
+
+    def checkRemoveRequirements(self, obj, row, site):
+        """
+        """
+        if obj.student.studycourse_locked:
+            return 'Studycourse is locked.'
         return None
 
@@ -736 +750 @@
         return
 
+    def checkCreateRequirements(self, parent, row, site):
+        """
+        """
+        if parent.student.studycourse_locked:
+            return 'Studycourse is locked.'
+        return None
+
     def checkUpdateRequirements(self, obj, row, site):
         """
@@ -744 +765 @@
             'unlock_score',None):
             return 'Score attribute is locked.'
+        return None
+
+    def checkRemoveRequirements(self, obj, row, site):
+        """
+        """
+        if obj.student.studycourse_locked:
+            return 'Studycourse is locked.'
         return None
 

main/waeup.kofa/trunk/src/waeup/kofa/students/tests/test_batching.py

@@ -496 +496 @@
         num, num_warns, fin_file, fail_file = self.processor.doImport(
             self.csv_file, STUDENT_HEADER_FIELDS)
-        shutil.rmtree(os.path.dirname(fin_file))
         self.assertRaises(
             FatalCSVError, self.processor.doImport, self.csv_file_update4,
             STUDENT_HEADER_FIELDS_UPDATE4, 'update')
+        shutil.rmtree(os.path.dirname(fin_file))
 
     def test_import_remove(self):
@@ -891 +891 @@
             dict(reg_number='1', level='100'), self.app) is False
         self.assertEqual(num_warns,3)
-
         shutil.rmtree(os.path.dirname(fin_file))
 
@@ -1009 +1008 @@
         num, num_warns, fin_file, fail_file = self.processor.doImport(
             self.csv_file, COURSETICKET_HEADER_FIELDS,'create')
-        shutil.rmtree(os.path.dirname(fin_file))
         num, num_warns, fin_file, fail_file = self.processor.doImport(
             self.csv_file, COURSETICKET_HEADER_FIELDS,'update')
@@ -1065 +1063 @@
             '1,COURSE1,<IGNORE>,300,2008,<IGNORE>,6,<IGNORE>,<IGNORE>,level object: does not exist\r\n'
             '1,COURSE1,<IGNORE>,300,2008X,<IGNORE>,6,<IGNORE>,<IGNORE>,level_session: Invalid value\r\n')
+        shutil.rmtree(os.path.dirname(fin_file))
+
+    def test_import_create_locked(self):
+        # In state 'transcript validated' course tickets can't be created
+        student = self.app['students']['X666666']
+        IWorkflowState(student).setState('transcript validated')
+        num, num_warns, fin_file, fail_file = self.processor.doImport(
+            self.csv_file, COURSETICKET_HEADER_FIELDS,'create')
+        fail_file = open(fail_file).read()
+        self.assertEqual(num_warns,7)
+        self.assertEqual(fail_file,
+            'reg_number,code,mandatory,level,level_session,ticket_session,score,unlock_score,matric_number,--ERRORS--\r\n'
+            '1,COURSE1,True,100,<IGNORE>,<IGNORE>,1,<IGNORE>,<IGNORE>,Studycourse is locked.\r\n'
+            '1,COURSE1,True,200,2008,<IGNORE>,1,<IGNORE>,<IGNORE>,Studycourse is locked.\r\n'
+            '1,COURSE1,<IGNORE>,nonsense,<IGNORE>,<IGNORE>,5,<IGNORE>,<IGNORE>,Not all parents do exist yet.\r\n'
+            '1,NONSENSE,<IGNORE>,100,<IGNORE>,<IGNORE>,5,<IGNORE>,<IGNORE>,code: non-existent\r\n'
+            '1,COURSE1,<IGNORE>,200,2004,<IGNORE>,6,<IGNORE>,<IGNORE>,level_session: does not match 2008\r\n'
+            '1,COURSE1,<IGNORE>,300,2008,<IGNORE>,6,<IGNORE>,<IGNORE>,level object: does not exist\r\n'
+            '1,COURSE1,<IGNORE>,300,2008X,<IGNORE>,6,<IGNORE>,<IGNORE>,level_session: Invalid value\r\n')
+        self.assertFalse(self.processor.entryExists(
+            dict(reg_number='1', level='100', code='COURSE1'),
+            self.app))
+        shutil.rmtree(os.path.dirname(fin_file))
+
+    def test_import_remove_locked(self):
+        # We perform the same import twice,
+        # the second time in remove mode. 
+        num, num_warns, fin_file, fail_file = self.processor.doImport(
+            self.csv_file, COURSETICKET_HEADER_FIELDS,'create')
+        shutil.rmtree(os.path.dirname(fin_file))
+        assert self.processor.entryExists(
+            dict(reg_number='1', level='100', code='COURSE1'), self.app) is True
+        # In state 'transcript validated' course tickets can't be removed
+        student = self.app['students']['X666666']
+        IWorkflowState(student).setState('transcript validated')
+        num, num_warns, fin_file, fail_file = self.processor.doImport(
+            self.csv_file, COURSETICKET_HEADER_FIELDS,'remove')
+        self.assertEqual(num_warns,7)
+        assert self.processor.entryExists(
+            dict(reg_number='1', level='100', code='COURSE1'), self.app) is True
+        logcontent = open(self.logfile).read()
+        self.assertFalse(
+            'INFO - system - X666666 - Course ticket in 100 removed: COURSE1'
+            in logcontent)
+        fail_file = open(fail_file).read()
+        self.assertEqual(fail_file,
+            'reg_number,code,matric_number,level,--ERRORS--\r\n'
+            '1,COURSE1,<IGNORE>,100,Studycourse is locked.\r\n'
+            '1,COURSE1,<IGNORE>,200,Studycourse is locked.\r\n'
+            '1,COURSE1,<IGNORE>,nonsense,Cannot remove: no such entry\r\n'
+            '1,NONSENSE,<IGNORE>,100,Cannot remove: no such entry\r\n'
+            '1,COURSE1,<IGNORE>,200,Studycourse is locked.\r\n'
+            '1,COURSE1,<IGNORE>,300,Cannot remove: no such entry\r\n'
+            '1,COURSE1,<IGNORE>,300,Cannot remove: no such entry\r\n'
+            )
         shutil.rmtree(os.path.dirname(fin_file))
 
@@ -1374 +1427 @@
             '... INFO - system - X666666 - Returned...',
             logcontent)
-
         shutil.rmtree(os.path.dirname(fin_file))