Changeset 9183

Timestamp:

14 Sep 2012, 05:43:25 (12 years ago)

Author:

Henrik Bettermann

Message:

Do it right and add test.

Location:

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

Files:

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

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

@@ -137 +137 @@
 
     @property
+    def current_verdict(self):
+        level = getattr(
+            self.get('studycourse', None), 'current_verdict', None)
+        return level
+
+    @property
     def current_mode(self):
         certificate = getattr(
@@ -142 +148 @@
         if certificate is not None:
             return certificate.study_mode
-        return None
-
-    @property
-    def current_verdict(self):
-        certificate = getattr(
-            self.get('studycourse', None), 'certificate', None)
-        if certificate is not None:
-            return certificate.current_verdict
         return None
 

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

@@ -403 +403 @@
 
     layer = FunctionalLayer
+
+    def test_student_properties(self):
+        self.student['studycourse'].current_level = 100
+        self.assertEqual(self.student.current_level, 100)
+        self.student['studycourse'].current_session = 2011
+        self.assertEqual(self.student.current_session, 2011)
+        self.student['studycourse'].current_verdict = 'A'
+        self.assertEqual(self.student.current_verdict, 'A')
+        return
 
     def test_basic_auth(self):