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

Timestamp:

21 Sep 2013, 08:27:47 (11 years ago)

Author:

Henrik Bettermann

Message:

Disable score editing on department manage page.

Add tests.

Location:

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

Files:

• browser.py (modified) (1 diff)
• reports/tests/test_student_statistics.py (modified) (2 diffs)
• tests/test_browser.py (modified) (1 diff)
• tests/test_move.py (modified) (1 diff)

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

@@ -3071 +3071 @@
         self.current_academic_session = grok.getSite()[
             'configuration'].current_academic_session
+        if self.context.__parent__.__parent__.score_editing_disabled:
+            self.flash(_('Score editing disabled.'))
+            self.redirect(self.url(self.context))
+            return
         if not self.current_academic_session:
             self.flash(_('Current academic session not set.'))

main/waeup.kofa/trunk/src/waeup/kofa/students/reports/tests/test_student_statistics.py

@@ -49 +49 @@
     def test_get_student_stats_session_multiple(self):
         # we can get a table with several students
-        self.create_cert('fac2', 'dept2', 'CERT2')
+        self.create_cert(u'fac2', u'dept2', u'CERT2')
         result1 = get_student_stats(2010, 'Undergraduate Full-Time')
         result2 = get_student_stats(2009, 'Undergraduate Full-Time')
@@ -69 +69 @@
 
     def test_create_pdf(self):
-        self.create_cert('FAC2', 'dept2', 'CERT2')
+        self.create_cert(u'FAC2', u'dept2', u'CERT2')
         report = StudentStatisticsReport(2010, 'Undergraduate Full-Time')
         result = report.create_pdf()

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

@@ -1683 +1683 @@
             self.student['studycourse']['100']['COURSE1'].course,
             self.course)
-        # Lecturer can neither access ths student ...
+        # Lecturer can neither access the student ...
         self.assertRaises(
             Unauthorized, self.browser.open, self.student_path)
-        # ... nor the respective course ticket since a
+        # ... nor the respective course ticket since
         # editing course tickets by lecturers is not feasible.
-        # Course results must be imported.
         self.assertTrue('COURSE1' in self.student['studycourse']['100'].keys())
         course_ticket_path = self.student_path + '/studycourse/100/COURSE1'
         self.assertRaises(
             Unauthorized, self.browser.open, course_ticket_path)
+        # Course results can be batch edited via the edit_courses view
+        self.app['faculties']['fac1']['dep1'].score_editing_disabled = True
+        self.browser.open("http://localhost/app/faculties/fac1/dep1/courses/COURSE1")
+        self.browser.getLink("Update scores").click()
+        self.assertTrue('Score editing disabled' in self.browser.contents)
+        self.app['faculties']['fac1']['dep1'].score_editing_disabled = False
+        self.browser.getLink("Update scores").click()
+        self.assertTrue('Current academic session not set' in self.browser.contents)
+        self.app['configuration'].current_academic_session = 2004
+        self.browser.getLink("Update scores").click()
+        self.assertFalse(
+            '<input type="text" name="scores" class="span1" />'
+            in self.browser.contents)
+        IWorkflowState(self.student).setState('courses validated')
+        # Student must be in state 'courses validated'
+        self.browser.open(
+            "http://localhost/app/faculties/fac1/dep1/courses/COURSE1/edit_scores")
+        self.assertTrue(
+            '<input type="text" name="scores" class="span1" />'
+            in self.browser.contents)
+        self.browser.getControl(name="scores", index=0).value = '55'
+        self.browser.getControl("Update scores").click()
+        # New score has been set
+        self.assertEqual(
+            self.student['studycourse']['100']['COURSE1'].score, 55)
+        # Score editing has been logged
+        logfile = os.path.join(
+            self.app['datacenter'].storage, 'logs', 'students.log')
+        logcontent = open(logfile).read()
+        self.assertTrue('mrslecturer - students.browser.EditScoresPage - '
+                        'K1000000 100/COURSE1 score updated (55)' in logcontent)
 
     def test_change_current_mode(self):

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

@@ -31 +31 @@
             self.app['faculties']['fac1']['dep1'].certificates['CERT1'],
             self.certificate)
-        self.app['faculties']['fac1']['dep2'] = Department(code='dep2')
+        self.app['faculties']['fac1']['dep2'] = Department(code=u'dep2')
         self.certificate.moveCertificate('fac1', 'dep2')
         self.assertEqual(