Skip to content

Commit 60e13d6

Browse files
frode-carlsendependabot[bot]
frode-carlsen
and
dependabot[bot]
authored
Feature/junit 5 2 (#90)
* Bump assertj-core from 3.18.1 to 3.19.0 Bumps [assertj-core](https://github.com/assertj/assertj-core) from 3.18.1 to 3.19.0. - [Release notes](https://github.com/assertj/assertj-core/releases) - [Commits](assertj/assertj@assertj-core-3.18.1...assertj-core-3.19.0) Signed-off-by: dependabot[bot] <[email protected]> * fix: porter til junit jupiter * fix: porter api kall til junit5 / assertj Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
1 parent 3fc2170 commit 60e13d6

File tree

22 files changed

+268
-322
lines changed

22 files changed

+268
-322
lines changed

‎ettersendelse/pom.xml‎

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -16,21 +16,6 @@
1616
<groupId>no.nav.k9</groupId>
1717
<artifactId>soknad</artifactId>
1818
</dependency>
19-
<dependency>
20-
<groupId>junit</groupId>
21-
<artifactId>junit</artifactId>
22-
<scope>test</scope>
23-
</dependency>
24-
<dependency>
25-
<groupId>org.assertj</groupId>
26-
<artifactId>assertj-core</artifactId>
27-
<scope>test</scope>
28-
</dependency>
29-
<dependency>
30-
<groupId>org.skyscreamer</groupId>
31-
<artifactId>jsonassert</artifactId>
32-
<scope>test</scope>
33-
</dependency>
3419
</dependencies>
3520

3621

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

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

33
import org.json.JSONException;
4-
import org.junit.Test;
4+
import org.junit.jupiter.api.Test;
55

66
import static no.nav.k9.ettersendelse.Ettersendelse.SerDes.deserialize;
77
import static no.nav.k9.ettersendelse.Ettersendelse.SerDes.serialize;

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

Lines changed: 13 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,16 @@
11
package no.nav.k9.ettersendelse;
22

3-
import no.nav.k9.søknad.ValideringsFeil;
4-
import no.nav.k9.søknad.felles.*;
5-
import org.junit.Test;
3+
import static no.nav.k9.ettersendelse.TestUtils.jsonForKomplettEttersendelse;
4+
import static no.nav.k9.ettersendelse.TestUtils.komplettBuilder;
5+
import static org.assertj.core.api.Assertions.assertThat;
66

77
import java.util.Collections;
88
import java.util.List;
99

10-
import static no.nav.k9.ettersendelse.TestUtils.jsonForKomplettEttersendelse;
11-
import static no.nav.k9.ettersendelse.TestUtils.komplettBuilder;
12-
import static org.hamcrest.CoreMatchers.is;
13-
import static org.hamcrest.CoreMatchers.not;
14-
import static org.hamcrest.MatcherAssert.assertThat;
15-
import static org.junit.Assert.assertEquals;
10+
import org.junit.jupiter.api.Test;
11+
12+
import no.nav.k9.søknad.ValideringsFeil;
13+
import no.nav.k9.søknad.felles.Feil;
1614

1715
public class EttersendelseValidatorTest {
1816
private static final EttersendelseValidator validator = new EttersendelseValidator();
@@ -23,7 +21,7 @@ public void ettersendelseUtenNoeSatt() {
2321
Ettersendelse ettersendelse = Ettersendelse.SerDes.deserialize("{\"versjon\":\"0.0.1\"}");
2422
List<Feil> builderFeil = verifyHarFeil(builder);
2523
List<Feil> jsonFeil = verifyHarFeil(ettersendelse);
26-
assertThat(builderFeil, is(jsonFeil));
24+
assertThat(builderFeil).containsAll(jsonFeil);
2725
}
2826

2927
@Test
@@ -41,7 +39,7 @@ public void ettersendelseForOmsorgspenger() {
4139
@Test
4240
public void ettersendelseUtenYtelse() {
4341
Ettersendelse.Builder builder = komplettBuilder().ytelse(null);
44-
assertEquals(1, verifyHarFeil(builder).size());
42+
assertThat(verifyHarFeil(builder)).hasSize(1);
4543
}
4644

4745
@Test
@@ -60,22 +58,22 @@ private List<Feil> valider(Ettersendelse.Builder builder) {
6058
}
6159
private List<Feil> verifyHarFeil(Ettersendelse.Builder builder) {
6260
final List<Feil> feil = valider(builder);
63-
assertThat(feil, is(not(Collections.emptyList())));
61+
assertThat(feil).isNotEmpty();
6462
return feil;
6563
}
6664
private List<Feil> verifyHarFeil(Ettersendelse ettersendelse) {
6765
final List<Feil> feil = validator.valider(ettersendelse);
68-
assertThat(feil, is(not(Collections.emptyList())));
66+
assertThat(feil).isNotEmpty();
6967
return feil;
7068
}
7169

7270
private void verifyIngenFeil(Ettersendelse.Builder builder) {
7371
final List<Feil> feil = valider(builder);
74-
assertThat(feil, is(Collections.emptyList()));
72+
assertThat(feil).isEmpty();
7573
}
7674

7775
private void verifyIngenFeil(Ettersendelse ettersendelse) {
7876
final List<Feil> feil = validator.valider(ettersendelse);
79-
assertThat(feil, is(Collections.emptyList()));
77+
assertThat(feil).isEmpty();
8078
}
8179
}

‎pom.xml‎

Lines changed: 43 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@
2222
<sha1></sha1>
2323
<changelist>-SNAPSHOT</changelist>
2424
<java.version>11</java.version>
25+
<junit.version>5.7.0</junit.version>
2526

2627
<maven.compiler.source>${java.version}</maven.compiler.source>
2728
<maven.compiler.target>${java.version}</maven.compiler.target>
@@ -67,15 +68,22 @@
6768
<artifactId>soknad</artifactId>
6869
<version>${project.version}</version>
6970
</dependency>
70-
<dependency>
71-
<groupId>junit</groupId>
72-
<artifactId>junit</artifactId>
73-
<version>4.13.1</version>
74-
</dependency>
7571
<dependency>
7672
<groupId>org.assertj</groupId>
7773
<artifactId>assertj-core</artifactId>
78-
<version>3.18.1</version>
74+
<version>3.19.0</version>
75+
</dependency>
76+
<dependency>
77+
<groupId>org.junit.jupiter</groupId>
78+
<artifactId>junit-jupiter-params</artifactId>
79+
<version>${junit.version}</version>
80+
<scope>test</scope>
81+
</dependency>
82+
<dependency>
83+
<groupId>org.junit.jupiter</groupId>
84+
<artifactId>junit-jupiter-api</artifactId>
85+
<version>${junit.version}</version>
86+
<scope>test</scope>
7987
</dependency>
8088
<dependency>
8189
<groupId>org.skyscreamer</groupId>
@@ -84,6 +92,34 @@
8492
</dependency>
8593
</dependencies>
8694
</dependencyManagement>
95+
96+
<dependencies>
97+
<dependency>
98+
<groupId>org.junit.jupiter</groupId>
99+
<artifactId>junit-jupiter-api</artifactId>
100+
<scope>test</scope>
101+
</dependency>
102+
<dependency>
103+
<groupId>org.junit.jupiter</groupId>
104+
<artifactId>junit-jupiter-params</artifactId>
105+
<scope>test</scope>
106+
</dependency>
107+
<dependency>
108+
<groupId>org.assertj</groupId>
109+
<artifactId>assertj-core</artifactId>
110+
<scope>test</scope>
111+
</dependency>
112+
<dependency>
113+
<groupId>org.skyscreamer</groupId>
114+
<artifactId>jsonassert</artifactId>
115+
<scope>test</scope>
116+
</dependency>
117+
<dependency>
118+
<groupId>org.glassfish</groupId>
119+
<artifactId>jakarta.el</artifactId>
120+
<scope>test</scope>
121+
</dependency>
122+
</dependencies>
87123

88124
<build>
89125
<pluginManagement>
@@ -200,7 +236,7 @@
200236
<artifactId>maven-source-plugin</artifactId>
201237
<executions>
202238
<execution>
203-
<?m2e ignore?>
239+
<?m2e ignore?>
204240
<id>attach-sources</id>
205241
<goals>
206242
<goal>jar</goal>

‎soknad-frisinn/pom.xml‎

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -31,25 +31,5 @@
3131
<groupId>no.nav.k9</groupId>
3232
<artifactId>soknad</artifactId>
3333
</dependency>
34-
<dependency>
35-
<groupId>org.glassfish</groupId>
36-
<artifactId>jakarta.el</artifactId>
37-
<scope>test</scope>
38-
</dependency>
39-
<dependency>
40-
<groupId>junit</groupId>
41-
<artifactId>junit</artifactId>
42-
<scope>test</scope>
43-
</dependency>
44-
<dependency>
45-
<groupId>org.assertj</groupId>
46-
<artifactId>assertj-core</artifactId>
47-
<scope>test</scope>
48-
</dependency>
49-
<dependency>
50-
<groupId>org.skyscreamer</groupId>
51-
<artifactId>jsonassert</artifactId>
52-
<scope>test</scope>
53-
</dependency>
5434
</dependencies>
5535
</project>

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

Lines changed: 20 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package no.nav.k9.søknad.frisinn;
22

3+
import static org.assertj.core.api.Assertions.assertThat;
4+
35
import java.io.IOException;
46
import java.math.BigDecimal;
57
import java.nio.file.Files;
@@ -8,58 +10,56 @@
810
import java.time.ZonedDateTime;
911
import java.util.Map;
1012

11-
import no.nav.k9.søknad.felles.*;
13+
import org.junit.jupiter.api.Test;
14+
import org.skyscreamer.jsonassert.JSONAssert;
15+
16+
import no.nav.k9.søknad.felles.Versjon;
1217
import no.nav.k9.søknad.felles.personopplysninger.Søker;
1318
import no.nav.k9.søknad.felles.type.NorskIdentitetsnummer;
1419
import no.nav.k9.søknad.felles.type.Periode;
1520
import no.nav.k9.søknad.felles.type.Språk;
1621
import no.nav.k9.søknad.felles.type.SøknadId;
1722

18-
import org.junit.Test;
19-
import org.skyscreamer.jsonassert.JSONAssert;
20-
21-
import static org.junit.Assert.*;
22-
2323
public class FrisinnSøknadTest {
2424

2525
@Test
2626
public void bygg_serialiser_og_deserialiser_roundtrip_førstegangssøknad() throws Exception {
2727
var søknad = førstegangssøknad();
2828
var json = FrisinnSøknad.SerDes.serialize(søknad);
29-
assertNotNull(json);
29+
assertThat(json).isNotNull();
3030
var deserialisert = FrisinnSøknad.SerDes.deserialize(json);
31-
assertNotNull(deserialisert);
32-
assertFalse(deserialisert.getInntekter().getSelvstendig().getInntekterFør().isEmpty());
33-
assertNotNull(deserialisert.getInntekter().getSelvstendig().getRegnskapsførerNavn());
34-
assertNotNull(deserialisert.getInntekter().getSelvstendig().getRegnskapsførerTlf());
31+
assertThat(deserialisert).isNotNull();
32+
assertThat(deserialisert.getInntekter().getSelvstendig().getInntekterFør()).isNotEmpty();
33+
assertThat(deserialisert.getInntekter().getSelvstendig().getRegnskapsførerNavn()).isNotNull();
34+
assertThat(deserialisert.getInntekter().getSelvstendig().getRegnskapsførerTlf()).isNotNull();
3535
JSONAssert.assertEquals(json, FrisinnSøknad.SerDes.serialize(deserialisert), true);
3636
}
3737

3838
@Test
3939
public void bygg_serialiser_og_deserialiser_roundtrip_påfølgende_søknad() throws Exception {
4040
var søknad = påfølgendeSøknad();
4141
var json = FrisinnSøknad.SerDes.serialize(søknad);
42-
assertNotNull(json);
42+
assertThat(json).isNotNull();
4343
var deserialisert = FrisinnSøknad.SerDes.deserialize(json);
44-
assertNotNull(deserialisert);
45-
assertTrue(deserialisert.getInntekter().getSelvstendig().getInntekterFør().isEmpty());
46-
assertNull(deserialisert.getInntekter().getSelvstendig().getRegnskapsførerNavn());
47-
assertNull(deserialisert.getInntekter().getSelvstendig().getRegnskapsførerTlf());
44+
assertThat(deserialisert).isNotNull();
45+
assertThat(deserialisert.getInntekter().getSelvstendig().getInntekterFør()).isEmpty();
46+
assertThat(deserialisert.getInntekter().getSelvstendig().getRegnskapsførerNavn()).isNull();
47+
assertThat(deserialisert.getInntekter().getSelvstendig().getRegnskapsførerTlf()).isNull();
4848
JSONAssert.assertEquals(json, FrisinnSøknad.SerDes.serialize(deserialisert), true);
4949
}
5050

5151
@Test
5252
public void deserilisere_og_validere_1_0_0_søknad() {
5353
var søknad = FrisinnSøknad.builder().json(jsonFromFile("1.0.0")).build();
54-
assertEquals(Versjon.of("1.0.0"), søknad.getVersjon());
55-
assertNull(søknad.getInntekter().getArbeidstaker());
54+
assertThat(Versjon.of("1.0.0")).isEqualTo(søknad.getVersjon());
55+
assertThat(søknad.getInntekter().getArbeidstaker()).isNull();
5656
}
5757

5858
@Test
5959
public void deserilisere_og_validere_2_0_0_søknad() {
6060
var søknad = FrisinnSøknad.builder().json(jsonFromFile("2.0.0")).build();
61-
assertEquals(Versjon.of("2.0.0"), søknad.getVersjon());
62-
assertNotNull(søknad.getInntekter().getArbeidstaker());
61+
assertThat(Versjon.of("2.0.0")).isEqualTo(søknad.getVersjon());
62+
assertThat(søknad.getInntekter().getArbeidstaker()).isNotNull();
6363
}
6464

6565
private FrisinnSøknad byggSøknad(

‎soknad-omsorgspenger-utbetaling/pom.xml‎

Lines changed: 0 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -12,29 +12,10 @@
1212
<artifactId>soknad-omsorgspenger-utbetaling</artifactId>
1313

1414
<dependencies>
15-
<dependency>
16-
<groupId>org.glassfish</groupId>
17-
<artifactId>jakarta.el</artifactId>
18-
</dependency>
1915
<dependency>
2016
<groupId>no.nav.k9</groupId>
2117
<artifactId>soknad</artifactId>
2218
</dependency>
23-
<dependency>
24-
<groupId>junit</groupId>
25-
<artifactId>junit</artifactId>
26-
<scope>test</scope>
27-
</dependency>
28-
<dependency>
29-
<groupId>org.assertj</groupId>
30-
<artifactId>assertj-core</artifactId>
31-
<scope>test</scope>
32-
</dependency>
33-
<dependency>
34-
<groupId>org.skyscreamer</groupId>
35-
<artifactId>jsonassert</artifactId>
36-
<scope>test</scope>
37-
</dependency>
3819
</dependencies>
3920

4021
</project>

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

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

3+
import static org.assertj.core.api.Assertions.assertThat;
4+
35
import org.json.JSONException;
4-
import org.junit.Test;
6+
import org.junit.jupiter.api.Test;
57
import org.skyscreamer.jsonassert.JSONAssert;
68

79
import static no.nav.k9.søknad.omsorgspenger.utbetaling.arbeidstaker.TestUtils.*;
8-
import static org.junit.Assert.assertEquals;
910

1011
@SuppressWarnings("removal")
1112
public class OmsorgspengerUtbetalingSøknadTest {
@@ -27,6 +28,6 @@ public void reserialisering() throws JSONException {
2728
public void deserialisereSøknadUtenBarn() {
2829
var json = jsonForSøknadUtenBarn();
2930
var søknad = OmsorgspengerUtbetalingSøknad.SerDes.deserialize(json);
30-
assertEquals(0, søknad.fosterbarn.size());
31+
assertThat(søknad.fosterbarn).isEmpty();
3132
}
3233
}

0 commit comments

Comments
 (0)