Merge remote-tracking branch 'origin/master'

src/main/java/seng202/group9/Controller/Dataset.java

@@ -16,19 +16,19 @@ public class Dataset {
 
     String name;
     public static boolean getExisting = true;//constructor variables for action
-    static boolean createNew = false;//constructor variables for action
-    ArrayList<Airline> airlines;
-    ArrayList<Airport> airports;
-    ArrayList<Route> routes;
-    ArrayList<FlightPath> flightPaths;
-    ArrayList<Country> countries;
-    ArrayList<City> cities;
-    LinkedHashMap<String, Airline> airlineDictionary;
-    LinkedHashMap<String, Airport> airportDictionary;
-    LinkedHashMap<String, Route> routeDictionary;
-    LinkedHashMap<Integer, FlightPath> flightPathDictionary;
-    LinkedHashMap<String, Country> countryDictionary;
-    LinkedHashMap<String, City> cityDictionary;
+    public static boolean createNew = false;//constructor variables for action
+    private ArrayList<Airline> airlines;
+    private ArrayList<Airport> airports;
+    private ArrayList<Route> routes;
+    private ArrayList<FlightPath> flightPaths;
+    private ArrayList<Country> countries;
+    private ArrayList<City> cities;
+    private LinkedHashMap<String, Airline> airlineDictionary;
+    private LinkedHashMap<String, Airport> airportDictionary;
+    private LinkedHashMap<String, Route> routeDictionary;
+    private LinkedHashMap<Integer, FlightPath> flightPathDictionary;
+    private LinkedHashMap<String, Country> countryDictionary;
+    private LinkedHashMap<String, City> cityDictionary;
 
     /**
      *

src/main/java/seng202/group9/Core/Airline.java

@@ -285,7 +285,7 @@ public class Airline{
 	 */
 	@Override
 	public String toString(){
-		return name + ", IATA:" + IATA + ", ICAO: " + ICAO;
+		return name + ", IATA: " + IATA + ", ICAO: " + ICAO;
 	}
 	
 }

src/test/java/seng202/group9/AirlineTest.java

@@ -123,7 +123,9 @@ public class AirlineTest extends TestCase {
     	//324,"All Nippon Airways","ANA All Nippon Airways","NH","ANA","ALL NIPPON","Japan","Y"
     	Airline allNipponAirways = new Airline(324, "All Nippon Airways", "ANA All Nippon Airways",
     			"NH", "ANA", "ALL NIPPON", "Japan", "Y");
-    	assertTrue(allNipponAirways.toString() == "All Nippon Airways");
-    	assertEquals(allNipponAirways + "" ,"All Nippon Airways");
+		//name + ", IATA:" + IATA + ", ICAO: " + ICAO
+		System.out.println(allNipponAirways.toString());
+    	assertTrue(allNipponAirways.toString().equals("All Nippon Airways, IATA: NH, ICAO: ANA"));
+    	assertEquals(allNipponAirways + "" ,"All Nippon Airways, IATA: NH, ICAO: ANA");
     }
 }