Skip to content

Commit 233ef62

Browse files
jolarsenjolarsen
committed
TFP-5966 fjerne ressurstype risikoklassifisering
1 parent f077a65 commit 233ef62

File tree

10 files changed

+36
-65
lines changed

10 files changed

+36
-65
lines changed

‎felles/abac/src/main/java/no/nav/vedtak/sikkerhet/abac/beskyttet/ResourceType.java

+1-3
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
import static no.nav.vedtak.sikkerhet.abac.policy.ForeldrepengerAttributter.RESOURCE_TYPE_FP_DRIFT;
66
import static no.nav.vedtak.sikkerhet.abac.policy.ForeldrepengerAttributter.RESOURCE_TYPE_FP_FAGSAK;
77
import static no.nav.vedtak.sikkerhet.abac.policy.ForeldrepengerAttributter.RESOURCE_TYPE_FP_OPPGAVESTYRING;
8-
import static no.nav.vedtak.sikkerhet.abac.policy.ForeldrepengerAttributter.RESOURCE_TYPE_FP_RISIKOKLASSIFISERING;
98
import static no.nav.vedtak.sikkerhet.abac.policy.ForeldrepengerAttributter.RESOURCE_TYPE_FP_UTTAKSPLAN;
109
import static no.nav.vedtak.sikkerhet.abac.policy.ForeldrepengerAttributter.RESOURCE_TYPE_FP_VENTEFRIST;
1110
import static no.nav.vedtak.sikkerhet.abac.policy.ForeldrepengerAttributter.RESOURCE_TYPE_INTERNAL_PIP;
@@ -21,8 +20,7 @@ public enum ResourceType {
2120
OPPGAVESTYRING_AVDELINGENHET(RESOURCE_TYPE_FP_AVDELINGENHET),
2221
OPPGAVESTYRING(RESOURCE_TYPE_FP_OPPGAVESTYRING),
2322
// OPPGAVEKØ(RESOURCE_TYPE_FP_OPPGAVEKØ), TODO: Vurder om skal brukes for å lese oppgaver for LOS. Nå brukes FAGSAK
24-
// Risk
25-
RISIKOKLASSIFISERING(RESOURCE_TYPE_FP_RISIKOKLASSIFISERING),
23+
2624
// Selvbetjening
2725
UTTAKSPLAN(RESOURCE_TYPE_FP_UTTAKSPLAN),
2826

‎felles/abac/src/main/java/no/nav/vedtak/sikkerhet/abac/policy/ForeldrepengerAttributter.java

-1
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,6 @@ private ForeldrepengerAttributter() {
5050
public static final String RESOURCE_TYPE_FP_AVDELINGENHET = "no.nav.abac.attributter.foreldrepenger.oppgavestyring.avdelingsenhet";
5151
// public static final String RESOURCE_TYPE_FP_OPPGAVEKØ = "no.nav.abac.attributter.foreldrepenger.oppgaveko"; TODO: Vurder om skal brukes for å lese oppgaver for LOS. Nå brukes FAGSAK. Evt bruk som dataAttributt.
5252
public static final String RESOURCE_TYPE_FP_OPPGAVESTYRING = "no.nav.abac.attributter.foreldrepenger.oppgavestyring";
53-
public static final String RESOURCE_TYPE_FP_RISIKOKLASSIFISERING = "no.nav.abac.attributter.foreldrepenger.risikoklassifisering";
5453
public static final String RESOURCE_TYPE_FP_UTTAKSPLAN = "no.nav.abac.attributter.resource.foreldrepenger.uttaksplan";
5554

5655
/**

‎felles/abac/src/main/java/no/nav/vedtak/sikkerhet/abac/policy/InternBrukerPolicies.java

-1
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,6 @@ public static Tilgangsvurdering vurderTilgang(BeskyttetRessursAttributter beskyt
3737
case ResourceType.VENTEFRIST -> ventefristPolicy(beskyttetRessursAttributter);
3838
case ResourceType.OPPGAVESTYRING_AVDELINGENHET -> avdelingEnhetPolicy(appRessursData);
3939
case ResourceType.OPPGAVESTYRING -> oppgavestyrerPolicy();
40-
case ResourceType.RISIKOKLASSIFISERING -> erVeilederEllerSaksbehandler(beskyttetRessursAttributter);
4140
default -> Tilgangsvurdering.avslåGenerell("InternBruker har ikke tilgang til ressurs " + beskyttetRessursAttributter.getResourceType());
4241
};
4342
}

‎felles/abac/src/test/java/no/nav/vedtak/sikkerhet/abac/PepImplTest.java

+5-6
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
package no.nav.vedtak.sikkerhet.abac;
22

3-
import static org.assertj.core.api.AssertionsForClassTypes.assertThat;
3+
import static org.assertj.core.api.Assertions.assertThat;
44
import static org.mockito.ArgumentMatchers.any;
55
import static org.mockito.Mockito.verify;
66
import static org.mockito.Mockito.verifyNoInteractions;
@@ -94,7 +94,7 @@ void skal_nekte_tilgang_til_saksbehandler_for_piptjeneste() {
9494
void skal_gi_tilgang_for_intern_azure_cc() {
9595
var token = new OpenIDToken(OpenIDProvider.AZUREAD, new TokenString("token"));
9696
when(tokenProvider.getUid()).thenReturn(LOCAL_APP);
97-
var attributter = lagBeskyttetRessursAttributterAzure(AvailabilityType.INTERNAL, token, LOCAL_APP, IdentType.Systemressurs);
97+
var attributter = lagBeskyttetRessursAttributterAzure(AvailabilityType.INTERNAL, token, IdentType.Systemressurs);
9898

9999
when(pdpRequestBuilder.lagAppRessursData(any())).thenReturn(AppRessursData.builder().build());
100100

@@ -108,7 +108,7 @@ void skal_gi_tilgang_for_intern_azure_cc() {
108108
void skal_gi_avslag_for_ekstern_azure_cc() {
109109
var token = new OpenIDToken(OpenIDProvider.AZUREAD, new TokenString("token"));
110110
when(tokenProvider.getUid()).thenReturn("vtp:annetnamespace:ukjentapplication");
111-
var attributter = lagBeskyttetRessursAttributterAzure(AvailabilityType.INTERNAL, token, "vtp:annetnamespace:ukjentapplication",
111+
var attributter = lagBeskyttetRessursAttributterAzure(AvailabilityType.INTERNAL, token,
112112
IdentType.Systemressurs);
113113

114114
when(pdpRequestBuilder.lagAppRessursData(any())).thenReturn(AppRessursData.builder().build());
@@ -123,7 +123,7 @@ void skal_gi_avslag_for_ekstern_azure_cc() {
123123
void skal_gi_avslag_for_godkjent_ekstern_azure_cc_men_i_feil_klusterklasse() {
124124
var token = new OpenIDToken(OpenIDProvider.AZUREAD, new TokenString("token"));
125125
when(tokenProvider.getUid()).thenReturn("dev-fss:annetnamespace:eksternapplication");
126-
var attributter = lagBeskyttetRessursAttributterAzure(AvailabilityType.INTERNAL, token, "vtp:annetnamespace:eksternapplication",
126+
var attributter = lagBeskyttetRessursAttributterAzure(AvailabilityType.INTERNAL, token,
127127
IdentType.Systemressurs);
128128

129129
when(pdpRequestBuilder.lagAppRessursData(any())).thenReturn(AppRessursData.builder().build());
@@ -139,7 +139,7 @@ void skal_gi_avslag_for_godkjent_ekstern_azure_cc_men_i_feil_klusterklasse() {
139139
void skal_gi_tilgang_for_godkjent_ekstern_azure_cc() {
140140
var token = new OpenIDToken(OpenIDProvider.AZUREAD, new TokenString("token"));
141141
when(tokenProvider.getUid()).thenReturn("vtp:annetnamespace:eksternapplication");
142-
var attributter = lagBeskyttetRessursAttributterAzure(AvailabilityType.ALL, token, "vtp:annetnamespace:eksternapplication",
142+
var attributter = lagBeskyttetRessursAttributterAzure(AvailabilityType.ALL, token,
143143
IdentType.Systemressurs);
144144

145145
when(pdpRequestBuilder.lagAppRessursData(any())).thenReturn(AppRessursData.builder().build());
@@ -194,7 +194,6 @@ private BeskyttetRessursAttributter lagBeskyttetRessursAttributterPip() {
194194

195195
private BeskyttetRessursAttributter lagBeskyttetRessursAttributterAzure(AvailabilityType availabilityType,
196196
OpenIDToken token,
197-
String brukerId,
198197
IdentType identType) {
199198
return BeskyttetRessursAttributter.builder()
200199
.medBrukerId(tokenProvider.getUid())

‎felles/abac/src/test/java/no/nav/vedtak/sikkerhet/abac/policy/EksternBrukerTest.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
package no.nav.vedtak.sikkerhet.abac.policy;
22

3-
import static org.assertj.core.api.AssertionsForClassTypes.assertThat;
3+
import static org.assertj.core.api.Assertions.assertThat;
44

55
import org.junit.jupiter.api.Test;
66
import org.junit.jupiter.api.extension.ExtendWith;

‎felles/abac/src/test/java/no/nav/vedtak/sikkerhet/abac/policy/InternBrukerFagsakTest.java

+9-9
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
package no.nav.vedtak.sikkerhet.abac.policy;
22

3-
import static org.assertj.core.api.AssertionsForClassTypes.assertThat;
3+
import static org.assertj.core.api.Assertions.assertThat;
44

55
import java.util.Set;
66
import java.util.UUID;
@@ -37,7 +37,7 @@ void tilgang_fagsak_read_veileder() {
3737

3838
var resultat = InternBrukerPolicies.vurderTilgang(attributter, ressurs);
3939
assertThat(resultat.fikkTilgang()).isTrue();
40-
assertThat(resultat.kreverGrupper().isEmpty()).isTrue();
40+
assertThat(resultat.kreverGrupper()).isEmpty();
4141
}
4242

4343
@Test
@@ -76,7 +76,7 @@ void tilgang_fagsak_read() {
7676

7777
var resultat = InternBrukerPolicies.vurderTilgang(attributter, ressurs);
7878
assertThat(resultat.fikkTilgang()).isTrue();
79-
assertThat(resultat.kreverGrupper().isEmpty()).isTrue();
79+
assertThat(resultat.kreverGrupper()).isEmpty();
8080
}
8181

8282
@Test
@@ -89,7 +89,7 @@ void tilgang_fagsak_read1() {
8989

9090
var resultat = InternBrukerPolicies.vurderTilgang(attributter, ressurs);
9191
assertThat(resultat.fikkTilgang()).isTrue();
92-
assertThat(resultat.kreverGrupper().isEmpty()).isTrue();
92+
assertThat(resultat.kreverGrupper()).isEmpty();
9393
}
9494

9595
@Test
@@ -101,7 +101,7 @@ void tilgang_fagsak_read2() {
101101

102102
var resultat = InternBrukerPolicies.vurderTilgang(attributter, ressurs);
103103
assertThat(resultat.fikkTilgang()).isTrue();
104-
assertThat(resultat.kreverGrupper().isEmpty()).isTrue();
104+
assertThat(resultat.kreverGrupper()).isEmpty();
105105
}
106106

107107
@Test
@@ -114,7 +114,7 @@ void tilgang_fagsak_create() {
114114

115115
var resultat = InternBrukerPolicies.vurderTilgang(attributter, ressurs);
116116
assertThat(resultat.fikkTilgang()).isTrue();
117-
assertThat(resultat.kreverGrupper().isEmpty()).isTrue();
117+
assertThat(resultat.kreverGrupper()).isEmpty();
118118
}
119119

120120
@Test
@@ -140,7 +140,7 @@ void tillat_fagsak_update_vanlig() {
140140

141141
var resultat = InternBrukerPolicies.vurderTilgang(attributter, ressurs);
142142
assertThat(resultat.fikkTilgang()).isTrue();
143-
assertThat(resultat.kreverGrupper().isEmpty()).isTrue();
143+
assertThat(resultat.kreverGrupper()).isEmpty();
144144
}
145145

146146
@Test
@@ -154,7 +154,7 @@ void tillat_fagsak_update_beslutter() {
154154

155155
var resultat = InternBrukerPolicies.vurderTilgang(attributter, ressurs);
156156
assertThat(resultat.fikkTilgang()).isTrue();
157-
assertThat(resultat.kreverGrupper().contains(AnsattGruppe.BESLUTTER)).isTrue();
157+
assertThat(resultat.kreverGrupper()).contains(AnsattGruppe.BESLUTTER);
158158
}
159159

160160
@Test
@@ -182,7 +182,7 @@ void tillat_fagsak_update_overstyr() {
182182

183183
var resultat = InternBrukerPolicies.vurderTilgang(attributter, ressurs);
184184
assertThat(resultat.fikkTilgang()).isTrue();
185-
assertThat(resultat.kreverGrupper().contains(AnsattGruppe.OVERSTYRER)).isTrue();
185+
assertThat(resultat.kreverGrupper()).contains(AnsattGruppe.OVERSTYRER);
186186
}
187187

188188
@Test

‎felles/abac/src/test/java/no/nav/vedtak/sikkerhet/abac/policy/InternBrukerTest.java

+16-40
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
package no.nav.vedtak.sikkerhet.abac.policy;
22

3-
import static org.assertj.core.api.AssertionsForClassTypes.assertThat;
3+
import static org.assertj.core.api.Assertions.assertThat;
44

55
import java.util.Set;
66
import java.util.UUID;
@@ -29,7 +29,7 @@ void tilgang_applikasjon_read() {
2929

3030
var resultat = InternBrukerPolicies.vurderTilgang(attributter, AppRessursData.builder().build());
3131
assertThat(resultat.fikkTilgang()).isTrue();
32-
assertThat(resultat.kreverGrupper().isEmpty()).isTrue();
32+
assertThat(resultat.kreverGrupper()).isEmpty();
3333
}
3434

3535
@Test
@@ -55,7 +55,7 @@ void tilgang_drift_read() {
5555

5656
var resultat = InternBrukerPolicies.vurderTilgang(attributter, AppRessursData.builder().build());
5757
assertThat(resultat.fikkTilgang()).isTrue();
58-
assertThat(resultat.kreverGrupper().contains(AnsattGruppe.DRIFT)).isTrue();
58+
assertThat(resultat.kreverGrupper()).contains(AnsattGruppe.DRIFT);
5959
}
6060

6161
@Test
@@ -64,7 +64,7 @@ void tilgang_drift_create() {
6464

6565
var resultat = InternBrukerPolicies.vurderTilgang(attributter, AppRessursData.builder().build());
6666
assertThat(resultat.fikkTilgang()).isTrue();
67-
assertThat(resultat.kreverGrupper().contains(AnsattGruppe.DRIFT)).isTrue();
67+
assertThat(resultat.kreverGrupper()).contains(AnsattGruppe.DRIFT);
6868
}
6969

7070
@Test
@@ -89,7 +89,7 @@ void tilgang_ventefrist_update_veileder() {
8989

9090
var resultat = InternBrukerPolicies.vurderTilgang(attributter, AppRessursData.builder().build());
9191
assertThat(resultat.fikkTilgang()).isTrue();
92-
assertThat(resultat.kreverGrupper().isEmpty()).isTrue();
92+
assertThat(resultat.kreverGrupper()).isEmpty();
9393
}
9494

9595
@Test
@@ -98,42 +98,17 @@ void tilgang_ventefrist_update_saksbehandler() {
9898

9999
var resultat = InternBrukerPolicies.vurderTilgang(attributter, AppRessursData.builder().build());
100100
assertThat(resultat.fikkTilgang()).isTrue();
101-
assertThat(resultat.kreverGrupper().isEmpty()).isTrue();
101+
assertThat(resultat.kreverGrupper()).isEmpty();
102102
}
103103

104-
@Test
105-
void tilgang_risiko_read() {
106-
var attributter = lagAttributter(AnsattGruppe.VEILEDER, ActionType.READ, ResourceType.RISIKOKLASSIFISERING);
107-
108-
var resultat = InternBrukerPolicies.vurderTilgang(attributter, AppRessursData.builder().build());
109-
assertThat(resultat.fikkTilgang()).isTrue();
110-
}
111-
112-
@Test
113-
void tilgang_risiko_create() {
114-
var attributter = lagAttributter(AnsattGruppe.SAKSBEHANDLER, ActionType.CREATE, ResourceType.RISIKOKLASSIFISERING);
115-
116-
var resultat = InternBrukerPolicies.vurderTilgang(attributter, AppRessursData.builder().build());
117-
assertThat(resultat.fikkTilgang()).isTrue();
118-
assertThat(resultat.kreverGrupper().isEmpty()).isTrue();
119-
}
120-
121-
@Test
122-
void tilgang_risiko_update() {
123-
var attributter = lagAttributter(AnsattGruppe.VEILEDER, ActionType.UPDATE, ResourceType.RISIKOKLASSIFISERING);
124-
125-
var resultat = InternBrukerPolicies.vurderTilgang(attributter, AppRessursData.builder().build());
126-
assertThat(resultat.fikkTilgang()).isTrue();
127-
assertThat(resultat.kreverGrupper().isEmpty()).isTrue();
128-
}
129104

130105
@Test
131106
void tilgang_oppgavestyrer_read() {
132107
var attributter = lagAttributter(AnsattGruppe.SAKSBEHANDLER, ActionType.READ, ResourceType.OPPGAVESTYRING);
133108

134109
var resultat = InternBrukerPolicies.vurderTilgang(attributter, AppRessursData.builder().build());
135110
assertThat(resultat.fikkTilgang()).isTrue();
136-
assertThat(resultat.kreverGrupper().contains(AnsattGruppe.OPPGAVESTYRER)).isTrue();
111+
assertThat(resultat.kreverGrupper()).contains(AnsattGruppe.OPPGAVESTYRER);
137112
}
138113

139114
@Test
@@ -142,7 +117,7 @@ void tilgang_oppgavestyrer_update() {
142117

143118
var resultat = InternBrukerPolicies.vurderTilgang(attributter, AppRessursData.builder().build());
144119
assertThat(resultat.fikkTilgang()).isTrue();
145-
assertThat(resultat.kreverGrupper().contains(AnsattGruppe.OPPGAVESTYRER)).isTrue();
120+
assertThat(resultat.kreverGrupper()).contains(AnsattGruppe.OPPGAVESTYRER);
146121
}
147122

148123

@@ -153,8 +128,8 @@ void tilgang_avdelingenhet_create() {
153128

154129
var resultat = InternBrukerPolicies.vurderTilgang(attributter, ressurs);
155130
assertThat(resultat.fikkTilgang()).isTrue();
156-
assertThat(resultat.kreverGrupper().contains(AnsattGruppe.OPPGAVESTYRER)).isTrue();
157-
assertThat(resultat.kreverGrupper().contains(AnsattGruppe.STRENGTFORTROLIG)).isFalse();
131+
assertThat(resultat.kreverGrupper()).contains(AnsattGruppe.OPPGAVESTYRER);
132+
assertThat(resultat.kreverGrupper()).doesNotContain(AnsattGruppe.STRENGTFORTROLIG);
158133
}
159134

160135
@Test
@@ -164,18 +139,19 @@ void tilgang_avdelingenhet_delete() {
164139

165140
var resultat = InternBrukerPolicies.vurderTilgang(attributter, ressurs);
166141
assertThat(resultat.fikkTilgang()).isTrue();
167-
assertThat(resultat.kreverGrupper().contains(AnsattGruppe.OPPGAVESTYRER)).isTrue();
168-
assertThat(resultat.kreverGrupper().contains(AnsattGruppe.STRENGTFORTROLIG)).isFalse();
142+
assertThat(resultat.kreverGrupper()).contains(AnsattGruppe.OPPGAVESTYRER);
143+
assertThat(resultat.kreverGrupper()).doesNotContain(AnsattGruppe.STRENGTFORTROLIG);
169144
}
170145

171146
@Test
172147
void tilgang_avdelingenhet_update_sf() {
173148
var attributter = lagAttributter(AnsattGruppe.SAKSBEHANDLER, ActionType.UPDATE, ResourceType.OPPGAVESTYRING_AVDELINGENHET);
174149
var ressurs = AppRessursData.builder().leggTilRessurs(ForeldrepengerDataKeys.AVDELING_ENHET, "2103").build();
175150

176-
var resultat = InternBrukerPolicies.vurderTilgang(attributter, ressurs);assertThat(resultat.fikkTilgang()).isTrue();
177-
assertThat(resultat.kreverGrupper().contains(AnsattGruppe.OPPGAVESTYRER)).isTrue();
178-
assertThat(resultat.kreverGrupper().contains(AnsattGruppe.STRENGTFORTROLIG)).isTrue();
151+
var resultat = InternBrukerPolicies.vurderTilgang(attributter, ressurs);
152+
assertThat(resultat.fikkTilgang()).isTrue();
153+
assertThat(resultat.kreverGrupper()).contains(AnsattGruppe.OPPGAVESTYRER);
154+
assertThat(resultat.kreverGrupper()).contains(AnsattGruppe.STRENGTFORTROLIG);
179155
}
180156

181157

‎felles/abac/src/test/java/no/nav/vedtak/sikkerhet/abac/policy/SystemressursTest.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
package no.nav.vedtak.sikkerhet.abac.policy;
22

3-
import static org.assertj.core.api.AssertionsForClassTypes.assertThat;
3+
import static org.assertj.core.api.Assertions.assertThat;
44

55
import org.junit.jupiter.api.AfterAll;
66
import org.junit.jupiter.api.BeforeAll;

‎felles/db/src/test/java/no/nav/vedtak/felles/jpa/converters/BooleanToStringConverterTest.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
package no.nav.vedtak.felles.jpa.converters;
22

3-
import static org.assertj.core.api.AssertionsForClassTypes.assertThat;
3+
import static org.assertj.core.api.Assertions.assertThat;
44

55
import org.junit.jupiter.api.Test;
66

@@ -35,7 +35,7 @@ void skal_konverte_FALSE_til_N() {
3535

3636
@Test
3737
void skal_konverte_null_boolean_til_null_streng() {
38-
assertThat(BOOLEAN_TO_STRING_CONVERTER.convertToDatabaseColumn(null)).isEqualTo(null);
38+
assertThat(BOOLEAN_TO_STRING_CONVERTER.convertToDatabaseColumn(null)).isNull();
3939
}
4040

4141
}

‎felles/db/src/test/java/no/nav/vedtak/felles/jpa/converters/PropertiesToStringConverterTest.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
package no.nav.vedtak.felles.jpa.converters;
22

3-
import static org.assertj.core.api.AssertionsForClassTypes.assertThat;
3+
import static org.assertj.core.api.Assertions.assertThat;
44
import static org.junit.jupiter.api.Assertions.assertNull;
55

66
import java.util.Properties;

0 commit comments

Comments
 (0)