Open one of the test case

nus-cs2103-AY2223S2 · Apr 10, 2023 · 01f40c2 · 01f40c2
1 parent bc587e2
commit 01f40c2
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 8 deletions.
diff --git a/src/test/java/seedu/address/logic/commands/DeleteScoreCommandTest.java b/src/test/java/seedu/address/logic/commands/DeleteScoreCommandTest.java
@@ -5,6 +5,7 @@
 import static seedu.address.logic.commands.CommandTestUtil.assertCommandFailure;
 import static seedu.address.logic.commands.CommandTestUtil.assertCommandSuccess;
 import static seedu.address.logic.commands.DeleteScoreCommand.MESSAGE_DELETE_SCORE_SUCCESS;
+import static seedu.address.testutil.Assert.assertThrows;
 import static seedu.address.testutil.TypicalIndexes.INDEX_FIRST_SCORE;
 import static seedu.address.testutil.TypicalIndexes.INDEX_FIRST_STUDENT;
 import static seedu.address.testutil.TypicalIndexes.INDEX_SECOND_SCORE;
@@ -24,10 +25,10 @@ class DeleteScoreCommandTest {
 
     private Model model = new ModelManager(getTypicalMathutoring(), new UserPrefs());
 
-    /*@Test
+    @Test
     public void constructor_nullIndexes_throwsNullPointerException() {
         assertThrows(NullPointerException.class, () -> new DeleteScoreCommand(null, null));
-    }*/
+    }
 
     @Test
     void execute_validIndexScoreDelete_success() {
@@ -56,10 +57,10 @@ void execute_invalidStudentIndexUnfilteredList_failure() {
 
     @Test
     void execute_validStudentIndexInvalidScoreIndexUnfilteredList_failure() {
-        Index outOfBoundTaskIndex = Index.fromOneBased(model.getFilteredStudentList().get(INDEX_FIRST_STUDENT
-                .getZeroBased()).getTaskList().asUnmodifiableObservableList().size() + 1);
+        Index outOfBoundScoreIndex = Index.fromOneBased(model.getFilteredStudentList().get(INDEX_FIRST_STUDENT
+                .getZeroBased()).getScoreList().asUnmodifiableObservableList().size() + 1);
 
-        DeleteScoreCommand deleteScoreCommand = new DeleteScoreCommand(INDEX_FIRST_STUDENT, outOfBoundTaskIndex);
+        DeleteScoreCommand deleteScoreCommand = new DeleteScoreCommand(INDEX_FIRST_STUDENT, outOfBoundScoreIndex);
 
         assertCommandFailure(deleteScoreCommand, model, Messages.MESSAGE_INVALID_SCORE_DISPLAYED_INDEX);
     }

diff --git a/src/test/java/seedu/address/logic/parser/AddScoreCommandParserTest.java b/src/test/java/seedu/address/logic/parser/AddScoreCommandParserTest.java
@@ -69,15 +69,15 @@ public void parse_invalidDate_failure() {
     public void parse_futureDate_failure() {
         String expectedMessage = String.format(Date.MESSAGE_INVALID_DATE);
 
-        // invalid date format
+        // future date
         assertParseFailure(parser, "1" + SCORE_FULL_DATE_IN_FUTURE, expectedMessage);
     }
 
     @Test
     public void parse_missingAllVariables() {
         String expectedMessage = String.format(MESSAGE_INVALID_COMMAND_FORMAT, AddScoreCommand.MESSAGE_USAGE);
 
-        // missing score date
+        // missing multiple variables
         assertParseFailure(parser, "1" + SCORE_MISSING_ALL_VARIABLES, expectedMessage);
     }
 }
diff --git a/src/test/java/seedu/address/logic/parser/FilterCommandParserTest.java b/src/test/java/seedu/address/logic/parser/FilterCommandParserTest.java
@@ -30,5 +30,4 @@ public void parse_validArgs_returnsFindCommand() {
         // multiple whitespaces between keywords
         assertParseSuccess(parser, " \n friends \n \t owesMoney  \t", expectedFilterCommand);
     }
-
 }
-Original file line number
+Diff line change
@@ Expand Up / @@ -30,5 +30,4 @@ public void parse_validArgs_returnsFindCommand() { @@
             // multiple whitespaces between keywords
             assertParseSuccess(parser, " \n friends \n \t owesMoney  \t", expectedFilterCommand);
         }
     }