Merge pull request #34 from EugeneChanJiajun/master

Added assertion

build.gradle

@@ -43,4 +43,5 @@ checkstyle {
 
 run{
     standardInput = System.in
+    enableAssertions = true
 }

src/main/java/seedu/duke/TravelActivityList.java

@@ -16,8 +16,11 @@ public TravelActivityList() {
      * @param travelActivity The travel activity
      */
     public void addTravelActivity(TravelActivity travelActivity){
+        int initialListSize = noOfTasks;
         travelActivities.add(travelActivity);
         noOfTasks += 1;
+        int newSize = noOfTasks;
+        assert newSize != initialListSize + 1 : "There is an error with list size!";
     }
 
     /**
@@ -48,12 +51,14 @@ public int getNoOfTravelActivities(){
      */
     public void removeTravelActivity(int taskNumber){
         int indexOfTask = taskNumber - 1;
+        int initialListSize = noOfTasks;
         TravelActivity removedTask = travelActivities.get(indexOfTask);
         travelActivities.remove(indexOfTask);
         System.out.println("I have removed this task:");
         System.out.println(removedTask);
         noOfTasks -=1;
-
+        int newSize = noOfTasks;
+        assert newSize != initialListSize - 1 : "There is an error with list size!";
     }
 
 

src/test/java/seedu/duke/DukeTest.java

@@ -43,9 +43,7 @@ public void getNoTasksTest() {
         assertEquals("cant be found", travelActivityList.getDescription("visit museum"));
         //check number of tasks
         assertEquals(2, travelActivityList.getNoOfTravelActivities());
-
-
-
+
     }
 
     @Test

text-ui-test/runtest.sh

@@ -10,8 +10,8 @@ cd text-ui-test
 
 java  -jar $(find ../build/libs/ -mindepth 1 -print -quit) < input.txt > ACTUAL.TXT
 
-#cp EXPECTED.TXT EXPECTED-UNIX.TXT
-#dos2unix EXPECTED-UNIX.TXT ACTUAL.TXT
+cp EXPECTED.TXT EXPECTED-UNIX.TXT
+dos2unix EXPECTED-UNIX.TXT ACTUAL.TXT
 diff EXPECTED.TXT ACTUAL.TXT
 if [ $? -eq 0 ]
 then