src/main/java/ch/zhaw/gartenverwaltung/io/JsonPlantDatabase.java

@@ -1,6 +1,5 @@
 package ch.zhaw.gartenverwaltung.io;
 
-import ch.zhaw.gartenverwaltung.types.GrowthPhase;
 import ch.zhaw.gartenverwaltung.types.HardinessZone;
 import ch.zhaw.gartenverwaltung.types.Plant;
 import com.fasterxml.jackson.databind.ObjectMapper;
@@ -83,12 +82,6 @@ public class JsonPlantDatabase implements PlantDatabase {
             result = mapper.readerForListOf(Plant.class).readValue(dataSource);
 
             for (Plant plant : result) {
-                // for discussion because of failing tests:
-                // for(GrowthPhase growthPhase: plant.lifecycle()) {
-                //    if(growthPhase.zone()==zone) {
-                //      keep in result, remove if no match
-                //    }
-                // }
                 plant.inZone(currentZone);
             }
 

src/main/resources/ch/zhaw/gartenverwaltung/io/plantdb.json

@@ -30,7 +30,7 @@
             "name": "Germinate",
             "relativeStartDate": -14,
             "relativeEndDate": null,
-            "description": "Take an egg carton and fill it with soil. Put the seedling deep enough so its half covered with soil. Keep it in 10-15 * Celsius with lots of light.",
+            "description": "\"Take an egg carton and fill it with soil. Put the seedling deep enaugh so its half covered with soil. Keep it in 10-15 * Celsius with lots of light.\"",
             "interval": null,
             "isOptional": false
           }
@@ -52,7 +52,7 @@
             "name": "hilling",
             "relativeStartDate": 0,
             "relativeEndDate": null,
-            "description": "When the plants are 20 cm tall, begin hilling the potatoes by gently mounding the soil from the center of your rows around the stems of the plant. Mound up the soil around the plant until just the top few leaves show above the soil. Two weeks later, hill up the soil again when the plants grow another 20 cm.",
+            "description": "\"When the plants are 20 cm tall, begin hilling the potatoes by gently mounding the soil from the center of your rows around the stems of the plant. Mound up the soil around the plant until just the top few leaves show above the soil. Two weeks later, hill up the soil again when the plants grow another 20 cm.\"",
             "interval": 21,
             "isOptional": false
           }

src/test/java/ch/zhaw/gartenverwaltung/io/JsonPlantDatabaseTest.java

@@ -26,7 +26,7 @@ public class JsonPlantDatabaseTest {
 
     @Test
     @DisplayName("Check if results are retrieved completely")
-    void getPlantListNotEmpty() {
+    void getPlantList() {
         List<Plant> testList;
         try {
             testList = testDatabase.getPlantList(HardinessZone.ZONE_8A);
@@ -42,47 +42,13 @@ public class JsonPlantDatabaseTest {
         Assertions.assertEquals(expected,names);
     }
 
-    @Test
-    @DisplayName("Check if results are retrieved correctly when empty")
-    void getPlantListEmpty() {
-        List<Plant> testList;
-        try {
-            testList = testDatabase.getPlantList(HardinessZone.ZONE_1A);
-        } catch (IOException e) {
-            throw new RuntimeException(e);
-        } catch (HardinessZoneNotSetException e) {
-            throw new RuntimeException(e);
-        }
-        Assertions.assertEquals(0, testList.size());
-
-
-    }
-
     @Test
     @DisplayName("Check whether single access works.")
-    void getPlantByIdAndZone() {
+    void getPlantById() {
         Optional<Plant> testPlant;
         try {
-            testPlant = testDatabase.getPlantById(HardinessZone.ZONE_8A, 1);
-        } catch (IOException e) {
-            throw new RuntimeException(e);
-        } catch (HardinessZoneNotSetException e) {
-            throw new RuntimeException(e);
-        }
-        Assertions.assertTrue(testPlant.isPresent());
-        Assertions.assertEquals("Early Carrot", testPlant.get().name());
-    }
-
-    @Test
-    @DisplayName("Check whether single access respects zone correctly.")
-    void getPlantByIdAndWrongZone() {
-        Optional<Plant> testPlant;
-        try {
-            testPlant = testDatabase.getPlantById(HardinessZone.ZONE_1A, 1);
-            Assertions.assertFalse(testPlant.isPresent());
-            testPlant = testDatabase.getPlantById(HardinessZone.ZONE_8A, 1);
-            Assertions.assertTrue(testPlant.isPresent());
-
+            testDatabase.getPlantList(HardinessZone.ZONE_8A);
+            testPlant = testDatabase.getPlantById(1);
         } catch (IOException e) {
             throw new RuntimeException(e);
         } catch (HardinessZoneNotSetException e) {
@@ -97,7 +63,8 @@ public class JsonPlantDatabaseTest {
     void getPlantByIdMustFail() {
         Optional<Plant> testPlant;
         try {
-            testPlant = testDatabase.getPlantById(HardinessZone.ZONE_8A, 99);
+            testDatabase.getPlantList(HardinessZone.ZONE_8A);
+            testPlant = testDatabase.getPlantById(99);
         } catch (IOException e) {
             throw new RuntimeException(e);
         } catch (HardinessZoneNotSetException e) {