PMT-15: Implement Tests for Responses

Reviewed-on: #10
Reviewed-by: SZUT-Ole <ole.kueck@hmmh.de>

.run/Test.run.xml

@@ -1,24 +1,12 @@
 <component name="ProjectRunConfigurationManager">
-  <configuration default="false" name="Test" type="GradleRunConfiguration" factoryName="Gradle">
-    <ExternalSystemSettings>
-      <option name="executionName" />
-      <option name="externalProjectPath" value="$PROJECT_DIR$" />
-      <option name="externalSystemIdString" value="GRADLE" />
-      <option name="scriptParameters" value="" />
-      <option name="taskDescriptions">
-        <list />
-      </option>
-      <option name="taskNames">
-        <list>
-          <option value="test" />
-        </list>
-      </option>
-      <option name="vmOptions" />
-    </ExternalSystemSettings>
-    <ExternalSystemDebugServerProcess>true</ExternalSystemDebugServerProcess>
-    <ExternalSystemReattachDebugProcess>true</ExternalSystemReattachDebugProcess>
-    <DebugAllEnabled>false</DebugAllEnabled>
-    <RunAsTest>false</RunAsTest>
-    <method v="2" />
+  <configuration default="false" name="Test" type="JUnit" factoryName="JUnit">
+    <module name="Project_Management_Tool.test" />
+    <option name="PACKAGE_NAME" value="" />
+    <option name="MAIN_CLASS_NAME" value="" />
+    <option name="METHOD_NAME" value="" />
+    <option name="TEST_OBJECT" value="package" />
+    <method v="2">
+      <option name="Make" enabled="true" />
+    </method>
   </configuration>
 </component>

api/pmt.yml

@@ -51,4 +51,30 @@ paths:
         401:
           $ref: "#/components/responses/UnAuthorized"
         500:
-          $ref: "#/components/responses/InternalError"
+          $ref: "#/components/responses/InternalError"
+
+  /project/{id}:
+    delete:
+      operationId: "deleteProject"
+      description: "Delete a specific Project"
+      parameters:
+        - in: path
+          name: id
+          schema:
+            type: integer
+            format: int64
+          required: true
+      responses:
+        204:
+          description: "Deletes a specific Project"
+        401:
+          $ref: "#/components/responses/UnAuthorized"
+        404:
+          description: "Project not found"
+          content:
+            text/plain:
+              schema:
+                type: string
+        500:
+          $ref: "#/components/responses/InternalError"
+

gen/config-pmt.json

@@ -4,7 +4,7 @@
     "invokerPackage": "de.hmmh.pmt",
     "java8": false,
     "java11": true,
-    "dateLibrary": "java11",
+    "dateLibrary": "java8-localdatetime",
     "library": "spring-boot3",
     "defaultInterfaces": false,
     "serializableModel": true

src/main/java/de/hmmh/pmt/ApiController.java

@@ -35,6 +35,16 @@ public class ApiController implements DefaultApi {
         return Optional.empty();
     }
 
