Skip to content

Commit 7a61554

Browse files
committed
Re-init provider in tests
1 parent 16d645b commit 7a61554

File tree

3 files changed

+18
-18
lines changed

3 files changed

+18
-18
lines changed

felles/kontekst/src/main/java/no/nav/vedtak/sikkerhet/kontekst/AnsattGruppeProvider.java

+5-4
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,11 @@ public static synchronized AnsattGruppeProvider instance() {
6262
return INSTANCE;
6363
}
6464

65+
// Test usage only
66+
static synchronized void refresh() {
67+
INSTANCE = new AnsattGruppeProvider();
68+
}
69+
6570
public UUID getAnsattGruppeOid(AnsattGruppe gruppe) {
6671
return ansattGruppeOid.get(gruppe);
6772
}
@@ -82,10 +87,6 @@ public Set<AnsattGruppe> getAnsattGrupperFra(List<UUID> values) {
8287
return values.stream().map(this::getAnsattGruppeFra).filter(Objects::nonNull).collect(Collectors.toSet());
8388
}
8489

85-
static String getPropertyNavn(AnsattGruppe ansattGruppe) {
86-
return PROPERTY_NAME.get(ansattGruppe);
87-
}
88-
8990

9091
private static Map<AnsattGruppe, UUID> init() {
9192
Map<AnsattGruppe, UUID> resultat = new LinkedHashMap<>();

felles/kontekst/src/test/java/no/nav/vedtak/sikkerhet/kontekst/AnsattGruppeProviderEnvTest.java

+11-14
Original file line numberDiff line numberDiff line change
@@ -5,34 +5,31 @@
55
import java.util.List;
66
import java.util.Set;
77

8-
import org.junit.jupiter.api.AfterEach;
9-
import org.junit.jupiter.api.BeforeEach;
8+
import org.junit.jupiter.api.AfterAll;
9+
import org.junit.jupiter.api.BeforeAll;
1010
import org.junit.jupiter.api.Test;
1111

1212
/**
1313
* Tester at environment tar presedens over oids fra bundlete ressursfiler
1414
*/
1515
class AnsattGruppeProviderEnvTest {
1616

17-
public void setupAll() {
18-
System.setProperty(AnsattGruppeProvider.getPropertyNavn(AnsattGruppe.BESLUTTER), "6e31f9db-7e46-409d-809c-143d3863e4f6");
19-
System.setProperty(AnsattGruppeProvider.getPropertyNavn(AnsattGruppe.OVERSTYRER), "542269ee-090b-4017-bbcc-6791580290ac");
17+
@BeforeAll
18+
public static void setUp() {
19+
System.setProperty("gruppe.oid.beslutter", "6e31f9db-7e46-409d-809c-143d3863e4f6");
20+
System.setProperty("gruppe.oid.overstyrer", "542269ee-090b-4017-bbcc-6791580290ac");
2021
}
2122

22-
@BeforeEach
23-
public void setUp() {
24-
setupAll();
25-
}
26-
27-
@AfterEach
28-
public void teardown() {
29-
System.clearProperty(AnsattGruppeProvider.getPropertyNavn(AnsattGruppe.BESLUTTER));
30-
System.clearProperty(AnsattGruppeProvider.getPropertyNavn(AnsattGruppe.OVERSTYRER));
23+
@AfterAll
24+
public static void teardown() {
25+
System.clearProperty("gruppe.oid.beslutter");
26+
System.clearProperty("gruppe.oid.overstyrer");
3127
}
3228

3329

3430
@Test
3531
void testStringGroupsForLocal() {
32+
AnsattGruppeProvider.refresh();
3633
var provider = AnsattGruppeProvider.instance();
3734
assertThat(provider.getAnsattGruppeOid(AnsattGruppe.BESLUTTER)).isNotNull();
3835
assertThat(provider.getAnsattGruppeOid(AnsattGruppe.BESLUTTER).toString()).isEqualTo("6e31f9db-7e46-409d-809c-143d3863e4f6");

felles/kontekst/src/test/java/no/nav/vedtak/sikkerhet/kontekst/AnsattGruppeProviderTest.java

+2
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ class AnsattGruppeProviderTest {
1212

1313
@Test
1414
void testStringGroupsForLocal() {
15+
AnsattGruppeProvider.refresh();
1516
var provider = AnsattGruppeProvider.instance();
1617
assertThat(provider.getAnsattGruppeOid(AnsattGruppe.BESLUTTER)).isNotNull();
1718
assertThat(provider.getAnsattGruppeOid(AnsattGruppe.BESLUTTER).toString()).isEqualTo("803b1fd5-27a0-46a2-b1b3-7152f44128b4");
@@ -26,6 +27,7 @@ void testStringGroupsForLocal() {
2627

2728
@Test
2829
void testUuidGroupsForLocal() {
30+
AnsattGruppeProvider.refresh();
2931
var provider = AnsattGruppeProvider.instance();
3032
assertThat(provider.getAnsattGruppeOid(AnsattGruppe.BESLUTTER)).isNotNull();
3133
assertThat(provider.getAnsattGruppeOid(AnsattGruppe.BESLUTTER)).isEqualTo(UUID.fromString("803b1fd5-27a0-46a2-b1b3-7152f44128b4"));

0 commit comments

Comments
 (0)