[#12329] Refactoring of sortable tables - Sessions table

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

@@ -88,7 +88,6 @@ public void testAll() {
 
         ______TS("verify loaded data");
         FeedbackSessionAttributes[] loadedSessions = { openSession, closedSession };
-        feedbackSessionsPage.sortByCourseId();
         feedbackSessionsPage.verifySessionsTable(loadedSessions);
 
         ______TS("verify response rate");
@@ -218,7 +217,6 @@ public void testAll() {
         FeedbackSessionAttributes[] sessionsForRestore = { openSession, newSession, closedSession, copiedSession2,
                 copiedSession };
         feedbackSessionsPage.restoreSession(closedSession);
-
         feedbackSessionsPage.verifyStatusMessage("The feedback session has been restored.");
         feedbackSessionsPage.sortBySessionsName();
         feedbackSessionsPage.verifySessionsTable(sessionsForRestore);
@@ -229,6 +227,8 @@ public void testAll() {
         ______TS("permanently delete session");
         FeedbackSessionAttributes[] sessionsForDelete = { copiedSession, copiedSession2, closedSession,
                 openSession };
+
+        feedbackSessionsPage.sortBySessionsName();
         feedbackSessionsPage.moveToRecycleBin(newSession);
         feedbackSessionsPage.deleteSession(newSession);
 
@@ -239,8 +239,7 @@ public void testAll() {
         verifyAbsentInDatabase(newSession);
 
         ______TS("restore all session");
-        FeedbackSessionAttributes[] sessionsForRestoreAll = { openSession, closedSession, copiedSession2,
-                copiedSession };
+        FeedbackSessionAttributes[] sessionsForRestoreAll = { copiedSession, copiedSession2, closedSession, openSession };
         feedbackSessionsPage.moveToRecycleBin(copiedSession);
         feedbackSessionsPage.moveToRecycleBin(copiedSession2);
         feedbackSessionsPage.restoreAllSessions();

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

@@ -207,8 +207,8 @@ public void testAll() {
         homePage.sortCoursesByCreationDate();
         courseIndex = 1;
         otherCourseIndex = 0;
-        FeedbackSessionAttributes[] otherCourseSessionsWithCopy2 = { copiedSession2, otherCourseSession };
-        homePage.verifyCourseTabDetails(otherCourseIndex, otherCourse, otherCourseSessionsWithCopy2);
+        FeedbackSessionAttributes[] otherCourseSessionsWithCopyTwo = { copiedSession2, otherCourseSession };
+        homePage.verifyCourseTabDetails(otherCourseIndex, otherCourse, otherCourseSessionsWithCopyTwo);
         assertNotNull(getSoftDeletedSession(copiedSession.getFeedbackSessionName(),
                 instructor.getGoogleId()));
 

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

@@ -96,7 +96,7 @@ public class InstructorFeedbackSessionsPage extends AppPage {
     @FindBy(id = "btn-create-session")
     private WebElement createSessionButton;
 
-    @FindBy(className = "sessions-table")
+    @FindBy(id = "sessions-table")
     private WebElement sessionsTable;
 
     @FindBy(id = "deleted-sessions-heading")
@@ -501,6 +501,8 @@ && getFeedbackSessionName(i).equals(sessionName)) {
 
     private int getSoftDeletedFeedbackSessionRowId(String courseId, String sessionName) {
         int i = 0;
+        assertEquals(courseId, getSoftDeletedFeedbackSessionCourseId(i));
+        assertEquals(sessionName, getSoftDeletedFeedbackSessionName(i));
 
         while (i < getNumSoftDeletedFeedbackSessions()) {
             if (getSoftDeletedFeedbackSessionCourseId(i).equals(courseId)

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

@@ -151,7 +151,7 @@ private String getCourseDetails(int courseTabIndex) {
     }
 
     private WebElement getSessionsTable(int courseTabIndex) {
-        return getCourseTab(courseTabIndex).findElement(By.className("sessions-table"));
+        return getCourseTab(courseTabIndex).findElement(By.id("sessions-table-" + courseTabIndex));
     }
 
     private String getDateString(Instant instant, String timeZone) {

src/e2e/resources/data/InstructorHomePageE2ETest.json

@@ -175,7 +175,7 @@
       "creatorEmail": "IHome.instructor.tmms@gmail.tmt",
       "instructions": "Please please fill in the following questions.",
       "createdTime": "2012-03-20T23:59:00Z",
-      "startTime": "2012-03-29T15:00:00Z",
+      "startTime": "2012-03-29T04:00:00Z",
       "endTime": "2012-05-01T04:00:00Z",
       "sessionVisibleFromTime": "2012-03-28T16:00:00Z",
       "resultsVisibleFromTime": "2012-05-01T16:00:00Z",

src/web/app/components/question-types/question-statistics/contribution-question-statistics/contribution-question-statistics.component.ts

@@ -88,27 +88,44 @@ export class ContributionQuestionStatisticsComponent
           value: statistics.results[email].claimed,
           customComponent: {
             component: ContributionComponent,
-            componentData: { value: statistics.results[email].claimed },
+            componentData: () => {
+              return {
+                value: statistics.results[email].claimed,
+              };
+            },
           },
         },
         {
           value: statistics.results[email].perceived,
           customComponent: {
             component: ContributionComponent,
-            componentData: { value: statistics.results[email].perceived },
+            componentData: () => {
+              return {
+                value: statistics.results[email].perceived,
+              };
+            },
           },
         },
         {
           value: this.emailToDiff[email],
           customComponent: {
             component: ContributionComponent,
-            componentData: { value: this.emailToDiff[email], diffOnly: true },
+            componentData: () => {
+              return {
+                value: this.emailToDiff[email],
+                diffOnly: true,
+              };
+            },
           },
         },
         {
           customComponent: {
             component: ContributionRatingsListComponent,
-            componentData: { ratingsList: statistics.results[email].perceivedOthers },
+            componentData: () => {
+              return {
+                ratingsList: statistics.results[email].perceivedOthers,
+              };
+            },
           },
         },
       ];