[#14097] Create Submission Reminders page

src/e2e/java/teammates/e2e/cases/InstructorFeedbackReportPageE2ETest.java

@@ -19,6 +19,7 @@
 import teammates.common.util.AppUrl;
 import teammates.common.util.Const;
 import teammates.e2e.pageobjects.InstructorFeedbackResultsPage;
+import teammates.e2e.pageobjects.InstructorSessionSendRemindersPage;
 import teammates.storage.entity.Course;
 import teammates.storage.entity.FeedbackQuestion;
 import teammates.storage.entity.FeedbackResponse;
@@ -352,9 +353,9 @@ private void testActions() {
         resultsPage.verifyNoResponsePanelDetails(notResponded);
 
         ______TS("remind all who have not responded to any question");
-        resultsPage.remindAllNonResponders();
+        InstructorSessionSendRemindersPage sendRemindersPage = resultsPage.remindAllNonResponders();
 
-        resultsPage.verifyStatusMessage("Reminder e-mails have been sent out to those students and instructors."
+        sendRemindersPage.verifyStatusMessage("Reminder e-mails have been sent out to those students and instructors."
                 + " Please allow up to 1 hour for all the notification emails to be sent out.");
     }
 

src/e2e/java/teammates/e2e/cases/InstructorFeedbackSessionsPageE2ETest.java

@@ -21,6 +21,7 @@
 import teammates.common.util.Const;
 import teammates.common.util.TimeHelper;
 import teammates.e2e.pageobjects.InstructorFeedbackSessionsPage;
+import teammates.e2e.pageobjects.InstructorSessionSendRemindersPage;
 import teammates.e2e.util.EntityCopyUtil;
 import teammates.storage.entity.Course;
 import teammates.storage.entity.FeedbackSession;
@@ -160,16 +161,19 @@ public void testAll() {
         verifySessionPublishedState(openSession, FeedbackSessionPublishStatus.PUBLISHED);
 
         ______TS("send reminder email to selected student");
-        feedbackSessionsPage.sendReminderEmailToSelectedStudent(openSession, studentToEmail);
+        InstructorSessionSendRemindersPage sendRemindersPage =
+                feedbackSessionsPage.sendReminderEmailToSelectedStudent(openSession, studentToEmail);
 
-        feedbackSessionsPage.verifyStatusMessage("Reminder e-mails have been sent out to those students"
+        sendRemindersPage.verifyStatusMessage("Reminder e-mails have been sent out to those students"
                 + " and instructors. Please allow up to 1 hour for all the notification emails to be sent out.");
+        feedbackSessionsPage = getNewPageInstance(url, InstructorFeedbackSessionsPage.class);
 
         ______TS("send reminder email to all student non-submitters");
-        feedbackSessionsPage.sendReminderEmailToNonSubmitters(openSession);
+        sendRemindersPage = feedbackSessionsPage.sendReminderEmailToNonSubmitters(openSession);
 
-        feedbackSessionsPage.verifyStatusMessage("Reminder e-mails have been sent out to those students"
+        sendRemindersPage.verifyStatusMessage("Reminder e-mails have been sent out to those students"
                 + " and instructors. Please allow up to 1 hour for all the notification emails to be sent out.");
+        feedbackSessionsPage = getNewPageInstance(url, InstructorFeedbackSessionsPage.class);
 
         ______TS("resend results link");
         feedbackSessionsPage.resendResultsLink(openSession, studentToEmail);

src/e2e/java/teammates/e2e/cases/InstructorHomePageE2ETest.java

@@ -20,6 +20,7 @@
 import teammates.common.util.AppUrl;
 import teammates.common.util.Const;
 import teammates.e2e.pageobjects.InstructorHomePage;
+import teammates.e2e.pageobjects.InstructorSessionSendRemindersPage;
 import teammates.e2e.util.EntityCopyUtil;
 import teammates.storage.entity.Course;
 import teammates.storage.entity.FeedbackSession;
@@ -151,18 +152,6 @@ public void testAll() {
         homePage.verifySessionDetails(courseIndex, sessionIndex, feedbackSessionOpen);
         verifySessionPublishedState(feedbackSessionOpen, true);
 
-        ______TS("send reminder email to selected student");
-        homePage.sendReminderEmailToSelectedStudent(courseIndex, sessionIndex, studentToEmail);
-
-        homePage.verifyStatusMessage("Reminder e-mails have been sent out to those students"
-                + " and instructors. Please allow up to 1 hour for all the notification emails to be sent out.");
-
-        ______TS("send reminder email to all student non-submitters");
-        homePage.sendReminderEmailToNonSubmitters(courseIndex, sessionIndex);
-
-        homePage.verifyStatusMessage("Reminder e-mails have been sent out to those students"
-                + " and instructors. Please allow up to 1 hour for all the notification emails to be sent out.");
-
         ______TS("resend results link");
         homePage.resendResultsLink(courseIndex, sessionIndex, studentToEmail);
 
@@ -205,6 +194,24 @@ public void testAll() {
                 + "You can restore it from the Recycle Bin manually.");
         homePage.verifyNumCourses(1);
         assertTrue(BACKDOOR.isCourseInRecycleBin(otherCourse.getId()));
+
+        ______TS("send reminder email to selected student");
+        courseIndex = 0;
+        sessionIndex = 0;
+        homePage.verifySessionDetails(courseIndex, sessionIndex, feedbackSessionOpen);
+        InstructorSessionSendRemindersPage sendRemindersPage =
+                homePage.sendReminderEmailToSelectedStudent(courseIndex, sessionIndex, studentToEmail);
+
+        sendRemindersPage.verifyStatusMessage("Reminder e-mails have been sent out to those students"
+                + " and instructors. Please allow up to 1 hour for all the notification emails to be sent out.");
+
+        ______TS("send reminder email to all student non-submitters");
+        homePage = getNewPageInstance(url, InstructorHomePage.class);
+        homePage.verifySessionDetails(courseIndex, sessionIndex, feedbackSessionOpen);
+        sendRemindersPage = homePage.sendReminderEmailToNonSubmitters(courseIndex, sessionIndex);
+
+        sendRemindersPage.verifyStatusMessage("Reminder e-mails have been sent out to those students"
+                + " and instructors. Please allow up to 1 hour for all the notification emails to be sent out.");
     }
 
     private String getExpectedResponseRate(FeedbackSession session) {

src/e2e/java/teammates/e2e/pageobjects/InstructorFeedbackResultsPage.java

@@ -134,9 +134,11 @@ public void verifyNoResponsePanelDetails(List<Student> noResponseStudents) {
         verifyTableBodyValues(getNoResponseTable(), getExpectedNoResponseDetails(noResponseStudents));
     }
 
-    public void remindAllNonResponders() {
+    public InstructorSessionSendRemindersPage remindAllNonResponders() {
         click(remindAllButton);
-        click(waitForElementPresence(By.id("btn-confirm-send-reminder")));
+        InstructorSessionSendRemindersPage sendRemindersPage = changePageType(InstructorSessionSendRemindersPage.class);
+        sendRemindersPage.submitReminderToPreselectedNonSubmitters();
+        return sendRemindersPage;
     }
 
     public void includeGroupingByTeam(boolean isIncluded) {

src/e2e/java/teammates/e2e/pageobjects/InstructorFeedbackSessionsPage.java

@@ -205,21 +205,25 @@ public void showDeleteTable() {
         }
     }
 
-    public void sendReminderEmailToSelectedStudent(FeedbackSession session, Student student) {
+    public InstructorSessionSendRemindersPage sendReminderEmailToSelectedStudent(
+            FeedbackSession session, Student student) {
         int rowId = getFeedbackSessionRowId(session.getCourseId(), session.getName());
 
         click(waitForElementPresence(By.className("btn-remind-" + rowId)));
         click(waitForElementPresence(By.className("btn-remind-selected-" + rowId)));
-        selectStudentToEmail(student.getEmail());
-        click(waitForElementPresence(By.id("btn-confirm-send-reminder")));
+        InstructorSessionSendRemindersPage sendRemindersPage = changePageType(InstructorSessionSendRemindersPage.class);
+        sendRemindersPage.submitReminderToSelectedStudent(student.getEmail());
+        return sendRemindersPage;
     }
 
-    public void sendReminderEmailToNonSubmitters(FeedbackSession session) {
+    public InstructorSessionSendRemindersPage sendReminderEmailToNonSubmitters(FeedbackSession session) {
         int rowId = getFeedbackSessionRowId(session.getCourseId(), session.getName());
 
         click(waitForElementPresence(By.className("btn-remind-" + rowId)));
         click(waitForElementPresence(By.className("btn-remind-all-" + rowId)));
-        click(waitForElementPresence(By.id("btn-confirm-send-reminder")));
+        InstructorSessionSendRemindersPage sendRemindersPage = changePageType(InstructorSessionSendRemindersPage.class);
+        sendRemindersPage.submitReminderToPreselectedNonSubmitters();
+        return sendRemindersPage;
     }
 
     public void resendResultsLink(FeedbackSession session, Student student) {
@@ -427,14 +431,11 @@ private void selectCourseToCopyToInModal(WebElement copyFsModal, String courseTo
 
     private void selectStudentToEmail(String studentEmail) {
         WebElement studentList = waitForElementVisibility(By.id("student-list-table"));
+        List<WebElement> rows = studentList.findElements(By.cssSelector("tbody tr"));
 
-        List<WebElement> rows = studentList.findElements(By.tagName("tr"));
         for (WebElement row : rows) {
             List<WebElement> cells = row.findElements(By.cssSelector("td"));
-            if (cells.isEmpty()) {
-                continue;
-            }
-            if (cells.get(4).getText().equals(studentEmail)) {
+            if (!cells.isEmpty() && cells.get(4).getText().equals(studentEmail)) {
                 click(cells.get(0).findElement(By.tagName("input")));
                 break;
             }

src/e2e/java/teammates/e2e/pageobjects/InstructorHomePage.java

@@ -73,27 +73,29 @@ public void unpublishSessionResults(int courseTabIndex, int sessionIndex) {
         clickAndConfirm(unpublishButtons.get(unpublishButtons.size() - 1));
     }
 
-    public void sendReminderEmailToSelectedStudent(int courseTabIndex, int sessionIndex, Student student) {
+    public InstructorSessionSendRemindersPage sendReminderEmailToSelectedStudent(
+            int courseTabIndex, int sessionIndex, Student student) {
         WebElement courseTab = getCourseTab(courseTabIndex);
         click(courseTab.findElement(By.className("btn-remind-" + sessionIndex)));
         List<WebElement> remindSelectedButtons = browser.driver.findElements(
                 By.className("btn-remind-selected-" + sessionIndex)
             );
         click(remindSelectedButtons.get(remindSelectedButtons.size() - 1));
-        selectStudentToEmail(student.getEmail());
-        click(browser.driver.findElement(By.id("btn-confirm-send-reminder")));
-        click(courseTab.findElement(By.className("btn-remind-" + sessionIndex)));
+        InstructorSessionSendRemindersPage sendRemindersPage = changePageType(InstructorSessionSendRemindersPage.class);
+        sendRemindersPage.submitReminderToSelectedStudent(student.getEmail());
+        return sendRemindersPage;
     }
 
-    public void sendReminderEmailToNonSubmitters(int courseTabIndex, int sessionIndex) {
+    public InstructorSessionSendRemindersPage sendReminderEmailToNonSubmitters(int courseTabIndex, int sessionIndex) {
         WebElement courseTab = getCourseTab(courseTabIndex);
         click(courseTab.findElement(By.className("btn-remind-" + sessionIndex)));
         List<WebElement> remindSelectedButtons = browser.driver.findElements(
                 By.className("btn-remind-all-" + sessionIndex)
             );
         click(remindSelectedButtons.get(remindSelectedButtons.size() - 1));
-        click(waitForElementPresence(By.id("btn-confirm-send-reminder")));
-        click(courseTab.findElement(By.className("btn-remind-" + sessionIndex)));
+        InstructorSessionSendRemindersPage sendRemindersPage = changePageType(InstructorSessionSendRemindersPage.class);
+        sendRemindersPage.submitReminderToPreselectedNonSubmitters();
+        return sendRemindersPage;
     }
 
     public void resendResultsLink(int courseTabIndex, int sessionIndex, Student student) {
@@ -203,14 +205,11 @@ private void selectCourseToCopyToInModal(WebElement copyFsModal, String courseTo
 
     private void selectStudentToEmail(String studentEmail) {
         WebElement studentList = waitForElementPresence(By.id("student-list-table"));
+        List<WebElement> rows = studentList.findElements(By.cssSelector("tbody tr"));
 
-        List<WebElement> rows = studentList.findElements(By.tagName("tr"));
         for (WebElement row : rows) {
             List<WebElement> cells = row.findElements(By.cssSelector("td"));
-            if (cells.isEmpty()) {
-                continue;
-            }
-            if (cells.get(4).getText().equals(studentEmail)) {
+            if (!cells.isEmpty() && cells.get(4).getText().equals(studentEmail)) {
                 click(cells.get(0).findElement(By.tagName("input")));
                 break;
             }

src/e2e/java/teammates/e2e/pageobjects/InstructorSessionSendRemindersPage.java

@@ -0,0 +1,46 @@
+package teammates.e2e.pageobjects;
+
+import java.util.List;
+
+import org.openqa.selenium.By;
+import org.openqa.selenium.WebElement;
+
+/**
+ * Represents the instructor send reminders page.
+ */
+public class InstructorSessionSendRemindersPage extends AppPage {
+    public InstructorSessionSendRemindersPage(Browser browser) {
+        super(browser);
+    }
+
+    @Override
+    protected boolean containsExpectedPageContents() {
+        return getPageTitle().contains("Send Reminders");
+    }
+
+    public void submitReminderToSelectedStudent(String studentEmail) {
+        selectStudentByEmail(studentEmail);
+        submitReminders();
+    }
+
+    public void submitReminderToPreselectedNonSubmitters() {
+        submitReminders();
+    }
+
+    private void submitReminders() {
+        click(waitForElementPresence(By.id("btn-confirm-send-reminder")));
+    }
+
+    private void selectStudentByEmail(String studentEmail) {
+        WebElement studentList = waitForElementVisibility(By.id("student-list-table"));
+        List<WebElement> rows = studentList.findElements(By.cssSelector("tbody tr"));
+
+        for (WebElement row : rows) {
+            List<WebElement> cells = row.findElements(By.cssSelector("td"));
+            if (!cells.isEmpty() && cells.get(4).getText().equals(studentEmail)) {
+                click(cells.get(0).findElement(By.tagName("input")));
+                break;
+            }
+        }
+    }
+}

src/main/java/teammates/common/util/Const.java

@@ -259,6 +259,8 @@ public static class WebPageURIs {
         public static final String INSTRUCTOR_SESSION_EDIT_PAGE = INSTRUCTOR_PAGE + "/sessions/{feedbackSessionId}/edit";
         public static final String INSTRUCTOR_SESSION_INDIVIDUAL_EXTENSION_PAGE = INSTRUCTOR_PAGE
                 + "/sessions/{feedbackSessionId}/individual-extension";
+        public static final String INSTRUCTOR_SESSION_SEND_REMINDERS_PAGE = INSTRUCTOR_PAGE
+                + "/sessions/{feedbackSessionId}/send-reminders";
         public static final String INSTRUCTOR_SESSION_RESULTS_PAGE = INSTRUCTOR_PAGE
                 + "/sessions/{feedbackSessionId}/result";
         public static final String INSTRUCTOR_SESSION_REPORT_PAGE = INSTRUCTOR_PAGE

src/web/app/components/sessions-table/respondent-list-info-table/respondent-list-info-table.component.html

@@ -1,5 +1,8 @@
 @if (studentListInfoTableRowModels.length > 0) {
   <div>
+    @if (studentTableLabel) {
+      <h1>{{ studentTableLabel }}</h1>
+    }
     <table id="student-list-table" class="table table-bordered">
       <thead>
         <tr class="background-color-medium-gray">
@@ -148,6 +151,9 @@ <h4 class="p-2 bg-info">There are no students enrolled to remind.</h4>
 <br />
 @if (instructorListInfoTableRowModels.length > 0) {
   <div>
+    @if (instructorTableLabel) {
+      <h1>{{ instructorTableLabel }}</h1>
+    }
     <table id="instructor-list-table" class="table table-bordered">
       <thead>
         <tr class="background-color-medium-gray">

src/web/app/components/sessions-table/respondent-list-info-table/respondent-list-info-table.component.ts

@@ -24,6 +24,12 @@ export class RespondentListInfoTableComponent {
   @Input()
   shouldDisplayHasSubmittedSessionColumn = false;
 
+  @Input()
+  studentTableLabel = '';
+
+  @Input()
+  instructorTableLabel = '';
+
   @Input()
   studentListInfoTableRowModels: StudentListInfoTableRowModel[] = [];