Skip to content

Commit 5cf7c53

Browse files
ramrock93ramrock93
authored
Refaktorerer annetData i DataBruktTilUtledning.java. (#367)
1 parent edaf9e5 commit 5cf7c53

File tree

2 files changed

+39
-80
lines changed

2 files changed

+39
-80
lines changed

‎soknad/src/main/java/no/nav/k9/søknad/ytelse/DataBruktTilUtledning.java‎

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,6 @@
88

99
import javax.validation.Valid;
1010
import javax.validation.constraints.AssertTrue;
11-
import java.util.HashMap;
12-
import java.util.Map;
1311

1412

1513
/**
@@ -64,10 +62,9 @@ public class DataBruktTilUtledning {
6462
* av klassens struktur og ikke er underlagt de samme valideringene eller sjekkene som andre felt.</p>
6563
*/
6664
@JsonProperty("annetData")
67-
private Map<String, Object> annetData;
65+
private String annetData;
6866

6967
public DataBruktTilUtledning() {
70-
this.annetData = new HashMap<>();
7168
}
7269

7370
public Boolean getHarForståttRettigheterOgPlikter() {
@@ -97,20 +94,15 @@ public DataBruktTilUtledning medSoknadDialogCommitSha(String soknadDialogCommitS
9794
return this;
9895
}
9996

100-
public Map<String, Object> getAnnetData() {
97+
public String getAnnetData() {
10198
return this.annetData;
10299
}
103100

104-
public DataBruktTilUtledning setAnnetData(Map<String, Object> annetData) {
101+
public DataBruktTilUtledning medAnnetData(String annetData) {
105102
this.annetData = annetData;
106103
return this;
107104
}
108105

109-
public DataBruktTilUtledning medData(String key, Object value) {
110-
this.annetData.put(key, value);
111-
return this;
112-
}
113-
114106
/**
115107
* Returnerer en JSON-representasjon av denne klassen eller toString fra superklassen.
116108
* Hvis serialisering feiler, returneres toString fra superklassen.

‎soknad/src/test/java/no/nav/k9/søknad/ytelse/DataBruktTilUtledningTest.java‎

Lines changed: 36 additions & 69 deletions
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,6 @@
66
import org.junit.jupiter.api.Test;
77
import org.skyscreamer.jsonassert.JSONAssert;
88

9-
import java.util.HashMap;
10-
import java.util.List;
11-
import java.util.Map;
12-
139
class DataBruktTilUtledningTest {
1410

1511
private DataBruktTilUtledning dataBruktTilUtledning;
@@ -21,90 +17,52 @@ void setUp() {
2117

2218
@Test
2319
public void testSerialiseringMedSetter() throws JSONException {
24-
Map<String, Object> data = new HashMap<>();
25-
data.put("string", "tekst");
26-
data.put("tall", 1000);
27-
data.put("boolean", true);
28-
data.put("objekt", Map.of("string", "tekst"));
29-
data.put("liste", List.of(Map.of("string", "tekst")));
30-
dataBruktTilUtledning
31-
.medHarBekreftetOpplysninger(true)
32-
.medHarForståttRettigheterOgPlikter(true)
33-
.medSoknadDialogCommitSha("1234567890")
34-
.setAnnetData(data);
3520

36-
String serialisertData = JsonUtils.toString(dataBruktTilUtledning);
37-
JSONAssert.assertEquals("""
21+
// language=JSON
22+
String annetData = """
3823
{
39-
"harBekreftetOpplysninger": true,
40-
"harForståttRettigheterOgPlikter": true,
41-
"soknadDialogCommitSha": "1234567890",
42-
"annetData" : {
43-
"string" : "tekst",
44-
"tall" : 1000,
45-
"boolean" : true,
46-
"objekt" : {
47-
"string" : "tekst"
48-
},
49-
"liste" : [ {
50-
"string" : "tekst"
51-
} ]
52-
}
24+
"string" : "tekst",
25+
"tall" : 1000,
26+
"boolean" : true,
27+
"objekt" : {
28+
"string" : "tekst"
29+
},
30+
"liste" : [ {
31+
"string" : "tekst"
32+
} ]
5333
}
54-
""", serialisertData, true);
55-
}
34+
""".translateEscapes();
35+
5636

57-
@Test
58-
public void testSerialiseringMedData() throws JSONException {
5937
dataBruktTilUtledning
6038
.medHarBekreftetOpplysninger(true)
6139
.medHarForståttRettigheterOgPlikter(true)
6240
.medSoknadDialogCommitSha("1234567890")
63-
.medData("string", "tekst")
64-
.medData("tall", 1000)
65-
.medData("boolean", true)
66-
.medData("objekt", Map.of("string", "tekst"))
67-
.medData("liste", List.of(Map.of("string", "tekst")));
41+
.medAnnetData(annetData);
6842

6943
String serialisertData = JsonUtils.toString(dataBruktTilUtledning);
7044
JSONAssert.assertEquals("""
7145
{
7246
"harBekreftetOpplysninger": true,
7347
"harForståttRettigheterOgPlikter": true,
7448
"soknadDialogCommitSha": "1234567890",
75-
"annetData" : {
76-
"string" : "tekst",
77-
"tall" : 1000,
78-
"boolean" : true,
79-
"objekt" : {
80-
"string" : "tekst"
81-
},
82-
"liste" : [ {
83-
"string" : "tekst"
84-
} ]
85-
}
49+
"annetData" : "{\\n \\"string\\" : \\"tekst\\",\\n \\"tall\\" : 1000,\\n \\"boolean\\" : true,\\n \\"objekt\\" : {\\n \\"string\\" : \\"tekst\\"\\n },\\n \\"liste\\" : [ {\\n \\"string\\" : \\"tekst\\"\\n } ]\\n}\\n",
50+
"harBekreftetOpplysninger" : true,
51+
"harForståttRettigheterOgPlikter" : true,
52+
"soknadDialogCommitSha" : "1234567890"
8653
}
8754
""", serialisertData, true);
8855
}
8956

9057
@Test
9158
public void testDeserialisering() throws JSONException {
59+
// language=JSON
9260
String jsonString = """
9361
{
9462
"harBekreftetOpplysninger": true,
9563
"harForståttRettigheterOgPlikter": true,
9664
"soknadDialogCommitSha": "1234567890",
97-
"annetData" : {
98-
"string" : "tekst",
99-
"tall" : 1000,
100-
"boolean" : true,
101-
"objekt" : {
102-
"string" : "tekst"
103-
},
104-
"liste" : [ {
105-
"string" : "tekst"
106-
} ]
107-
}
65+
"annetData" : "{\\n \\"string\\" : \\"tekst\\",\\n \\"tall\\" : 1000,\\n \\"boolean\\" : true,\\n \\"objekt\\" : {\\n \\"string\\" : \\"tekst\\"\\n },\\n \\"liste\\" : [ {\\n \\"string\\" : \\"tekst\\"\\n } ]\\n}\\n"
10866
}
10967
""";
11068

@@ -113,13 +71,22 @@ public void testDeserialisering() throws JSONException {
11371
.medHarBekreftetOpplysninger(true)
11472
.medHarForståttRettigheterOgPlikter(true)
11573
.medSoknadDialogCommitSha("1234567890")
116-
.setAnnetData(Map.of(
117-
"string", "tekst",
118-
"tall", 1000,
119-
"boolean", true,
120-
"objekt", Map.of("string", "tekst"),
121-
"liste", List.of(Map.of("string", "tekst"))
122-
));
74+
.medAnnetData(
75+
// language=JSON
76+
"""
77+
{
78+
"string" : "tekst",
79+
"tall" : 1000,
80+
"boolean" : true,
81+
"objekt" : {
82+
"string" : "tekst"
83+
},
84+
"liste" : [ {
85+
"string" : "tekst"
86+
} ]
87+
}
88+
"""
89+
);
12390
JSONAssert.assertEquals(forventetDataBruktTilUtledning.toString(), deserialisertData.toString(), true);
12491
}
12592
}

0 commit comments

Comments
 (0)