Update tests for GetStudentsActionTest

TEAMMATES · Mar 12, 2025 · 9782fe3 · 9782fe3
1 parent fa2ffc5
commit 9782fe3
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/test/java/teammates/sqlui/webapi/GetStudentsActionTest.java b/src/test/java/teammates/sqlui/webapi/GetStudentsActionTest.java
@@ -129,7 +129,7 @@ void testExecute_instructorWithPermission_success() {
     }
 
     @Test
-    void testExecute_instructorWithoutPermission_emptyList() {
+    void testExecute_instructorWithoutPermissionWithInvalidTeamParams_emptyList() {
         loginAsInstructor(stubInstructorWithoutPrivileges.getGoogleId());
         when(mockLogic.getInstructorByGoogleId(stubCourse.getId(), stubInstructorWithoutPrivileges.getGoogleId()))
                 .thenReturn(stubInstructorWithoutPrivileges);
@@ -198,7 +198,7 @@ void testExecute_instructorWithDifferentSectionPrivilegesAsStudents_emptyList()
                         false, Const.DEFAULT_DISPLAY_NAME_FOR_INSTRUCTOR, customRole, wrongPrivileges);
         loginAsInstructor(stubInstructorWithOnlyViewPrivilegesForDifferentSection.getGoogleId());
         when(mockLogic.getInstructorByGoogleId(stubCourse.getId(),
-                stubInstructorWithOnlyViewSectionPrivileges.getGoogleId()))
+                stubInstructorWithOnlyViewPrivilegesForDifferentSection.getGoogleId()))
                 .thenReturn(stubInstructorWithOnlyViewPrivilegesForDifferentSection);
         when(mockLogic.getStudentsForCourse(stubCourse.getId())).thenReturn(stubStudentListTwo);
 
@@ -512,7 +512,7 @@ void testSpecificAccessControl_invalidParams_throwsInvalidHttpParameterException
     }
 
     @Test
-    void testSpecificAccessControl_anotherCourseFromInstructor_cannotAccess() {
+    void testSpecificAccessControl_wrongCourse_cannotAccess() {
         loginAsInstructor(stubInstructorWithAllPrivileges.getGoogleId());
         when(mockLogic.getInstructorByGoogleId("another-course-id", stubInstructorWithAllPrivileges.getGoogleId()))
                 .thenReturn(null);