+    @Override
+    public ResponseEntity<Void> deleteProject(Long id) {
+        if (!projectRepository.existsById(id)) {
+            return ResponseEntity.notFound().build();
+        }
+
+        projectRepository.deleteById(id);
+        return ResponseEntity.noContent().build();
+    }
+
     @Override
     public ResponseEntity<GetAllProjectsDTO> getAllProjects() {
         GetAllProjectsDTO response = new GetAllProjectsDTO();

src/test/java/de/hmmh/pmt/IntegrationTest.java

@@ -1,17 +1,22 @@
 package de.hmmh.pmt;
 
+import com.fasterxml.jackson.databind.ObjectMapper;
 import de.hmmh.pmt.db.Project;
 import de.hmmh.pmt.db.ProjectRepository;
+import de.hmmh.pmt.employee.api.EmployeeControllerApi;
+import de.hmmh.pmt.employee.api.QualificationControllerApi;
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.test.autoconfigure.web.servlet.AutoConfigureMockMvc;
 import org.springframework.boot.test.context.SpringBootTest;
+import org.springframework.boot.test.mock.mockito.MockBean;
 import org.springframework.test.context.ActiveProfiles;
 import org.springframework.test.web.servlet.MockMvc;
 
 import java.time.LocalDateTime;
-import java.util.List;
+import java.util.HashMap;
+import java.util.Map;
 
 @SpringBootTest
 @AutoConfigureMockMvc(addFilters = false)
@@ -23,8 +28,15 @@ public abstract class IntegrationTest {
     @Autowired
     protected MockMvc mvc;
     @Autowired
+    protected ObjectMapper objectMapper;
+    @Autowired
     protected ProjectRepository projectRepository;
 
+    @MockBean
+    protected EmployeeControllerApi mockEmployeeApi;
+    @MockBean
+    protected QualificationControllerApi mockQualificationApi;
+    
     @BeforeEach
     void setUp() {
         projectRepository.deleteAll();
@@ -35,70 +47,65 @@ public abstract class IntegrationTest {
         projectRepository.deleteAll();
     }
 
-    protected List<Project> createTestProjectData() {
-        Project project1 = new Project(
-                1L,
-                "Build a Dream Space Station",
-                "Launch a self-sustaining space habitat!",
-                42L,
-                101L,
-                LocalDateTime.of(2024, 3, 1, 10, 0),
-                LocalDateTime.of(2028, 6, 30, 18, 0),
-                LocalDateTime.of(2029, 12, 15, 16, 30)
-        );
-        Project project2 = new Project(
-                2L,
-                "Underwater Research Lab",
-                "Discover new marine species!",
-                73L,
-                202L,
-                LocalDateTime.of(2025, 5, 22, 8, 45),
-                LocalDateTime.of(2027, 11, 5, 17, 0),
-                LocalDateTime.of(2027, 10, 20, 14, 0)
-        );
-        Project project3 = new Project(
-                3L,
-                "AI-Powered Smart City",
-                "Create the world's most advanced smart city!",
-                89L,
-                303L,
-                LocalDateTime.of(2026, 9, 14, 12, 0),
-                LocalDateTime.of(2030, 4, 1, 9, 30),
-                LocalDateTime.of(2030, 5, 2, 15, 0)
-        );
-        Project project4 = new Project(
-                4L,
-                "Renewable Energy Revolution",
-                "Replace all fossil fuels with renewables!",
-                56L,
-                404L,
-                LocalDateTime.of(2023, 7, 19, 11, 30),
-                LocalDateTime.of(2029, 12, 31, 20, 0),
-                LocalDateTime.of(2029, 10, 5, 18, 45)
-        );
-        Project project5 = new Project(
-                5L,
-                "Virtual Reality Theme Park",
-                "Build a fully immersive VR theme park!",
-                99L,
-                505L,
-                LocalDateTime.of(2024, 2, 28, 9, 15),
-                LocalDateTime.of(2026, 9, 30, 17, 0),
-                LocalDateTime.of(2026, 8, 15, 13, 45)
-        );
+    protected Map<String, Project> createTestProjectData() {
+        Map<String, Project> projects = new HashMap<>();
+        
+        Project researchLabProject = new Project();
+        researchLabProject.setName("Underwater Research Lab");
+        researchLabProject.setGoal( "Discover new marine species!");
+        researchLabProject.setCustomerId(73L);
+        researchLabProject.setAdministratorId(202L);
+        researchLabProject.setStart(LocalDateTime.of(2025, 5, 22, 8, 45));
+        researchLabProject.setPlannedEnd(LocalDateTime.of(2026, 5, 22, 8, 45));
+        projects.put("research-lab", researchLabProject);
+        
+        Project spaceStationProject = new Project();
+        spaceStationProject.setName("International Space Station Expansion");
+        spaceStationProject.setGoal("Build new modules for extended research.");
+        spaceStationProject.setCustomerId(85L);
+        spaceStationProject.setAdministratorId(150L);
+        spaceStationProject.setStart(LocalDateTime.of(2024, 10, 15, 10, 0));
+        spaceStationProject.setPlannedEnd(LocalDateTime.of(2025, 12, 15, 10, 0));
+        projects.put("space-station", spaceStationProject);
+        
+        Project aiResearchProject = new Project();
+        aiResearchProject.setName("AI Human Interaction Study");
+        aiResearchProject.setGoal("Study AI interactions in healthcare.");
+        aiResearchProject.setCustomerId(63L);
+        aiResearchProject.setAdministratorId(180L);
+        aiResearchProject.setStart(LocalDateTime.of(2023, 11, 3, 9, 30));
+        aiResearchProject.setPlannedEnd(LocalDateTime.of(2024, 6, 3, 9, 30));
+        projects.put("ai-research", aiResearchProject);
+        
+        Project renewableEnergyProject = new Project();
+        renewableEnergyProject.setName("Renewable Energy Storage System");
+        renewableEnergyProject.setGoal("Develop new battery tech for solar power.");
+        renewableEnergyProject.setCustomerId(90L);
+        renewableEnergyProject.setAdministratorId(175L);
+        renewableEnergyProject.setStart(LocalDateTime.of(2024, 1, 10, 11, 0));
+        renewableEnergyProject.setPlannedEnd(LocalDateTime.of(2025, 1, 10, 11, 0));
+        projects.put("renewable-energy", renewableEnergyProject);
+        
+        Project climateChangeProject = new Project();
+        climateChangeProject.setName("Climate Change Impact Analysis");
+        climateChangeProject.setGoal("Study global warming effects on ecosystems.");
+        climateChangeProject.setCustomerId(72L);
+        climateChangeProject.setAdministratorId(190L);
+        climateChangeProject.setStart(LocalDateTime.of(2025, 3, 12, 8, 0));
+        climateChangeProject.setPlannedEnd(LocalDateTime.of(2026, 3, 12, 8, 0));
+        projects.put("climate change", climateChangeProject);
+            
+        
+        Project medicalResearchProject = new Project();
+        medicalResearchProject.setName("Cancer Treatment Innovation");
+        medicalResearchProject.setGoal("Develop new gene therapy techniques.");
+        medicalResearchProject.setCustomerId(95L);
+        medicalResearchProject.setAdministratorId(155L);
+        medicalResearchProject.setStart(LocalDateTime.of(2024, 8, 20, 9, 15));
+        medicalResearchProject.setPlannedEnd(LocalDateTime.of(2026, 8, 20, 9, 15));
+        projects.put("medical-research", medicalResearchProject);
 
-        projectRepository.save(project1);
-        projectRepository.save(project2);
-        projectRepository.save(project3);
-        projectRepository.save(project4);
-        projectRepository.save(project5);
-
-        return List.of(
-                project1,
-                project2,
-                project3,
-                project4,
-                project5
-        );
+        projectRepository.saveAllAndFlush(projects.values());
+        return projects;
     }
 }

src/test/java/de/hmmh/pmt/project/DeleteTest.java

@@ -0,0 +1,30 @@
+package de.hmmh.pmt.project;
+
+import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.delete;
+import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status;
+
+import java.util.List;
+
+import org.junit.jupiter.api.Test;
+
+import de.hmmh.pmt.IntegrationTest;
+import de.hmmh.pmt.db.Project;
+
+public class DeleteTest extends IntegrationTest {
+    @Test
+    void projectNotFound() throws Exception {
+        mvc
+            .perform(delete(baseUri + "/project/1"))
+            .andExpect(status().isNotFound())
+        ;
+    }
+
+    @Test
+    void deletedSuccessfully() throws Exception {
+        List<Project> allProjects = createTestProjectData();
+        mvc
+            .perform(delete(baseUri + "/project/" + allProjects.get(1).getId()))
+            .andExpect(status().isNoContent())
+        ;
+    }
+}

src/test/java/de/hmmh/pmt/project/GetAllTest.java

@@ -5,7 +5,7 @@ import de.hmmh.pmt.IntegrationTest;
 import de.hmmh.pmt.db.Project;
 import org.junit.jupiter.api.Test;
 
-import java.util.List;
+import java.util.Map;
 
 import static org.hamcrest.Matchers.empty;
 import static org.hamcrest.Matchers.hasSize;
@@ -25,7 +25,7 @@ public class GetAllTest extends IntegrationTest {
 
     @Test
     void multipleProjects() throws Exception {
-        List<Project> allProjects = createTestProjectData();
+        Map<String, Project> allProjects = createTestProjectData();
 
         mvc
             .perform(get(baseUri + "/project"))