Changeset 11595 for main/waeup.kofa/trunk/src/waeup/kofa/students/tests

Timestamp:

23 Apr 2014, 08:26:03 (11 years ago)

Author:

Henrik Bettermann

Message:

constructMatricNumber must not be private. We will be calling it in custom packages.

File:

• main/waeup.kofa/trunk/src/waeup/kofa/students/tests/test_utils.py (modified) (2 diffs)

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

@@ -43 +43 @@
         site = grok.getSite()
         utils = getUtility(IStudentsUtils)
-        msg, mnumber = utils.setMatricNumber(self.student)
-        self.assertEqual(msg, 'Matriculation number cannot be set.')
-        self.assertEqual(mnumber, None)
-        site['configuration'].next_matric_integer = 1
+        # Matric number can't be set twice.
         msg, mnumber = utils.setMatricNumber(self.student)
         self.assertEqual(msg, 'Matriculation number already set.')
@@ -52 +49 @@
         self.assertEqual(self.student.matric_number, '234')
         self.student.matric_number = None
+        # Matric number can't be set if next_matric_integer is 0.
+        msg, mnumber = utils.setMatricNumber(self.student)
+        self.assertEqual(msg, 'Matriculation number cannot be set.')
+        self.assertEqual(mnumber, None)
+        site['configuration'].next_matric_integer = 1
+        # Now all requirements are met and matric number can be set.
         msg, mnumber = utils.setMatricNumber(self.student)
         self.assertEqual(msg, None)
-        self.assertEqual(mnumber, 1)
+        self.assertEqual(mnumber, '1')
         self.assertEqual(self.student.matric_number, '1')
         self.assertEqual(site['configuration'].next_matric_integer, 2)
-        # Student can be found in catalog.
+        # Student can be found in catalog with new matric number.
         cat = queryUtility(ICatalog, name='students_catalog')
         results = list(cat.searchResults(matric_number=('1', '1')))