sort tasks only when comparing panels

CS2103AUG2016-W11-C2 · Nov 5, 2016 · 9aeda2e · 9aeda2e
1 parent 835b967
commit 9aeda2e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/src/test/java/guitests/ToDoListGuiTest.java b/src/test/java/guitests/ToDoListGuiTest.java
@@ -127,6 +127,7 @@ protected void assertResultMessage(String expected) {
      * Asserts the task shown in each panel will match
      */
     protected void assertAllPanelsMatch(TestTask[] expectedList) {
+        TestUtil.sortTasks(expectedList);
         TestTask[] expectedDoItSoonTasks = TestUtil.getDoItSoonTasks(expectedList);
         TestTask[] expectedDoItAnytimeTasks = TestUtil.getDoItAnytimeTasks(expectedList);
         TestTask[] expectedDoneTasks = TestUtil.getDoneTasks(expectedList);

diff --git a/src/test/java/seedu/agendum/testutil/TestUtil.java b/src/test/java/seedu/agendum/testutil/TestUtil.java
@@ -295,7 +295,6 @@ public static TestTask[] removeTaskFromList(final TestTask[] list, int targetInd
      */
     public static TestTask[] replaceTaskFromList(TestTask[] tasks, TestTask task, int index) {
         tasks[index] = task;
-        sortTasks(tasks);
         return tasks;
     }
 
@@ -309,7 +308,6 @@ public static TestTask[] addTasksToList(final TestTask[] tasks, TestTask... task
         List<TestTask> listOfTasks = asList(tasks);
         listOfTasks.addAll(asList(tasksToAdd));
         TestTask[] newTasks = listOfTasks.toArray(new TestTask[listOfTasks.size()]);
-        sortTasks(newTasks);
         return newTasks;
     }
 
@@ -319,7 +317,7 @@ private static <T> List<T> asList(T[] objs) {
         return list;
     }
 
-    private static void sortTasks(TestTask[] tasks) {
+    public static void sortTasks(TestTask[] tasks) {
         Arrays.sort(tasks);
     }