https://codereview.appspot.com/76410043/diff/60001/src/test/java/teammates/te... File src/test/java/teammates/test/cases/ui/browsertests/InstructorFeedbackEditPageUiTest.java (right): https://codereview.appspot.com/76410043/diff/60001/src/test/java/teammates/te... src/test/java/teammates/test/cases/ui/browsertests/InstructorFeedbackEditPageUiTest.java:211: private void testTwoNewEssayQuestionsWithRecipientBeingNobodySpecific() { indent? https://codereview.appspot.com/76410043/diff/60001/src/test/java/teammates/te... src/test/java/teammates/test/cases/ui/browsertests/InstructorFeedbackEditPageUiTest.java:220: // move this to const file can keep this here for now as it''s only used here. Remove comment or add TODO? https://codereview.appspot.com/76410043/diff/60001/src/test/java/teammates/te... src/test/java/teammates/test/cases/ui/browsertests/InstructorFeedbackEditPageUiTest.java:223: feedbackEditPage.numberOfRowsInQuestionTableNewParticipateTable()); indent? https://codereview.appspot.com/76410043/diff/60001/src/test/java/teammates/te... src/test/java/teammates/test/cases/ui/browsertests/InstructorFeedbackEditPageUiTest.java:228: } Maybe also check the content of the rows? It's to ensure we're not hiding the wrong one.