Skip to content

Commit

Permalink
KVP kontroll 20: Forbedret feilmeldingstekst
Browse files Browse the repository at this point in the history
  • Loading branch information
Hagemo, Jon Ole Johansen committed Mar 9, 2022
1 parent 7341abf commit 0bcf8e3
Show file tree
Hide file tree
Showing 2 changed files with 28 additions and 9 deletions.
32 changes: 24 additions & 8 deletions src/main/java/no/ssb/kostra/control/sosial/s11c_kvalifiseringsstonad/Main.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -240,13 +240,13 @@ public static boolean control11Bu18AntBu18(ErrorReport errorReport, Record r) {
, " "
, "Kontroll 11 Det bor barn under 18 år i husholdningen. Mangler antall barn."
, "Det er krysset av for at det bor barn under 18 år i husholdningen som mottaker eller ektefelle/samboer har forsørgerplikt for, "
+ "men det er ikke oppgitt hvor mange barn '(" + r.getFieldAsInteger(ANTBU18) + ")' som bor i husholdningen. "
+ "men det er ikke oppgitt hvor mange barn '(" + r.getFieldAsIntegerDefaultEquals0(ANTBU18) + ")' som bor i husholdningen. "
+ "Feltet er obligatorisk å fylle ut når det er oppgitt at det bor barn under 18 år i husholdningen."
, Constants.CRITICAL_ERROR
)
, r.getFieldAsString("BU18")
, List.of("1")
, r.getFieldAsInteger(ANTBU18)
, r.getFieldAsIntegerDefaultEquals0(ANTBU18)
, ">"
, 0
);
Expand All @@ -263,13 +263,13 @@ public static boolean control12AntBu18Bu18(ErrorReport errorReport, Record r) {
, r.getFieldAsString(FNR)
, " "
, "Kontroll 12 Det bor barn under 18 år i husholdningen."
, "Det er oppgitt " + r.getFieldAsInteger(ANTBU18) + " barn under 18 år som bor i husholdningen som "
, "Det er oppgitt " + r.getFieldAsIntegerDefaultEquals0(ANTBU18) + " barn under 18 år som bor i husholdningen som "
+ "mottaker eller ektefelle/samboer har forsørgerplikt for, men det er ikke "
+ "oppgitt at det bor barn i husholdningen. "
+ "Feltet er obligatorisk å fylle ut når det er oppgitt antall barn under 18 år som bor i husholdningen."
, Constants.CRITICAL_ERROR
)
, r.getFieldAsInteger(ANTBU18)
, r.getFieldAsIntegerDefaultEquals0(ANTBU18)
, ">"
, 0
, r.getFieldAsString("BU18")
Expand All @@ -288,10 +288,10 @@ public static boolean control13AntBu18(ErrorReport errorReport, Record r) {
, r.getFieldAsString(FNR)
, " "
, "Kontroll 13 Det bor barn under 18 år i husholdningen."
, "Antall barn (" + r.getFieldAsTrimmedString(ANTBU18) + ") under 18 år i husholdningen er 10 eller flere, er dette riktig?"
, "Antall barn (" + r.getFieldAsTrimmedString(ANTBU18) + ") under 18 år i husholdningen er 11 eller flere, er dette riktig?"
, Constants.CRITICAL_ERROR
)
, r.getFieldAsInteger(ANTBU18)
, r.getFieldAsIntegerDefaultEquals0(ANTBU18)
, "<="
, 10
);
Expand Down Expand Up @@ -386,6 +386,21 @@ public static boolean control20KvalifiseringsprogramIAnnenKommuneKommunenummer(E

final String KOMMNR_KVP_KOMM = "KOMMNR_KVP_KOMM";
final String KVP_KOMM = "KVP_KOMM";
final String kvp_komm = r.getFieldAsString(KVP_KOMM);
final String kommnr_kvp_komm = r.getFieldAsString(KOMMNR_KVP_KOMM);

final Code kvpKommCode = r.getFieldDefinitionByName(KVP_KOMM)
.getCodeList()
.stream()
.filter(item -> kvp_komm.equalsIgnoreCase(item.getCode()))
.findFirst()
.orElse(new Code(" ", "Uoppgitt"));

final List<String> kommnrKvpKommCodes = r.getFieldDefinitionByName(KOMMNR_KVP_KOMM)
.getCodeList()
.stream()
.map(Code::getCode)
.collect(Collectors.toList());

return ControlFelt1InneholderKodeFraKodelisteSaaFelt2InneholderKodeFraKodeliste.doControl(
errorReport
Expand All @@ -395,13 +410,14 @@ public static boolean control20KvalifiseringsprogramIAnnenKommuneKommunenummer(E
, r.getFieldAsString(FNR)
, " "
, "Kontroll 20 Kvalifiseringsprogram i annen kommune. Kommunenummer."
, "Deltakeren kommer fra kvalifiseringsprogram i annen kommune ('" + r.getFieldAsString(KOMMNR_KVP_KOMM) + "'), men kommunenummer mangler eller er ugyldig. Feltet er obligatorisk å fylle ut."
, String.format("Det er svart '%s' på om deltakeren kommer fra kvalifiseringsprogram i annen kommune "
+ ", men kommunenummer ('%s') mangler eller er ugyldig. Feltet er obligatorisk å fylle ut.", kvpKommCode, kommnr_kvp_komm)
, Constants.CRITICAL_ERROR
)
, r.getFieldAsString(KVP_KOMM)
, List.of("1")
, r.getFieldAsString(KOMMNR_KVP_KOMM)
, r.getFieldDefinitionByName(KOMMNR_KVP_KOMM).getCodeList().stream().map(Code::getCode).collect(Collectors.toList())
, kommnrKvpKommCodes
);
}

