From 8b4ccfb35039b4007e47cab1a0f42732f70c1fff Mon Sep 17 00:00:00 2001
From: Fan-Wu Yang <fwy13@uclive.ac.nz>
Date: Wed, 5 Apr 2017 19:50:59 +1200
Subject: [PATCH] Unit tests for stop and route

---
 src/test/java/model/DataManagerTest.java      |  42 +++++++++++++++++-
 .../test-classes/model/DataManagerTest.class  | Bin 1127 -> 2684 bytes
 2 files changed, 41 insertions(+), 1 deletion(-)

diff --git a/src/test/java/model/DataManagerTest.java b/src/test/java/model/DataManagerTest.java
index a53653d..cedd1f2 100644
--- a/src/test/java/model/DataManagerTest.java
+++ b/src/test/java/model/DataManagerTest.java
@@ -4,12 +4,13 @@ package model;/**
 
 import static org.junit.Assert.*;
 
+import javafx.collections.FXCollections;
 import org.junit.Test;
 
 public class DataManagerTest {
 
     @Test
-    public void AddRide(){
+    public void addRide(){
         DataManager dataManager = new DataManager();
         Ride ride = new Ride("Nissan March", "Blue", "HB9999", 2008, 4);
         dataManager.addRides(ride);
@@ -23,4 +24,43 @@ public class DataManagerTest {
         assertTrue(dataManager.getRides().size() == 0);
     }
 
+    @Test
+    public void addStop(){
+        DataManager dataManager = new DataManager();
+        Stop stop1 = new Stop("Stop 1");
+        Stop stop2 = new Stop("Stop 2");
+        Stop stop3 = new Stop("Stop 1");
+        dataManager.addStop(stop1);
+        assertTrue(dataManager.getStops().size() == 1);
+        assertTrue(!dataManager.addStop(stop1));
+        assertTrue(!dataManager.addStop(stop3));
+        assertTrue(dataManager.addStop(stop2));
+        assertTrue(dataManager.getStops().size() == 2);
+    }
+
+    @Test
+    public void addRoute(){
+        DataManager dataManager = new DataManager();
+        Stop stop1 = new Stop("Stop 1");
+        Stop stop2 = new Stop("Stop 2");
+        Stop stop3 = new Stop("Stop 3");
+        Stop stop4 = new Stop("Stop 4");
+        Stop stop5 = new Stop("Stop 5");
+        Stop[] stops = {stop1, stop2, stop3, stop4, stop5};
+        Route route = new Route("Route1", FXCollections.observableArrayList(stops));
+        dataManager.addRoute(route);
+        assertTrue(dataManager.getRoutes().size() == 1);
+        //does not add duplicate routes
+        assertTrue(!dataManager.addRoute(route));
+        assertTrue(dataManager.getRoutes().size() == 1);
+        Route route2 = new Route("Route 2", FXCollections.observableArrayList(stops));
+        assertTrue(!dataManager.addRoute(route2));
+        assertTrue(dataManager.getRoutes().size() == 1);
+        //can acutally add another route
+        Stop[] stops1 = {stop1, stop2};
+        Route route3 = new Route("Route 3", FXCollections.observableArrayList(stops1));
+        assertTrue(dataManager.addRoute(route3));
+        assertTrue(dataManager.getRoutes().size() == 2);
+    }
+
 }
\ No newline at end of file
diff --git a/target/test-classes/model/DataManagerTest.class b/target/test-classes/model/DataManagerTest.class
index 81e07dede49a4ce41a7c01f07394e8e230981dc4..6d5f7df26e1d31aff27bb14455a2da4c43983f57 100644
GIT binary patch
literal 2684
zcma)7OIK4@6#fpm$-UfM1Okdt6nxbnU<4mk@Db2jsR3JJ(O_Sf5b<J2GznDgOYP$?
zbghm$YA2m^P|#vK>!fRS-f>5rbgd3MsI}idH-S`GRl+)FpS}0lXYcRZJNfya?|%ev
z2%p;6i4ha8*r=2JyoIENQ43=>3|tWHtFpWn#yC<oF5-1LPRQ|+NS95dElk>I!W9c|
zi0VyQ-m>tvg?B8tCZ;T8Y)m6-BZ{kHoss351<%5)iR&hE3Zdg(-YcI}2*kQ36pYh_
z>8wIy(#vPZ=I5re#f$D#j+#iaka2SpZqbu-MQN0;c_oFm<Q#A1;%D5lJL=}$8Q#j4
z%7Y4~J3T$_NyPB<s!^e3?G1VxMeXWTjB=6MhE&<jTpx9B=tl_}@luG6&*#hDTz0}M
zc}!?1pD&c%vRBA6pQdD?I1`_pXJPT0L{jC#4TWHdTc1Mp718n)t?sYv2Q@lr1xxZT
z%$He!b~seo4_9_R)^MegZZ%NR_iw0HAxAzC(t5rSk_1Z3z}N7tQzT2lP8H^hne16l
z;BI{i%l)(Ng6m)xb}KZmL7SI-osYFM=9NlrzH`(qX08#|;oLk!hRzKi<DUeXbC5^D
z!42FrQFKs3*~Gkq1-$Fv7H&JZgZD%#tVEXT7(=$sK8f*u_0;dMqd#!)A#PSx2de_{
z@~;rEa3w__o{)yM&%{R#?&6+AyzgKTA3OL&0-4(0NxUi&V{TkixUfsU1)O-!&CkRy
zOwDF9R1L<GtIAZl=;dbypGPw?B2%DgXZMlb##og{=|*jw$(99Y8u!GybkM7};+aA&
z$Csr6nM5gDT#z|QdU_0YNG|{D#Q9@bmKYoHL)MnV?rB=>=%Es$)75shaIq-l8nH>q
z%`Rr=3JY1k4GhMTkuG_6=)w6Ok@#y-5T}(gbr*L{5(DNV2bS5nPWoprpI)t)16^3D
zd8k-)Z%g3{M`KrNp7XgGd_KsIu9N6<U@f&Ytx%>%Pl~A*&g4|W4t{p>TNwPNRWeXq
z_wW?wuDA#HEkS)tp$;!_4QZu~y_6l_0Woy(q_B^d>-6Q{VIn}?+%oFYO9&*FVWcC$
zNGM`9o4=Y%utvLk7E#~52s`i);X3qu1=IK%M&JRQ?!Y1%#Pkpw6uzi2=sR2**Rx8J
z0iEPN*pxTGLL;{DH=zT~IE)qypcP}-gcRCv8Jm&gFM$-&=_gw$#6lL`=wU%y=-G??
zEGkZ~7K9#y6MDczUqZDdRC_|9p9aC?Aa}*VuOOgyGh~pud#eaEr6bKth>R|SQVXRq
zEs1Wx5I9;dv8;`{XyeQOR^3{QLe!vdt)iD_!L52bSaW+7mHh}ap>0HEJ0irSoptXe
z`@Q6Un9Pqe`3&j1P7LqTBFG%_i&#PCb_JOT3Pkh>k!dGdFH#E7e-cM285*r!u%6%~
zf+mh>$OcTvKafxx5~?wwwyr^Mu!>&Yc?NL^XvZ_r+oaLcK(>h()Z%6lL)H&OI?$6!
z8@;J?FlwaIp=dCbHlv}`fVqrlI<h5dKET!`v=3O_QEL&~gn{jA8PK*`#MkrE5j$2Q
z)^|ti8Sz=Sf;CAF*g8gM)UIR|?v92T!{^q|Ykh9V+1VgB7W+0rM%&r7d&xsT4&Vg(
z$V@+G*`zng=x5~Ub8_|#*?NRy_=T+ehU54XCphpYRXt9r?MP_;@Ax!S`B!aK{#8Sj
ze-&8a|E~)F{rH3Nj<9!5F!BjXA!hIs+u;<y7-sPuqYp9HN9?y@O7(m@j?=W-%%B5j
xs0-^Z8iw-(_fh{;7x}$p;w2MjrK8wREeZ8EI(XLNV`(<Au4%HDiBlcU{R<}{Hvs?u

delta 387
zcmXYs&o)9)6vlt&-s)V}^{+o7sgRIJ6biYO3=FJA6H@~-UV&%GeG<#SS_U4#3wQ)C
zV2EX(at7bGzy0Ui`+Ubgos0M1*EgWbsmm10nu3dEc*SAWVa=iF5@OwCO9nPZ*`(~U
zMa9T%BX>+x)zmb*{l{z7>VI8^3gQu*sY;Ns@^+M1(6!p7D0?3J)IAy;XbwG&Xljl<
zd|DoDIvyu<O;mqw+8xznfl0Z{JPU;w3C2a0g_Ur)_=0*GloNuHL2`s?iM?S1NoGVT
zW@XqS9(;uFA53o%S8xo6aZ*f>l~pxa(U87#67<C7N~UjR-uUIP1@Hd-9ByQ4PJC0+
knPy(RGg6Nc`Jtl8YI43x`YP=!`Ni<GMPbDfVe_5*A2{nJo&W#<