Skip to content

Commit ab2d6a9

Browse files
tendestadtendestad
authored
Fjernet Buildere og konstruktører som har vært deprecated lenge (#324)
1 parent db10f9e commit ab2d6a9

File tree

21 files changed

+163
-739
lines changed

21 files changed

+163
-739
lines changed

‎ettersendelse/src/test/java/no/nav/k9/ettersendelse/TestUtils.java‎

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
11
package no.nav.k9.ettersendelse;
22

3-
import no.nav.k9.søknad.felles.personopplysninger.Søker;
4-
import no.nav.k9.søknad.felles.type.NorskIdentitetsnummer;
5-
import no.nav.k9.søknad.felles.type.SøknadId;
6-
73
import java.io.IOException;
84
import java.nio.file.Files;
95
import java.nio.file.Path;
106
import java.time.ZonedDateTime;
117

8+
import no.nav.k9.søknad.felles.personopplysninger.Søker;
9+
import no.nav.k9.søknad.felles.type.NorskIdentitetsnummer;
10+
import no.nav.k9.søknad.felles.type.SøknadId;
11+
1212
class TestUtils {
1313

1414
static String jsonForKomplettEttersendelse() {
@@ -22,9 +22,7 @@ static String jsonForKomplettEttersendelse() {
2222
static Ettersendelse.Builder komplettBuilder() {
2323
return Ettersendelse
2424
.builder()
25-
.søker(Søker.builder()
26-
.norskIdentitetsnummer(NorskIdentitetsnummer.of("11111111111"))
27-
.build())
25+
.søker(new Søker(NorskIdentitetsnummer.of("11111111111")))
2826
.mottattDato(ZonedDateTime.parse("2019-10-20T07:15:36.124Z"))
2927
.søknadId(SøknadId.of("123-123-123"))
3028
.ytelse(Ytelse.valueOf("PLEIEPENGER_SYKT_BARN"));

‎soknad-frisinn/src/test/java/no/nav/k9/søknad/frisinn/FrisinnSøknadTest.java‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -95,9 +95,7 @@ public class FrisinnSøknadTest {
9595
.søknadsperiode("2020-04-01/2020-04-30")
9696
.mottattDato(ZonedDateTime.parse("2020-04-20T07:15:36.124Z"))
9797
.språk(Språk.of("nb"))
98-
.søker(Søker.builder()
99-
.norskIdentitetsnummer(NorskIdentitetsnummer.of("12345678901"))
100-
.build())
98+
.søker(new Søker(NorskIdentitetsnummer.of("12345678901")))
10199
.inntekter(new Inntekter(frilanser, selvstendig, arbeidstaker))
102100
.build();
103101
return søknad;

‎soknad-omsorgspenger-utbetaling/src/test/java/no/nav/k9/søknad/omsorgspenger/utbetaling/arbeidstaker/OmsorgspengerUtbetalingSøknadValidatorTest.java‎

Lines changed: 13 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,20 @@
11
package no.nav.k9.søknad.omsorgspenger.utbetaling.arbeidstaker;
22

3-
import no.nav.k9.søknad.ValideringsFeil;
4-
import no.nav.k9.søknad.felles.Feil;
5-
import no.nav.k9.søknad.felles.personopplysninger.Barn;
6-
import no.nav.k9.søknad.felles.type.NorskIdentitetsnummer;
7-
import org.junit.jupiter.api.Test;
3+
import static no.nav.k9.søknad.omsorgspenger.utbetaling.arbeidstaker.TestUtils.jsonForKomplettSøknad;
4+
import static no.nav.k9.søknad.omsorgspenger.utbetaling.arbeidstaker.TestUtils.komplettBuilder;
5+
import static org.assertj.core.api.Assertions.assertThat;
86

97
import java.time.LocalDate;
108
import java.util.ArrayList;
119
import java.util.Collections;
1210
import java.util.List;
1311

14-
import static no.nav.k9.søknad.omsorgspenger.utbetaling.arbeidstaker.TestUtils.jsonForKomplettSøknad;
15-
import static no.nav.k9.søknad.omsorgspenger.utbetaling.arbeidstaker.TestUtils.komplettBuilder;
16-
import static org.assertj.core.api.Assertions.assertThat;
12+
import org.junit.jupiter.api.Test;
13+
14+
import no.nav.k9.søknad.ValideringsFeil;
15+
import no.nav.k9.søknad.felles.Feil;
16+
import no.nav.k9.søknad.felles.personopplysninger.Barn;
17+
import no.nav.k9.søknad.felles.type.NorskIdentitetsnummer;
1718

1819
@SuppressWarnings("removal")
1920
public class OmsorgspengerUtbetalingSøknadValidatorTest {
@@ -41,14 +42,14 @@ public class OmsorgspengerUtbetalingSøknadValidatorTest {
4142
var builder = komplettBuilder();
4243
builder.barn = new ArrayList<>();
4344
verifyIngenFeil(builder);
44-
builder.fosterbarn(Barn.builder().build());
45+
builder.fosterbarn(new Barn());
4546
assertThat(verifyHarFeil(builder)).hasSize(1);
4647
builder.barn = new ArrayList<>();
47-
builder.fosterbarn(Barn.builder().fødselsdato(LocalDate.now()).norskIdentitetsnummer(NorskIdentitetsnummer.of("123")).build());
48+
builder.fosterbarn(new Barn().medFødselsdato(LocalDate.now()).medNorskIdentitetsnummer(NorskIdentitetsnummer.of("123")));
4849
assertThat(verifyHarFeil(builder)).hasSize(1);
4950
builder.barn = new ArrayList<>();
50-
builder.fosterbarn(Barn.builder().fødselsdato(LocalDate.now()).build());
51-
builder.fosterbarn(Barn.builder().norskIdentitetsnummer(NorskIdentitetsnummer.of("123")).build());
51+
builder.fosterbarn(new Barn().medFødselsdato(LocalDate.now()));
52+
builder.fosterbarn(new Barn().medNorskIdentitetsnummer(NorskIdentitetsnummer.of("123")));
5253
verifyIngenFeil(builder);
5354
}
5455

‎soknad-omsorgspenger-utbetaling/src/test/java/no/nav/k9/søknad/omsorgspenger/utbetaling/arbeidstaker/TestUtils.java‎

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,16 @@
11
package no.nav.k9.søknad.omsorgspenger.utbetaling.arbeidstaker;
22

3-
import no.nav.k9.søknad.felles.personopplysninger.Barn;
4-
import no.nav.k9.søknad.felles.personopplysninger.Søker;
5-
import no.nav.k9.søknad.felles.type.NorskIdentitetsnummer;
6-
import no.nav.k9.søknad.felles.type.SøknadId;
7-
83
import java.io.IOException;
94
import java.nio.file.Files;
105
import java.nio.file.Path;
116
import java.time.LocalDate;
127
import java.time.ZonedDateTime;
138

9+
import no.nav.k9.søknad.felles.personopplysninger.Barn;
10+
import no.nav.k9.søknad.felles.personopplysninger.Søker;
11+
import no.nav.k9.søknad.felles.type.NorskIdentitetsnummer;
12+
import no.nav.k9.søknad.felles.type.SøknadId;
13+
1414
class TestUtils {
1515

1616
static String jsonForKomplettSøknad() {
@@ -41,11 +41,9 @@ class TestUtils {
4141
static OmsorgspengerUtbetalingSøknad.Builder komplettBuilder() {
4242
return OmsorgspengerUtbetalingSøknad
4343
.builder()
44-
.søker(Søker.builder()
45-
.norskIdentitetsnummer(NorskIdentitetsnummer.of("11111111111"))
46-
.build())
47-
.fosterbarn(Barn.builder().norskIdentitetsnummer(NorskIdentitetsnummer.of("11111111113")).build())
48-
.fosterbarn(Barn.builder().fødselsdato(LocalDate.parse("2000-01-01")).build())
44+
.søker(new Søker(NorskIdentitetsnummer.of("11111111111")))
45+
.fosterbarn(new Barn().medNorskIdentitetsnummer(NorskIdentitetsnummer.of("11111111113")))
46+
.fosterbarn(new Barn().medFødselsdato(LocalDate.parse("2000-01-01")))
4947
.mottattDato(ZonedDateTime.parse("2019-10-20T07:15:36.124Z"))
5048
.søknadId(SøknadId.of("123-123-123"));
5149
}

‎soknad-omsorgspenger-utbetaling/src/test/java/no/nav/k9/søknad/omsorgspenger/utbetaling/snf/OmsorgspengerUtbetalingSøknadValidatorTest.java‎

Lines changed: 59 additions & 61 deletions
Original file line numberDiff line numberDiff line change
@@ -4,13 +4,13 @@
44
import static no.nav.k9.søknad.omsorgspenger.utbetaling.snf.TestUtils.jsonForKomplettSøknad;
55
import static no.nav.k9.søknad.omsorgspenger.utbetaling.snf.TestUtils.komplettBuilder;
66
import static org.assertj.core.api.Assertions.assertThat;
7-
import static org.junit.jupiter.api.Assertions.assertThrows;
87

98
import java.math.BigDecimal;
109
import java.time.LocalDate;
1110
import java.util.ArrayList;
1211
import java.util.Collections;
1312
import java.util.List;
13+
import java.util.Map;
1414

1515
import org.assertj.core.api.Assertions;
1616
import org.junit.jupiter.api.Test;
@@ -52,14 +52,14 @@ public class OmsorgspengerUtbetalingSøknadValidatorTest {
5252
var builder = komplettBuilder();
5353
builder.barn = new ArrayList<>();
5454
verifyIngenFeil(builder);
55-
builder.fosterbarn(Barn.builder().build());
55+
builder.fosterbarn(new Barn());
5656
Assertions.assertThat(verifyHarFeil(builder)).hasSize(1);
5757
builder.barn = new ArrayList<>();
58-
builder.fosterbarn(Barn.builder().fødselsdato(LocalDate.now()).norskIdentitetsnummer(NorskIdentitetsnummer.of("123")).build());
58+
builder.fosterbarn(new Barn().medFødselsdato(LocalDate.now()).medNorskIdentitetsnummer(NorskIdentitetsnummer.of("123")));
5959
Assertions.assertThat(verifyHarFeil(builder)).hasSize(1);
6060
builder.barn = new ArrayList<>();
61-
builder.fosterbarn(Barn.builder().fødselsdato(LocalDate.now()).build());
62-
builder.fosterbarn(Barn.builder().norskIdentitetsnummer(NorskIdentitetsnummer.of("123")).build());
61+
builder.fosterbarn(new Barn().medFødselsdato(LocalDate.now()));
62+
builder.fosterbarn(new Barn().medNorskIdentitetsnummer(NorskIdentitetsnummer.of("123")));
6363
verifyIngenFeil(builder);
6464
}
6565

@@ -74,8 +74,7 @@ public class OmsorgspengerUtbetalingSøknadValidatorTest {
7474
public void selvstending_næringsdrivende_mangler_organisajonsnummer_virksomhetsnavn_og_perioder() {
7575
var builder = komplettBuilder();
7676
List<SelvstendigNæringsdrivende> selvstendingeVirksomheter = of(
77-
SelvstendigNæringsdrivende.builder()
78-
.build()
77+
new SelvstendigNæringsdrivende()
7978
);
8079
builder.selvstendigNæringsdrivende(selvstendingeVirksomheter);
8180
Assertions.assertThat(verifyHarFeil(builder)).hasSize(2);
@@ -85,34 +84,36 @@ public class OmsorgspengerUtbetalingSøknadValidatorTest {
8584
@Test
8685
public void selvstending_næringsdrivende_mangler_påkrevde_felter() {
8786
var builder = komplettBuilder();
88-
assertThrows(NullPointerException.class, () -> {
89-
List<SelvstendigNæringsdrivende> selvstendingeVirksomheter = of(
90-
SelvstendigNæringsdrivende.builder()
91-
.organisasjonsnummer(Organisasjonsnummer.of("816338352"))
92-
.periode(
93-
new Periode(LocalDate.now().minusMonths(2), LocalDate.now()),
94-
SelvstendigNæringsdrivende.SelvstendigNæringsdrivendePeriodeInfo.builder()
95-
.build()
96-
).build()
97-
);
98-
});
87+
List<SelvstendigNæringsdrivende> selvstendingeVirksomheter = of(
88+
new SelvstendigNæringsdrivende()
89+
.medOrganisasjonsnummer(Organisasjonsnummer.of("816338352"))
90+
.medPerioder(Map.of(
91+
new Periode(LocalDate.of(2022, 9, 8), LocalDate.of(2022, 11, 8)),
92+
new SelvstendigNæringsdrivende.SelvstendigNæringsdrivendePeriodeInfo())
93+
)
94+
);
95+
builder.selvstendigNæringsdrivende(selvstendingeVirksomheter);
96+
List<Feil> feil = verifyHarFeil(builder);
97+
Assertions.assertThat(feil.stream().map(f -> "[" + f.getFelt() + "] " + f.getFeilmelding()).toList()).containsOnly(
98+
"[selvstendigNæringsdrivende[0].perioder[2022-09-08/2022-11-08].virksomhetstyper] must not be null",
99+
"[selvstendigNæringsdrivende[0].perioder[2022-09-08/2022-11-08].virksomhetstyper] must not be empty"
100+
);
99101
}
100102

101103
@Test
102104
public void selvstending_næringsdrivende_har_varig_endring_uten_påkrevde_felter() {
103105
var builder = komplettBuilder();
104106
List<SelvstendigNæringsdrivende> selvstendingeVirksomheter = of(
105-
SelvstendigNæringsdrivende.builder()
106-
.organisasjonsnummer(Organisasjonsnummer.of("816338352"))
107-
.virksomhetNavn("ABC")
108-
.periode(
107+
new SelvstendigNæringsdrivende()
108+
.medOrganisasjonsnummer(Organisasjonsnummer.of("816338352"))
109+
.medVirksomhetNavn("ABC")
110+
.medPerioder(Map.of(
109111
new Periode(LocalDate.now().minusMonths(2), LocalDate.now()),
110-
SelvstendigNæringsdrivende.SelvstendigNæringsdrivendePeriodeInfo.builder()
111-
.bruttoInntekt(BigDecimal.valueOf(500_00))
112-
.virksomhetstyper(of(VirksomhetType.JORDBRUK_SKOGBRUK))
113-
.erVarigEndring(true)
114-
.build()
115-
).build()
112+
new SelvstendigNæringsdrivende.SelvstendigNæringsdrivendePeriodeInfo()
113+
.medBruttoInntekt(BigDecimal.valueOf(500_00))
114+
.medVirksomhetstyper(of(VirksomhetType.JORDBRUK_SKOGBRUK))
115+
.medErVarigEndring(true)
116+
))
116117
);
117118
builder.selvstendigNæringsdrivende(selvstendingeVirksomheter);
118119
Assertions.assertThat(verifyHarFeil(builder)).hasSize(2);
@@ -122,17 +123,16 @@ public class OmsorgspengerUtbetalingSøknadValidatorTest {
122123
public void selvstending_næringsdrivende_er_registrert_i_utlandet_uten_landkode() {
123124
var builder = komplettBuilder();
124125
List<SelvstendigNæringsdrivende> selvstendingeVirksomheter = of(
125-
SelvstendigNæringsdrivende.builder()
126-
.organisasjonsnummer(Organisasjonsnummer.of("816338352"))
127-
.virksomhetNavn("ABC")
128-
.periode(
126+
new SelvstendigNæringsdrivende()
127+
.medOrganisasjonsnummer(Organisasjonsnummer.of("816338352"))
128+
.medVirksomhetNavn("ABC")
129+
.medPerioder(Map.of(
129130
new Periode(LocalDate.now().minusMonths(2), LocalDate.now()),
130-
SelvstendigNæringsdrivende.SelvstendigNæringsdrivendePeriodeInfo.builder()
131-
.bruttoInntekt(BigDecimal.valueOf(500_000))
132-
.virksomhetstyper(of(VirksomhetType.JORDBRUK_SKOGBRUK))
133-
.registrertIUtlandet(true)
134-
.build()
135-
).build()
131+
new SelvstendigNæringsdrivende.SelvstendigNæringsdrivendePeriodeInfo()
132+
.medBruttoInntekt(BigDecimal.valueOf(500_000))
133+
.medVirksomhetstyper(of(VirksomhetType.JORDBRUK_SKOGBRUK))
134+
.medRegistrertIUtlandet(true))
135+
)
136136
);
137137
builder.selvstendigNæringsdrivende(selvstendingeVirksomheter);
138138
Assertions.assertThat(verifyHarFeil(builder)).hasSize(1);
@@ -142,18 +142,17 @@ public class OmsorgspengerUtbetalingSøknadValidatorTest {
142142
public void selvstending_næringsdrivende_er_registrert_i_utlandet_med_lankode_blankt() {
143143
var builder = komplettBuilder();
144144
List<SelvstendigNæringsdrivende> selvstendingeVirksomheter = of(
145-
SelvstendigNæringsdrivende.builder()
146-
.organisasjonsnummer(Organisasjonsnummer.of("816338352"))
147-
.virksomhetNavn("ABC")
148-
.periode(
145+
new SelvstendigNæringsdrivende()
146+
.medOrganisasjonsnummer(Organisasjonsnummer.of("816338352"))
147+
.medVirksomhetNavn("ABC")
148+
.medPerioder(Map.of(
149149
new Periode(LocalDate.now().minusMonths(2), LocalDate.now()),
150-
SelvstendigNæringsdrivende.SelvstendigNæringsdrivendePeriodeInfo.builder()
151-
.bruttoInntekt(BigDecimal.valueOf(500_000))
152-
.virksomhetstyper(of(VirksomhetType.JORDBRUK_SKOGBRUK))
153-
.registrertIUtlandet(true)
154-
.landkode(Landkode.of(""))
155-
.build()
156-
).build()
150+
new SelvstendigNæringsdrivende.SelvstendigNæringsdrivendePeriodeInfo()
151+
.medBruttoInntekt(BigDecimal.valueOf(500_000))
152+
.medVirksomhetstyper(of(VirksomhetType.JORDBRUK_SKOGBRUK))
153+
.medRegistrertIUtlandet(true)
154+
.medLandkode(Landkode.of(""))
155+
))
157156
);
158157
builder.selvstendigNæringsdrivende(selvstendingeVirksomheter);
159158
Assertions.assertThat(verifyHarFeil(builder)).hasSize(1);
@@ -163,18 +162,17 @@ public class OmsorgspengerUtbetalingSøknadValidatorTest {
163162
public void selvstending_næringsdrivende_er_registrert_i_utlandet_med_ugyldig_landkode() {
164163
var builder = komplettBuilder();
165164
List<SelvstendigNæringsdrivende> selvstendingeVirksomheter = of(
166-
SelvstendigNæringsdrivende.builder()
167-
.organisasjonsnummer(Organisasjonsnummer.of("816338352"))
168-
.virksomhetNavn("ABC")
169-
.periode(
165+
new SelvstendigNæringsdrivende()
166+
.medOrganisasjonsnummer(Organisasjonsnummer.of("816338352"))
167+
.medVirksomhetNavn("ABC")
168+
.medPerioder(Map.of(
170169
new Periode(LocalDate.now().minusMonths(2), LocalDate.now()),
171-
SelvstendigNæringsdrivende.SelvstendigNæringsdrivendePeriodeInfo.builder()
172-
.bruttoInntekt(BigDecimal.valueOf(500_000))
173-
.virksomhetstyper(of(VirksomhetType.JORDBRUK_SKOGBRUK))
174-
.registrertIUtlandet(true)
175-
.landkode(Landkode.of("UKJENT"))
176-
.build()
177-
).build()
170+
new SelvstendigNæringsdrivende.SelvstendigNæringsdrivendePeriodeInfo()
171+
.medBruttoInntekt(BigDecimal.valueOf(500_000))
172+
.medVirksomhetstyper(of(VirksomhetType.JORDBRUK_SKOGBRUK))
173+
.medRegistrertIUtlandet(true)
174+
.medLandkode(Landkode.of("UKJENT"))
175+
))
178176
);
179177
builder.selvstendigNæringsdrivende(selvstendingeVirksomheter);
180178
Assertions.assertThat(verifyHarFeil(builder)).hasSize(1);
@@ -183,7 +181,7 @@ public class OmsorgspengerUtbetalingSøknadValidatorTest {
183181
@Test
184182
public void frilanser_mangler_startdato() {
185183
var builder = komplettBuilder();
186-
builder.frilanser(Frilanser.builder().build());
184+
builder.frilanser(new Frilanser());
187185
Assertions.assertThat(verifyHarFeil(builder)).hasSize(1);
188186
}
189187

0 commit comments

Comments
 (0)