Expand Down
5 changes: 4 additions & 1 deletion .../java/no/ssb/kostra/control/sosial/s11c_kvalifiseringsstonad/KvalifiseringstonadTest.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -179,6 +179,7 @@ static Stream<TestRecordInputAndResult> control12AntBu18Bu18Provider() {

static Stream<TestRecordInputAndResult> control13AntBu18Provider() {
return Stream.of(
new TestRecordInputAndResult(arguments, new Record(Map.of("SAKSBEHANDLER", "Sara Sak", "PERSON_JOURNALNR", "123", "PERSON_FODSELSNR", "19096632188", "ANTBU18", " "), definitions), false, Constants.NO_ERROR),
new TestRecordInputAndResult(arguments, new Record(Map.of("SAKSBEHANDLER", "Sara Sak", "PERSON_JOURNALNR", "123", "PERSON_FODSELSNR", "19096632188", "ANTBU18", "1"), definitions), false, Constants.NO_ERROR),
new TestRecordInputAndResult(arguments, new Record(Map.of("SAKSBEHANDLER", "Sara Sak", "PERSON_JOURNALNR", "123", "PERSON_FODSELSNR", "12028012345", "ANTBU18", "10"), definitions), false, Constants.NO_ERROR),
new TestRecordInputAndResult(arguments, new Record(Map.of("SAKSBEHANDLER", "Sara Sak", "PERSON_JOURNALNR", "123", "PERSON_FODSELSNR", "12028012345", "ANTBU18", "11"), definitions), true, Constants.CRITICAL_ERROR)
Expand Down Expand Up @@ -219,7 +220,9 @@ static Stream<TestRecordInputAndResult> control19KvalifiseringsprogramIAnnenKomm

static Stream<TestRecordInputAndResult> control20KvalifiseringsprogramIAnnenKommuneKommunenummerProvider() {
return Stream.of(
new TestRecordInputAndResult(arguments, new Record(Map.of("SAKSBEHANDLER", "Sara Sak", "PERSON_JOURNALNR", "123", "PERSON_FODSELSNR", "19096632188", "KVP_KOMM", "1", "KOMMNR_KVP_KOMM", "0301"), definitions), false, Constants.NO_ERROR),
new TestRecordInputAndResult(argumOslo, new Record(Map.of("SAKSBEHANDLER", "Sara Sak", "PERSON_JOURNALNR", "123", "PERSON_FODSELSNR", "19096632188", "KVP_KOMM", " ", "KOMMNR_KVP_KOMM", "0301"), definitions), false, Constants.NO_ERROR),
new TestRecordInputAndResult(argumOslo, new Record(Map.of("SAKSBEHANDLER", "Sara Sak", "PERSON_JOURNALNR", "123", "PERSON_FODSELSNR", "19096632188", "KVP_KOMM", "1", "KOMMNR_KVP_KOMM", "0301"), definitions), false, Constants.NO_ERROR),
new TestRecordInputAndResult(argumOslo, new Record(Map.of("SAKSBEHANDLER", "Sara Sak", "PERSON_JOURNALNR", "123", "PERSON_FODSELSNR", "19096632188", "KVP_KOMM", "2", "KOMMNR_KVP_KOMM", " "), definitions), false, Constants.NO_ERROR),
new TestRecordInputAndResult(arguments, new Record(Map.of("SAKSBEHANDLER", "Sara Sak", "PERSON_JOURNALNR", "123", "PERSON_FODSELSNR", "12028012345", "KVP_KOMM", "1", "KOMMNR_KVP_KOMM", "0101"), definitions), true, Constants.CRITICAL_ERROR),
new TestRecordInputAndResult(arguments, new Record(Map.of("SAKSBEHANDLER", "Sara Sak", "PERSON_JOURNALNR", "123", "PERSON_FODSELSNR", "12028012345", "KVP_KOMM", "1", "KOMMNR_KVP_KOMM", " "), definitions), true, Constants.CRITICAL_ERROR),
new TestRecordInputAndResult(arguments, new Record(Map.of("SAKSBEHANDLER", "Sara Sak", "PERSON_JOURNALNR", "123", "PERSON_FODSELSNR", "12028012345", "KVP_KOMM", "2", "KOMMNR_KVP_KOMM", " "), definitions), false, Constants.NO_ERROR)
Expand Down

0 comments on commit 0bcf8e3

Please sign in to comment.