continue GardenPlan tests

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

@@ -1,5 +1,6 @@
 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;
@@ -82,6 +83,12 @@ 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 enaugh 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 enough 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/JsonGardenPlanTest.java

@@ -0,0 +1,109 @@
+package ch.zhaw.gartenverwaltung.io;
+
+import ch.zhaw.gartenverwaltung.types.Crop;
+import ch.zhaw.gartenverwaltung.types.HardinessZone;
+import ch.zhaw.gartenverwaltung.types.Plant;
+import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.DisplayName;
+import org.junit.jupiter.api.Test;
+
+import java.io.IOException;
+import java.text.ParseException;
+import java.text.SimpleDateFormat;
+import java.time.LocalDate;
+import java.time.format.DateTimeFormatter;
+import java.util.Arrays;
+import java.util.List;
+import java.util.Optional;
+import java.util.stream.Collectors;
+
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+public class JsonGardenPlanTest {
+    GardenPlan testDatabase;
+    DateTimeFormatter formatter = DateTimeFormatter.ofPattern("dd.MM.yyyy");
+
+    @BeforeEach
+    void connectToDb() {
+        testDatabase = new JsonGardenPlan();
+    }
+
+
+    @Test
+    @DisplayName("Check if results are retrieved completely.")
+    void getCropsNotEmpty() {
+        List<Crop> testList;
+        try {
+            testList = testDatabase.getCrops();
+        } catch (IOException e) {
+            throw new RuntimeException(e);
+        }
+        Assertions.assertEquals(3, testList.size());
+
+        List<Long> plantIds = testList.stream().map(Crop::getPlantId).collect(Collectors.toList());
+        List<Long> expected = Arrays.asList(1l, 1l, 0l);
+        Assertions.assertEquals(expected, plantIds);
+    }
+
+    @Test
+    @DisplayName("Check whether single access works.")
+    void getCropById() {
+        Optional<Crop> testCrop;
+        try {
+            testCrop = testDatabase.getCropById(1);
+        } catch (IOException e) {
+            throw new RuntimeException(e);
+        }
+        assertTrue(testCrop.isPresent());
+        Assertions.assertEquals(1, testCrop.get().getPlantId());
+    }
+
+
+    @Test
+    @DisplayName("Check for a nonexisting crop.")
+    void getCropByIdMustFail() {
+        Optional<Crop> testCrop;
+        try {
+            testCrop = testDatabase.getCropById(99);
+        } catch (IOException e) {
+            throw new RuntimeException(e);
+        }
+        Assertions.assertFalse(testCrop.isPresent());
+    }
+
+    @Test
+    @DisplayName("Add new Crop.")
+    void addNewCrop() {
+        Crop crop = new Crop(2l, LocalDate.parse("22.02.2023", formatter));
+        try {
+            testDatabase.saveCrop(crop);
+            assertTrue(crop.getCropId().isPresent());
+            Optional<Crop> testCrop;
+            try {
+                testCrop = testDatabase.getCropById(crop.getCropId().get());
+            } catch (IOException e) {
+                throw new RuntimeException(e);
+            }
+            assertTrue(testCrop.isPresent());
+            Assertions.assertEquals(2l, testCrop.get().getPlantId());
+        } catch (IOException e) {
+            throw new RuntimeException(e);
+        }
+    }
+
+    @Test
+    @DisplayName("Remove crop")
+    void removeCrop(){
+        try {
+            Optional<Crop> crop = testDatabase.getCropById(2l);
+            Assertions.assertTrue(crop.isPresent());
+            testDatabase.removeCrop(crop.get());
+            crop = testDatabase.getCropById(2l);
+            Assertions.assertFalse(crop.isPresent());
+        } catch (IOException e) {
+            throw new RuntimeException(e);
+        }
+    }
+}
+

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 getPlantList() {
+    void getPlantListNotEmpty() {
         List<Plant> testList;
         try {
             testList = testDatabase.getPlantList(HardinessZone.ZONE_8A);
@@ -42,13 +42,47 @@ 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 getPlantById() {
+    void getPlantByIdAndZone() {
         Optional<Plant> testPlant;
         try {
-            testDatabase.getPlantList(HardinessZone.ZONE_8A);
-            testPlant = testDatabase.getPlantById(1);
+            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());
+
         } catch (IOException e) {
             throw new RuntimeException(e);
         } catch (HardinessZoneNotSetException e) {
@@ -63,8 +97,7 @@ public class JsonPlantDatabaseTest {
     void getPlantByIdMustFail() {
         Optional<Plant> testPlant;
         try {
-            testDatabase.getPlantList(HardinessZone.ZONE_8A);
-            testPlant = testDatabase.getPlantById(99);
+            testPlant = testDatabase.getPlantById(HardinessZone.ZONE_8A, 99);
         } catch (IOException e) {
             throw new RuntimeException(e);
         } catch (HardinessZoneNotSetException e) {