Skip to content

Commit 4d1f5fc

Browse files
authored
Add optional replacedByHref property to replacedSsin issue type (#68)
Fix #67.
1 parent 1fccb25 commit 4d1f5fc

File tree

3 files changed

+23
-0
lines changed

3 files changed

+23
-0
lines changed

belgif-rest-problem/src/main/java/io/github/belgif/rest/problem/api/InputValidationIssues.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -137,6 +137,10 @@ public static InputValidationIssue replacedSsin(InEnum in, String name, String s
137137
.additionalProperty("replacedBy", newSsin);
138138
}
139139

140+
public static InputValidationIssue replacedSsin(InEnum in, String name, String ssin, String newSsin, URI newHref) {
141+
return replacedSsin(in, name, ssin, newSsin).additionalProperty("replacedByHref", String.valueOf(newHref));
142+
}
143+
140144
public static InputValidationIssue canceledSsin(InEnum in, String name, String ssin) {
141145
return new InputValidationIssue(ISSUE_TYPE_CANCELED_SSIN, "SSIN has been canceled")
142146
.detail(String.format("SSIN %s has been canceled", ssin))

belgif-rest-problem/src/test/java/io/github/belgif/rest/problem/api/InputValidationIssuesTest.java

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
import static org.assertj.core.api.Assertions.*;
44

5+
import java.net.URI;
56
import java.time.LocalDate;
67
import java.time.OffsetDateTime;
78
import java.time.OffsetTime;
@@ -168,6 +169,23 @@ void replacedSsin() {
168169
assertThat(issue).extracting("href", "inputs").allMatch(this::isEmpty);
169170
}
170171

172+
@Test
173+
void replacedSsinWithReplacedByHref() {
174+
InputValidationIssue issue =
175+
InputValidationIssues.replacedSsin(InEnum.BODY, "ssin", "00000000196", "00000000295",
176+
URI.create("https://api.company.com/v1/employees?ssin=00000000295"));
177+
assertThat(issue.getType()).hasToString("urn:problem-type:cbss:input-validation:replacedSsin");
178+
assertThat(issue.getTitle()).isEqualTo("SSIN has been replaced, use new SSIN");
179+
assertThat(issue.getIn()).isEqualTo(InEnum.BODY);
180+
assertThat(issue.getName()).isEqualTo("ssin");
181+
assertThat(issue.getValue()).isEqualTo("00000000196");
182+
assertThat(issue.getDetail()).isEqualTo("SSIN 00000000196 has been replaced by 00000000295");
183+
assertThat(issue.getAdditionalProperties()).containsExactly(
184+
entry("replacedBy", "00000000295"),
185+
entry("replacedByHref", "https://api.company.com/v1/employees?ssin=00000000295"));
186+
assertThat(issue).extracting("href", "inputs").allMatch(this::isEmpty);
187+
}
188+
171189
@Test
172190
void canceledSsin() {
173191
InputValidationIssue issue =

src/main/asciidoc/index.adoc

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ The library consists of these modules:
3434

3535
* Removed deprecated InvalidParamProblem: use InputValidationProblem, which supports both the legacy invalidParams[] and the new issues[] structure
3636
* Replace specific `urn:problem-type:cbss:input-validation:unknownSsin` issue type by generic `urn:problem-type:cbss:input-validation:referencedResourceNotFound`
37+
* Added optional replacedByHref property to replacedSsin issue type
3738

3839
*belgif-rest-problem-validator:*
3940

0 commit comments

Comments
 (0)