Changeset 6649

Timestamp:

29 Aug 2011, 08:07:36 (13 years ago)

Author:

Henrik Bettermann

Message:

Add manage page for study course objects.

Further increase test coverage.

Location:

main/waeup.sirp/trunk/src/waeup/sirp/students

Files:

• browser.py (modified) (3 diffs)
• tests/test_browser.py (modified) (3 diffs)

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

@@ -232 +232 @@
         student.student_id = student_id
         self.applyData(student, **data)
-        #import pdb; pdb.set_trace()
         try:
             self.context.addStudent(student)
@@ -337 +336 @@
     grok.context(IStudent)
     grok.name('edit_clearance')
-    grok.require('waeup.viewStudents')
+    grok.require('waeup.manageStudents')
     form_fields = grok.AutoFields(IStudentClearance)
     label = 'Edit clearance data'
@@ -391 +390 @@
         return '%s: Study Course' % self.context.__parent__.name
 
+class StudyCourseManageActionButton(ManageActionButton):
+    grok.order(1)
+    grok.context(IStudentStudyCourse)
+    grok.view(StudyCourseDisplayFormPage)
+    grok.require('waeup.manageStudents')
+    text = 'Edit'
+    target = 'edit'
+
+class StudyCourseManageFormPage(WAeUPEditFormPage):
+    """ Page to edit the student study course data
+    """
+    grok.context(IStudentStudyCourse)
+    grok.name('edit')
+    grok.require('waeup.manageStudents')
+    form_fields = grok.AutoFields(IStudentStudyCourse)
+    label = 'Edit clearance data'
+    title = 'Study Course'
+    label = 'Edit study course'
+    pnav = 4
+
 class PaymentsDisplayFormPage(WAeUPDisplayFormPage):
     """ Page to display the student payments

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

@@ -150 +150 @@
         self.browser.getControl("Create student record").click()
         self.assertTrue('Student record created' in self.browser.contents)
+        self.browser.open(self.manage_container_path)
+        self.browser.getLink("Add student").click()
+        self.browser.getControl(name="form.student_id").value = 'A123456'
+        self.browser.getControl(name="form.name").value = 'Bob Brother'
+        self.browser.getControl("Create student record").click()
+        self.assertTrue('The student id chosen already exists' in self.browser.contents)
 
         self.browser.open(self.container_path)
@@ -200 +206 @@
         self.assertEqual(self.browser.headers['Status'], '200 Ok')
         self.assertEqual(self.browser.url, self.manage_student_path)
-        # Managers can fire transitions
+        # Managers can edit base data and fire transitions
         self.browser.getControl(name="transition").value = ['admit']
+        self.browser.getControl(name="form.name").value = 'John Tester'
         self.browser.getControl("Save").click()
         self.assertTrue('Form has been saved' in self.browser.contents)
@@ -225 +232 @@
         self.assertEqual(self.browser.headers['Status'], '200 Ok')
         self.assertEqual(self.browser.url, self.studycourse_student_path)
+        self.browser.getLink("Edit").click()
+        self.assertTrue('Edit study course' in self.browser.contents)
 
         self.browser.open(self.student_path)