Changeset 7522 for main/waeup.sirp/trunk/src/waeup/sirp/students/tests

Timestamp:

27 Jan 2012, 16:33:02 (13 years ago)

Author:

Henrik Bettermann

Message:

Add history messages and log file entries when importing students.

Do not accept empty reg_state fields. If the the reg_state column exists a valid value must be provided.

Location:

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

Files:

• sample_student_data.csv (modified) (1 diff)
• sample_student_data_migration.csv (modified) (1 diff)
• test_batching.py (modified) (3 diffs)

main/waeup.sirp/trunk/src/waeup/sirp/students/tests/sample_student_data.csv

@@ -1 @@
-firstname,lastname,reg_number,date_of_birth,matric_number,email,phone,reg_state
+firstname,lastname,reg_number,date_of_birth,matric_number,email,phone
 Aaren,Pieri,1,1990-01-02,100000,aa@aa.ng,1234
-Aaren,Finau,2,1990-01-03,100001,aa@aa.ng,1234,admitted
-Aaren,Berson,3,1990-01-04,100002,aa@aa.ng,1234,cleared
-John,Wolter,4,1990-01-04,100003,aa@aa.ng,1234,nonsense
+Aaren,Finau,2,1990-01-03,100001,aa@aa.ng,1234
+Aaren,Berson,3,1990-01-04,100002,aa@aa.ng,1234

main/waeup.sirp/trunk/src/waeup/sirp/students/tests/sample_student_data_migration.csv

@@ -3 +3 @@
 B123456,Aaren,Finau,2,1990-01-03,100001,aa@aa.ng,1234,mypw1,cleared
 C123456,Aaren,Berson,3,1990-01-04,100002,aa@aa.ng,1234,mypw1,admitted
-D123456,John,Wolter,4,1990-01-04,100003,aa@aa.ng,1234,mypw1,nonsense
+D123456,John,Wolter,4,1990-01-05,100003,aa@aa.ng,1234,mypw1,nonsense
+E123456,John,Kennedy,5,1990-01-06,100004,aa@aa.ng,1234,

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

@@ -163 +163 @@
         num, num_warns, fin_file, fail_file = self.importer.doImport(
             self.csv_file, STUDENT_HEADER_FIELDS)
-        self.assertEqual(num_warns,1)
+        self.assertEqual(num_warns,0)
         assert len(self.app['students'].keys()) == 4
         shutil.rmtree(os.path.dirname(fin_file))
@@ -197 +197 @@
         num, num_warns, fin_file, fail_file = self.importer.doImport(
             self.csv_file_migration, STUDENT_HEADER_FIELDS_MIGRATION)
-        self.assertEqual(num_warns,1)
+        self.assertEqual(num_warns,2)
         assert len(self.app['students'].keys()) == 4
         self.assertTrue('A123456' in self.app['students'].keys())
@@ -206 +206 @@
         self.assertEqual(self.app['students']['B123456'].date_of_birth,
             datetime.date(1990, 1, 3))
+        history = ' '.join(self.app['students']['A123456'].history.messages)
+        self.assertTrue(
+            "State 'school fee paid' set by system" in history)
         shutil.rmtree(os.path.dirname(fin_file))