Skip to content

Commit 2d95cc9

Browse files
committed
finpussing som ble glemt fra PR
1 parent 9ecf08e commit 2d95cc9

File tree

9 files changed

+10
-9
lines changed

9 files changed

+10
-9
lines changed

behandling/søknadsbehandling/domain/src/main/kotlin/behandling/søknadsbehandling/domain/KunneIkkeOppretteSøknadsbehandling.kt

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,6 @@ package behandling.søknadsbehandling.domain
33
sealed interface KunneIkkeOppretteSøknadsbehandling {
44
data object ErLukket : KunneIkkeOppretteSøknadsbehandling
55
data object ManglerOppgave : KunneIkkeOppretteSøknadsbehandling
6-
data object KanIkkeStarteNyBehandling : KunneIkkeOppretteSøknadsbehandling
6+
data object HarÅpenSøknadsbehandling : KunneIkkeOppretteSøknadsbehandling
77
data object FeilVedOpprettingAvOppgave : KunneIkkeOppretteSøknadsbehandling
88
}

behandling/søknadsbehandling/presentation/src/main/kotlin/behandling/søknadsbehandling/presentation/KunneIkkeOppretteSøknadsbehandlingEx.kt

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import no.nav.su.se.bakover.common.infrastructure.web.errorJson
88
fun KunneIkkeOppretteSøknadsbehandling.tilResultat() = when (this) {
99
KunneIkkeOppretteSøknadsbehandling.ErLukket -> Feilresponser.søknadErLukket
1010
KunneIkkeOppretteSøknadsbehandling.ManglerOppgave -> Feilresponser.søknadManglerOppgave
11-
KunneIkkeOppretteSøknadsbehandling.KanIkkeStarteNyBehandling -> HttpStatusCode.BadRequest.errorJson(
11+
KunneIkkeOppretteSøknadsbehandling.HarÅpenSøknadsbehandling -> HttpStatusCode.BadRequest.errorJson(
1212
"Det finnes en eksisterende åpen søknadsbehandling. Iverksett eller lukk denne først.",
1313
"finnes_åpen_søknadsbehandling",
1414
)

domain/src/main/kotlin/no/nav/su/se/bakover/domain/sak/Sak.kt

+1-1
Original file line numberDiff line numberDiff line change
@@ -404,7 +404,7 @@ data class Sak(
404404
// det skal kun finnes 1 søknadsbehandling åpen om gangen
405405
val søknadsbehandlingSomSkalLukkes =
406406
søknadensBehandlinger.singleOrNull { it.søknad.id == søknadId && it.erÅpen() }
407-
?: throw IllegalStateException("Fant ingen, en/flere ikke åpen behandling, eller flere åpne søknadsbehandlinger for søknad $søknadId. Antall behandlinger funnet ${søknadensBehandlinger.size}")
407+
?: throw IllegalStateException("Fant ingen, eller flere åpne søknadsbehandlinger for søknad $søknadId. Antall behandlinger funnet ${søknadensBehandlinger.size}")
408408

409409
// Finnes søknadsbehandling. Lukker søknadsbehandlingen, som i sin tur lukker søknaden.
410410
søknadsbehandlingSomSkalLukkes.lukkSøknadsbehandlingOgSøknad(

domain/src/main/kotlin/no/nav/su/se/bakover/domain/søknadsbehandling/IverksattSøknadsbehandling.kt

+1
Original file line numberDiff line numberDiff line change
@@ -168,6 +168,7 @@ sealed interface IverksattSøknadsbehandling : Søknadsbehandling, KanGenerereBr
168168
saksbehandler: NavIdentBruker.Saksbehandler,
169169
clock: Clock,
170170
): Either<KunneIkkeOppretteSøknadsbehandling, Søknadsbehandling> {
171+
// TODO - må sjekke stønadsperioden ikke overlapper. Dette blir stoppet ved iverksetting, men dem kan få tilbakemelding mye tidligere
171172
return erSøknadÅpen.whenever(
172173
isFalse = { KunneIkkeOppretteSøknadsbehandling.ErLukket.left() },
173174
isTrue = {

domain/src/main/kotlin/no/nav/su/se/bakover/domain/søknadsbehandling/opprett/OpprettNySøknadsbehandling.kt

+1-1
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ fun Sak.opprettNySøknadsbehandling(
4545
oppdaterOppgave: ((oppgaveId: OppgaveId, saksbehandler: NavIdentBruker.Saksbehandler) -> Either<Unit, OppgaveHttpKallResponse>)?,
4646
): Either<KunneIkkeOppretteSøknadsbehandling, Triple<Sak, VilkårsvurdertSøknadsbehandling.Uavklart, StatistikkEvent.Behandling.Søknad.Opprettet>> {
4747
if (harÅpenSøknadsbehandling()) {
48-
return KunneIkkeOppretteSøknadsbehandling.KanIkkeStarteNyBehandling.left()
48+
return KunneIkkeOppretteSøknadsbehandling.HarÅpenSøknadsbehandling.left()
4949
}
5050
val søknad = hentSøknad(søknadId).fold(
5151
ifLeft = { throw IllegalArgumentException("Fant ikke søknad $søknadId") },

domain/src/test/kotlin/no/nav/su/se/bakover/domain/søknadsbehandling/opprett/OpprettNySøknadsbehandlingTest.kt

+1-1
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ internal class OpprettNySøknadsbehandlingTest {
3838
søknadId = søknad.id,
3939
clock = fixedClock,
4040
saksbehandler = saksbehandler,
41-
) { _, _ -> nyOppgaveHttpKallResponse().right() } shouldBe KunneIkkeOppretteSøknadsbehandling.KanIkkeStarteNyBehandling.left()
41+
) { _, _ -> nyOppgaveHttpKallResponse().right() } shouldBe KunneIkkeOppretteSøknadsbehandling.HarÅpenSøknadsbehandling.left()
4242
}
4343

4444
@Test

service/src/test/kotlin/no/nav/su/se/bakover/service/søknad/AvslåSøknadManglendeDokumentasjonServiceImplTest.kt

+1-1
Original file line numberDiff line numberDiff line change
@@ -406,7 +406,7 @@ internal class AvslåSøknadManglendeDokumentasjonServiceImplTest {
406406
fritekstTilBrev = "fritekstTilBrev",
407407
),
408408
) shouldBe KunneIkkeAvslåSøknad.KunneIkkeOppretteSøknadsbehandling(
409-
KunneIkkeOppretteSøknadsbehandling.KanIkkeStarteNyBehandling,
409+
KunneIkkeOppretteSøknadsbehandling.HarÅpenSøknadsbehandling,
410410
).left()
411411

412412
verify(it.sakService).hentSakForSøknad(nySøknad.id)

service/src/test/kotlin/no/nav/su/se/bakover/service/søknad/lukk/LukkSøknadServiceImpl_lukkSøknadOgSøknadsbehandlingTest.kt

+2-2
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,7 @@ internal class LukkSøknadServiceImpl_lukkSøknadOgSøknadsbehandlingTest {
150150
).let { serviceAndMocks ->
151151
shouldThrow<IllegalStateException> {
152152
serviceAndMocks.lukkSøknad()
153-
}.message shouldBe "Fant ingen, en/flere ikke åpen behandling, eller flere åpne søknadsbehandlinger for søknad ${søknad.id}. Antall behandlinger funnet 1"
153+
}.message shouldBe "Fant ingen, eller flere åpne søknadsbehandlinger for søknad ${søknad.id}. Antall behandlinger funnet 1"
154154

155155
serviceAndMocks.verifyHentSakForSøknad()
156156
serviceAndMocks.verifyNoMoreInteractions()
@@ -173,7 +173,7 @@ internal class LukkSøknadServiceImpl_lukkSøknadOgSøknadsbehandlingTest {
173173
).let { serviceAndMocks ->
174174
shouldThrow<IllegalStateException> {
175175
serviceAndMocks.lukkSøknad()
176-
}.message shouldBe "Fant ingen, en/flere ikke åpen behandling, eller flere åpne søknadsbehandlinger for søknad ${søknad.id}. Antall behandlinger funnet 1"
176+
}.message shouldBe "Fant ingen, eller flere åpne søknadsbehandlinger for søknad ${søknad.id}. Antall behandlinger funnet 1"
177177

178178
serviceAndMocks.verifyHentSakForSøknad()
179179
serviceAndMocks.verifyNoMoreInteractions()

service/src/test/kotlin/no/nav/su/se/bakover/service/søknadsbehandling/SøknadsbehandlingServiceOpprettetTest.kt

+1-1
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,7 @@ internal class SøknadsbehandlingServiceOpprettetTest {
134134
sakId = nySøknad.sakId,
135135
saksbehandler = saksbehandler,
136136
),
137-
) shouldBe KunneIkkeOppretteSøknadsbehandling.KanIkkeStarteNyBehandling.left()
137+
) shouldBe KunneIkkeOppretteSøknadsbehandling.HarÅpenSøknadsbehandling.left()
138138
}
139139
}
140140

0 commit comments

Comments
 (0)