[#12048] Migrate UpdateStudentAction

src/it/java/teammates/it/storage/sqlsearch/StudentSearchIT.java

@@ -43,6 +43,7 @@ public void allTests() throws Exception {
         Student stu1InCourse1 = typicalBundle.students.get("student1InCourse1");
         Student stu2InCourse1 = typicalBundle.students.get("student2InCourse1");
         Student stu3InCourse1 = typicalBundle.students.get("student3InCourse1");
+        Student stu4InCourse1 = typicalBundle.students.get("student4InCourse1");
         Student stu1InCourse2 = typicalBundle.students.get("student1InCourse2");
         Student unregisteredStuInCourse1 = typicalBundle.students.get("unregisteredStudentInCourse1");
         Student stu1InCourse3 = typicalBundle.students.get("student1InCourse3");
@@ -80,12 +81,12 @@ public void allTests() throws Exception {
         ______TS("success: search for students in whole system; students should be searchable by course id");
 
         results = usersDb.searchStudentsInWholeSystem("\"course-1\"");
-        verifySearchResults(results, stu1InCourse1, stu2InCourse1, stu3InCourse1, unregisteredStuInCourse1);
+        verifySearchResults(results, stu1InCourse1, stu2InCourse1, stu3InCourse1, stu4InCourse1, unregisteredStuInCourse1);
 
         ______TS("success: search for students in whole system; students should be searchable by course name");
 
         results = usersDb.searchStudentsInWholeSystem("\"Typical Course 1\"");
-        verifySearchResults(results, stu1InCourse1, stu2InCourse1, stu3InCourse1, unregisteredStuInCourse1);
+        verifySearchResults(results, stu1InCourse1, stu2InCourse1, stu3InCourse1, stu4InCourse1, unregisteredStuInCourse1);
 
         ______TS("success: search for students in whole system; students should be searchable by their name");
 

src/it/java/teammates/it/ui/webapi/DeleteStudentsActionIT.java

@@ -48,7 +48,7 @@ protected void testExecute() throws Exception {
 
         List<Student> studentsToDelete = logic.getStudentsForCourse(courseId);
 
-        assertEquals(4, studentsToDelete.size());
+        assertEquals(5, studentsToDelete.size());
 
         String[] params = new String[] {
                 Const.ParamsNames.COURSE_ID, courseId,

src/it/java/teammates/it/ui/webapi/EnrollStudentsActionIT.java

@@ -73,7 +73,7 @@ public void testExecute() throws Exception {
         };
 
         List<Student> students = new ArrayList<>(logic.getStudentsForCourse(courseId));
-        assertEquals(4, students.size());
+        assertEquals(5, students.size());
 
         ______TS("Typical Success Case For Enrolling a Student");
 
@@ -83,7 +83,7 @@ public void testExecute() throws Exception {
         EnrollStudentsData data = (EnrollStudentsData) res.getOutput();
         assertEquals(1, data.getStudentsData().getStudents().size());
         List<Student> studentsInCourse = logic.getStudentsForCourse(courseId);
-        assertEquals(5, studentsInCourse.size());
+        assertEquals(6, studentsInCourse.size());
 
         ______TS("Fail to enroll due to duplicate team name across sections");
 
@@ -111,7 +111,7 @@ public void testExecute() throws Exception {
         data = (EnrollStudentsData) res.getOutput();
         assertEquals(1, data.getStudentsData().getStudents().size());
         studentsInCourse = logic.getStudentsForCourse(courseId);
-        assertEquals(5, studentsInCourse.size());
+        assertEquals(6, studentsInCourse.size());
 
         // Verify that changes have cascaded to feedback responses
         String giverEmail = "[email protected]";

src/it/java/teammates/it/ui/webapi/FeedbackSessionClosingRemindersActionIT.java

@@ -120,10 +120,10 @@ private void textExecute_typicalSuccess1() {
         assertTrue(session.isClosingSoonEmailSent());
         assertTrue(session.getDeadlineExtensions().stream().allMatch(de -> !de.isClosingSoonEmailSent()));
 
-        // 6 email tasks queued:
-        // 1 co-owner, 4 students and 3 instructors,
+        // 7 email tasks queued:
+        // 1 co-owner, 5 students and 3 instructors,
         // but 1 student and 1 instructor have deadline extensions (should not receive email)
-        verifySpecifiedTasksAdded(Const.TaskQueue.SEND_EMAIL_QUEUE_NAME, 6);
+        verifySpecifiedTasksAdded(Const.TaskQueue.SEND_EMAIL_QUEUE_NAME, 7);
     }
 
     private void textExecute_typicalSuccess2() {
@@ -149,11 +149,11 @@ private void textExecute_typicalSuccess2() {
         assertTrue(session.isClosingSoonEmailSent());
         assertTrue(de.isClosingSoonEmailSent());
 
-        // 7 email tasks queued:
-        // - 6 emails: 1 co-owner, 4 students and 3 instructors,
+        // 8 email tasks queued:
+        // - 7 emails: 1 co-owner, 5 students and 3 instructors,
         //             but 1 student and 1 instructor have deadline extensions (should not receive email)
         // - 1 email:  1 student deadline extension
-        verifySpecifiedTasksAdded(Const.TaskQueue.SEND_EMAIL_QUEUE_NAME, 7);
+        verifySpecifiedTasksAdded(Const.TaskQueue.SEND_EMAIL_QUEUE_NAME, 8);
     }
 
     private void textExecute_typicalSuccess3() {

src/it/java/teammates/it/ui/webapi/GetStudentsActionIT.java

@@ -57,7 +57,7 @@ protected void testExecute() throws Exception {
         StudentsData response = (StudentsData) jsonResult.getOutput();
         List<StudentData> students = response.getStudents();
 
-        assertEquals(4, students.size());
+        assertEquals(5, students.size());
 
         StudentData firstStudentInStudents = students.get(0);
 

src/it/java/teammates/it/ui/webapi/SearchStudentsActionIT.java

@@ -96,7 +96,7 @@ public void execute_adminSearchName_success() {
         JsonResult result = getJsonResult(a);
         StudentsData response = (StudentsData) result.getOutput();
 
-        assertEquals(10, response.getStudents().size());
+        assertEquals(11, response.getStudents().size());
     }
 
     @Test
@@ -114,7 +114,7 @@ public void execute_adminSearchCourseId_success() {
         JsonResult result = getJsonResult(a);
         StudentsData response = (StudentsData) result.getOutput();
 
-        assertEquals(10, response.getStudents().size());
+        assertEquals(11, response.getStudents().size());
     }
 
     @Test