chore: update

Author: chrootlogin

README.md

@@ -77,7 +77,15 @@ Die aktivierbare Engineering-Aufgabe läuft mit:
 ./mvnw -Pguided-gaps test
 ```
 
-Dieser Test schlägt am Anfang erwartbar fehl. Er wird erst grün, wenn die Migration datenbankseitige Regeln enthält.
+Dieser Test schlägt am Anfang erwartbar fehl. Er wird erst grün, wenn eine neue Migration datenbankseitige Regeln enthält.
+
+Die Lösung wird nicht in `V1__starter_ticket_schema.sql` eingetragen. `V1` bleibt der dokumentierte Startzustand. Studierende schreiben stattdessen eine neue Migration, zum Beispiel:
+
+```text
+src/main/resources/db/migration/V2__enforce_ticket_rules.sql
+```
+
+Diese `V2` ergänzt Pflichtfelder und einen Status-Constraint für die Ticket-Tabelle.
 
 ## Teststrategie
 

docs/guided-gaps.md

@@ -13,7 +13,13 @@ Analysiere `V1__starter_ticket_schema.sql`.
 
 ## Aufgabe 2: Datenbanklogik ergaenzen
 
-Verbessere die Migration so, dass PostgreSQL mindestens Folgendes garantiert:
+Erstelle eine neue Migration:
+
+```text
+src/main/resources/db/migration/V2__enforce_ticket_rules.sql
+```
+
+Aendere `V1__starter_ticket_schema.sql` nicht. `V1` bleibt der bewusst schwache Ausgangszustand. Die neue `V2` soll PostgreSQL mindestens Folgendes garantieren:
 
 - ein Ticket hat immer einen Titel
 - ein Ticket hat immer einen Status
@@ -33,4 +39,3 @@ Analysiere den Datenfluss fuer `GET /api/tickets?status=open`.
 - Welche Methode erzeugt die Datenbankabfrage?
 - Welche Rolle hat der Service?
 - Warum ist die Repository Method fuer diesen Fall noch lesbar?
-

pom.xml

@@ -19,6 +19,8 @@
 
     <properties>
         <java.version>24</java.version>
+        <mapstruct.version>1.6.3</mapstruct.version>
+        <lombok-mapstruct-binding.version>0.2.0</lombok-mapstruct-binding.version>
         <testcontainers.version>1.21.4</testcontainers.version>
     </properties>
 
@@ -39,6 +41,11 @@
             <groupId>org.springframework.boot</groupId>
             <artifactId>spring-boot-starter-flyway</artifactId>
         </dependency>
+        <dependency>
+            <groupId>org.mapstruct</groupId>
+            <artifactId>mapstruct</artifactId>
+            <version>${mapstruct.version}</version>
+        </dependency>
         <dependency>
             <groupId>org.springdoc</groupId>
             <artifactId>springdoc-openapi-starter-webmvc-ui</artifactId>
@@ -106,6 +113,16 @@
                             <groupId>org.projectlombok</groupId>
                             <artifactId>lombok</artifactId>
                         </path>
+                        <path>
+                            <groupId>org.projectlombok</groupId>
+                            <artifactId>lombok-mapstruct-binding</artifactId>
+                            <version>${lombok-mapstruct-binding.version}</version>
+                        </path>
+                        <path>
+                            <groupId>org.mapstruct</groupId>
+                            <artifactId>mapstruct-processor</artifactId>
+                            <version>${mapstruct.version}</version>
+                        </path>
                     </annotationProcessorPaths>
                 </configuration>
             </plugin>

src/main/java/ch/hftm/db2/ticketsystem/OpenApiConfig.java

@@ -9,7 +9,7 @@
         info = @Info(
                 title = "DB-2 Ticket System API",
                 version = "0.1.0",
-                description = "Spring-Boot-Starterprojekt für DB-2 mit bewusst offenen Guided Gaps in der Datenbanklogik."
+                description = "Spring-Boot-Projekt für DB-2"
         )
 )
 class OpenApiConfig {

src/main/java/ch/hftm/db2/ticketsystem/ticket/CreateTicketRequest.java

@@ -9,14 +9,14 @@
 @Data
 @NoArgsConstructor
 @AllArgsConstructor
-@Schema(description = "Eingabe für ein neues Starter-Ticket")
+@Schema(description = "Eingabe für ein neues Ticket")
 class CreateTicketRequest {
 
     @NotBlank
     @Schema(description = "Kurzer Tickettitel", example = "Datenbankverbindung prüfen")
     private String title;
 
     @NotBlank
-    @Schema(description = "Ticketstatus im Starterprojekt", example = "open")
+    @Schema(description = "Ticketstatus", example = "open")
     private String status;
 }

src/main/java/ch/hftm/db2/ticketsystem/ticket/TicketController.java

@@ -31,7 +31,7 @@ List<TicketResponse> findTickets(
 
     @PostMapping
     @ResponseStatus(HttpStatus.CREATED)
-    @Operation(summary = "Ticket erstellen", description = "Erstellt ein minimales Ticket im Starter-Schema app_starter.")
+    @Operation(summary = "Ticket erstellen", description = "Erstellt ein Ticket.")
     TicketResponse createTicket(@Valid @RequestBody CreateTicketRequest request) {
         return ticketService.createTicket(request);
     }

src/main/java/ch/hftm/db2/ticketsystem/ticket/TicketMapper.java

@@ -1,20 +1,14 @@
 package ch.hftm.db2.ticketsystem.ticket;
 
-import org.springframework.stereotype.Component;
+import org.mapstruct.Mapper;
+import org.mapstruct.Mapping;
 
-@Component
-class TicketMapper {
+@Mapper(componentModel = "spring")
+interface TicketMapper {
 
-    TicketResponse toResponse(TicketEntity entity) {
-        return new TicketResponse(
-                entity.getId(),
-                entity.getTitle(),
-                entity.getStatus(),
-                entity.getCreatedAt()
-        );
-    }
+    TicketResponse toResponse(TicketEntity entity);
 
-    TicketEntity toEntity(CreateTicketRequest request) {
-        return new TicketEntity(request.getTitle(), request.getStatus());
-    }
+    @Mapping(target = "id", ignore = true)
+    @Mapping(target = "createdAt", ignore = true)
+    TicketEntity toEntity(CreateTicketRequest request);
 }

src/test/java/ch/hftm/db2/ticketsystem/ticket/DatabaseEngineeringGapTest.java

@@ -5,24 +5,41 @@
 import java.io.IOException;
 import java.nio.file.Files;
 import java.nio.file.Path;
+import java.util.Locale;
 import org.junit.jupiter.api.Tag;
 import org.junit.jupiter.api.Test;
 
 @Tag("guided-gap")
 class DatabaseEngineeringGapTest {
 
     @Test
-    void migrationContainsDatabaseRulesForTickets() throws IOException {
-        String migration = Files.readString(Path.of("src/main/resources/db/migration/V1__starter_ticket_schema.sql"));
+    void ticketRulesAreAddedWithNewMigration() throws IOException {
+        Path v1Path = Path.of("src/main/resources/db/migration/V1__starter_ticket_schema.sql");
+        Path v2Path = Path.of("src/main/resources/db/migration/V2__enforce_ticket_rules.sql");
 
-        assertThat(migration)
-                .as("Tickets brauchen datenbankseitige Pflichtfelder.")
-                .contains("title TEXT NOT NULL")
-                .contains("status TEXT NOT NULL");
+        String v1 = Files.readString(v1Path);
+        assertThat(v1)
+                .as("V1 bleibt der bewusst schwache Ausgangszustand. Schreibe eine neue V2-Migration.")
+                .doesNotContain("title TEXT NOT NULL")
+                .doesNotContain("status TEXT NOT NULL")
+                .doesNotContain("CHECK (status IN");
 
-        assertThat(migration)
+        assertThat(v2Path)
+                .as("Lege eine neue Migration src/main/resources/db/migration/V2__enforce_ticket_rules.sql an.")
+                .exists();
+
+        String v2 = Files.readString(v2Path).replaceAll("\\s+", " ").toUpperCase(Locale.ROOT);
+
+        assertThat(v2)
+                .as("Tickets brauchen datenbankseitige Pflichtfelder in einer neuen Migration.")
+                .contains("ALTER TABLE APP_STARTER.TICKETS ALTER COLUMN TITLE SET NOT NULL")
+                .contains("ALTER TABLE APP_STARTER.TICKETS ALTER COLUMN STATUS SET NOT NULL");
+
+        assertThat(v2)
                 .as("Gueltige Ticket-Status sollen in PostgreSQL abgesichert werden.")
-                .contains("CHECK (status IN");
+                .contains("CHECK (STATUS IN")
+                .contains("'OPEN'")
+                .contains("'WAITING'")
+                .contains("'CLOSED'");
     }
 }
-

src/test/java/ch/hftm/db2/ticketsystem/ticket/TicketMapperTest.java

@@ -3,10 +3,11 @@
 import static org.assertj.core.api.Assertions.assertThat;
 
 import org.junit.jupiter.api.Test;
+import org.mapstruct.factory.Mappers;
 
 class TicketMapperTest {
 
-    private final TicketMapper mapper = new TicketMapper();
+    private final TicketMapper mapper = Mappers.getMapper(TicketMapper.class);
 
     @Test
     void mapsCreateRequestToEntity() {

src/test/java/ch/hftm/db2/ticketsystem/ticket/TicketServiceTest.java

@@ -8,11 +8,13 @@
 
 import java.util.List;
 import org.junit.jupiter.api.Test;
+import org.mapstruct.factory.Mappers;
 
 class TicketServiceTest {
 
     private final TicketRepository ticketRepository = mock(TicketRepository.class);
-    private final TicketService ticketService = new TicketService(ticketRepository, new TicketMapper());
+    private final TicketMapper ticketMapper = Mappers.getMapper(TicketMapper.class);
+    private final TicketService ticketService = new TicketService(ticketRepository, ticketMapper);
 
     @Test
     void usesStatusRepositoryMethodWhenStatusFilterIsPresent() {
@@ -36,4 +38,3 @@ void savesMappedTicketWhenCreatingTicket() {
         assertThat(ticket.status()).isEqualTo("open");
     }
 }
-