diff options
| author | Szymon Szukalski <szymon@skas.io> | 2024-07-25 20:36:11 +1000 |
|---|---|---|
| committer | Szymon Szukalski <szymon@skas.io> | 2024-07-25 20:36:11 +1000 |
| commit | aa9bdd514ab90d0da0391b879255a22c29450e9a (patch) | |
| tree | 9ddd1de0ab7e376ead06f55bdb32a6190d3647d5 /src/main/java/com/stileeducation/markr/dto/SummaryMarksDTO.java | |
| parent | ec81d98e90f9fdb4dd12138a365fbbbb3a8efa5f (diff) | |
Validate import payload and return create/update stats
- Add validation to /import payload
- Move import logic to service bean
- Track whether entities have been created or update
- Report number of created and updated entities as return value for the
import endpoint
- Add some test coverage to exercise the validators
Diffstat (limited to 'src/main/java/com/stileeducation/markr/dto/SummaryMarksDTO.java')
| -rw-r--r-- | src/main/java/com/stileeducation/markr/dto/SummaryMarksDTO.java | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/src/main/java/com/stileeducation/markr/dto/SummaryMarksDTO.java b/src/main/java/com/stileeducation/markr/dto/SummaryMarksDTO.java index a67d19c..788eea0 100644 --- a/src/main/java/com/stileeducation/markr/dto/SummaryMarksDTO.java +++ b/src/main/java/com/stileeducation/markr/dto/SummaryMarksDTO.java @@ -1,5 +1,7 @@ package com.stileeducation.markr.dto; +import jakarta.validation.constraints.Min; +import jakarta.validation.constraints.NotNull; import jakarta.xml.bind.annotation.XmlAttribute; import jakarta.xml.bind.annotation.XmlRootElement; @@ -7,8 +9,14 @@ import java.util.Objects; @XmlRootElement(name = "summary-marks") public class SummaryMarksDTO { - private int available; - private int obtained; + + @NotNull(message = "Available marks must not be null") + @Min(value = 0, message = "Available marks must be non-negative") + private Integer available; + + @NotNull(message = "Obtained marks must not be null") + @Min(value = 0, message = "Obtained marks must be non-negative") + private Integer obtained; @XmlAttribute(name = "available") public int getAvailable() { |
