From 8ea94ce3d555b158ec5cedf329c7df778f84717a Mon Sep 17 00:00:00 2001 From: amiraliw Date: Fri, 6 Dec 2024 09:19:30 +0100 Subject: [PATCH 01/11] =?UTF-8?q?first=20commit.=20bare=20for=20=C3=A5=20t?= =?UTF-8?q?este?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../no/nav/familie/tilbake/oppgave/OppdaterOppgaveTask.kt | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterOppgaveTask.kt b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterOppgaveTask.kt index dd4f7f298..1eeeaee1e 100644 --- a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterOppgaveTask.kt +++ b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterOppgaveTask.kt @@ -60,6 +60,10 @@ class OppdaterOppgaveTask( beskrivelse + System.lineSeparator() + (oppgave?.beskrivelse ?: "") val prioritet = oppgavePrioritetService.utledOppgaveprioritet(behandlingId, oppgave) + log.info("=====>>>>> TEST saksbehandler <<<====== {}", saksbehandler) + log.info("=====>>>>> TEST oppgave.tildeltEnhetsnr <<<====== {}", oppgave.tildeltEnhetsnr) + log.info("=====>>>>> TEST oppgave.tilordnetRessurs <<<====== {}", oppgave.tilordnetRessurs) + var patchetOppgave = oppgave.copy( From d14b5ccc7417fb39c7c134c9f16f5e9b9bd81a75 Mon Sep 17 00:00:00 2001 From: amiraliw Date: Fri, 6 Dec 2024 12:35:04 +0100 Subject: [PATCH 02/11] la til logging for testing --- .../tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt index 6f1ab6359..e867e5682 100644 --- a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt +++ b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt @@ -30,7 +30,10 @@ class OppdaterAnsvarligSaksbehandlerTask( val behandling = behandlingRepository.findByIdOrThrow(behandlingId) val oppgave = oppgaveService.finnOppgaveForBehandlingUtenOppgaveType(behandlingId) val prioritet = oppgavePrioritetService.utledOppgaveprioritet(behandlingId, oppgave) - + log.info("=====>>>> oppgave.tilordnetRessurs: {}", oppgave.tilordnetRessurs) + log.info("=====>>>> behandling.ansvarligSaksbehandler: {}", behandling.ansvarligSaksbehandler) + log.info("=====>>>> oppgave.prioritet: {}", oppgave.prioritet) + log.info("=====>>>> prioritet: {}", prioritet) if (oppgave.tilordnetRessurs != behandling.ansvarligSaksbehandler || oppgave.prioritet != prioritet) { oppgaveService.patchOppgave(oppgave.copy(tilordnetRessurs = behandling.ansvarligSaksbehandler, prioritet = prioritet)) } From 2399eba4c578561f5056d61170819d38a35d6da6 Mon Sep 17 00:00:00 2001 From: amiraliw Date: Tue, 10 Dec 2024 09:49:59 +0100 Subject: [PATCH 03/11] =?UTF-8?q?la=20til=20en=20try-catch=20for=20en=20en?= =?UTF-8?q?kelt=20l=C3=B8sning=20for=20n=C3=A5?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt index e867e5682..734102142 100644 --- a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt +++ b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt @@ -37,6 +37,12 @@ class OppdaterAnsvarligSaksbehandlerTask( if (oppgave.tilordnetRessurs != behandling.ansvarligSaksbehandler || oppgave.prioritet != prioritet) { oppgaveService.patchOppgave(oppgave.copy(tilordnetRessurs = behandling.ansvarligSaksbehandler, prioritet = prioritet)) } + try{ + oppgaveService.patchOppgave(oppgave.copy(tilordnetRessurs = behandling.ansvarligSaksbehandler, prioritet = prioritet)) + }catch (e:Exception) { + log.info("Eneht på oppgaven og enhet hos saksbehandleren er IKKE det samme!") + oppgaveService.patchOppgave(oppgave.copy(prioritet = prioritet)) + } } companion object { From 4c01172ed9fbd2b426a7ef15dffe384afe55a5c2 Mon Sep 17 00:00:00 2001 From: amiraliw Date: Tue, 10 Dec 2024 10:01:29 +0100 Subject: [PATCH 04/11] =?UTF-8?q?Glemte=20=C3=A5=20fjerne=20gammel=20kode.?= =?UTF-8?q?=20N=C3=A5=20er=20det=20gjort?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt index 734102142..8cb719ee7 100644 --- a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt +++ b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt @@ -34,9 +34,9 @@ class OppdaterAnsvarligSaksbehandlerTask( log.info("=====>>>> behandling.ansvarligSaksbehandler: {}", behandling.ansvarligSaksbehandler) log.info("=====>>>> oppgave.prioritet: {}", oppgave.prioritet) log.info("=====>>>> prioritet: {}", prioritet) - if (oppgave.tilordnetRessurs != behandling.ansvarligSaksbehandler || oppgave.prioritet != prioritet) { + /* if (oppgave.tilordnetRessurs != behandling.ansvarligSaksbehandler || oppgave.prioritet != prioritet) { oppgaveService.patchOppgave(oppgave.copy(tilordnetRessurs = behandling.ansvarligSaksbehandler, prioritet = prioritet)) - } + }*/ try{ oppgaveService.patchOppgave(oppgave.copy(tilordnetRessurs = behandling.ansvarligSaksbehandler, prioritet = prioritet)) }catch (e:Exception) { From c65d07ce51b2df537515c53b85acf3ab428b7e06 Mon Sep 17 00:00:00 2001 From: amiraliw Date: Tue, 10 Dec 2024 16:38:54 +0100 Subject: [PATCH 05/11] La til tester --- .../OppdaterAnsvarligSaksbehandlerTask.kt | 21 +++++------- .../tilbake/oppgave/OppdaterOppgaveTask.kt | 6 +--- .../OppdaterAnsvarligSaksbehandlerTaskTest.kt | 33 ++++++++++++++----- 3 files changed, 34 insertions(+), 26 deletions(-) diff --git a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt index 8cb719ee7..0930beacc 100644 --- a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt +++ b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt @@ -30,19 +30,14 @@ class OppdaterAnsvarligSaksbehandlerTask( val behandling = behandlingRepository.findByIdOrThrow(behandlingId) val oppgave = oppgaveService.finnOppgaveForBehandlingUtenOppgaveType(behandlingId) val prioritet = oppgavePrioritetService.utledOppgaveprioritet(behandlingId, oppgave) - log.info("=====>>>> oppgave.tilordnetRessurs: {}", oppgave.tilordnetRessurs) - log.info("=====>>>> behandling.ansvarligSaksbehandler: {}", behandling.ansvarligSaksbehandler) - log.info("=====>>>> oppgave.prioritet: {}", oppgave.prioritet) - log.info("=====>>>> prioritet: {}", prioritet) - /* if (oppgave.tilordnetRessurs != behandling.ansvarligSaksbehandler || oppgave.prioritet != prioritet) { - oppgaveService.patchOppgave(oppgave.copy(tilordnetRessurs = behandling.ansvarligSaksbehandler, prioritet = prioritet)) - }*/ - try{ - oppgaveService.patchOppgave(oppgave.copy(tilordnetRessurs = behandling.ansvarligSaksbehandler, prioritet = prioritet)) - }catch (e:Exception) { - log.info("Eneht på oppgaven og enhet hos saksbehandleren er IKKE det samme!") - oppgaveService.patchOppgave(oppgave.copy(prioritet = prioritet)) - } + oppgave.tildeltEnhetsnr + + try{ + oppgaveService.patchOppgave(oppgave.copy(tilordnetRessurs = behandling.ansvarligSaksbehandler, prioritet = prioritet)) + }catch (e:Exception) { + oppgaveService.patchOppgave(oppgave.copy(prioritet = prioritet)) + log.info("Eneht på oppgaven og enhet hos saksbehandleren er IKKE det samme!") + } } companion object { diff --git a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterOppgaveTask.kt b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterOppgaveTask.kt index 1eeeaee1e..05f31a68a 100644 --- a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterOppgaveTask.kt +++ b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterOppgaveTask.kt @@ -60,11 +60,7 @@ class OppdaterOppgaveTask( beskrivelse + System.lineSeparator() + (oppgave?.beskrivelse ?: "") val prioritet = oppgavePrioritetService.utledOppgaveprioritet(behandlingId, oppgave) - log.info("=====>>>>> TEST saksbehandler <<<====== {}", saksbehandler) - log.info("=====>>>>> TEST oppgave.tildeltEnhetsnr <<<====== {}", oppgave.tildeltEnhetsnr) - log.info("=====>>>>> TEST oppgave.tilordnetRessurs <<<====== {}", oppgave.tilordnetRessurs) - - + var patchetOppgave = oppgave.copy( fristFerdigstillelse = frist, diff --git a/src/test/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTaskTest.kt b/src/test/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTaskTest.kt index 3350deb19..04bf50b5e 100644 --- a/src/test/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTaskTest.kt +++ b/src/test/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTaskTest.kt @@ -1,11 +1,9 @@ package no.nav.familie.tilbake.oppgave -import io.mockk.clearMocks -import io.mockk.every -import io.mockk.mockk -import io.mockk.verify +import io.mockk.* import no.nav.familie.kontrakter.felles.oppgave.Oppgave import no.nav.familie.kontrakter.felles.oppgave.OppgavePrioritet +import no.nav.familie.kontrakter.felles.oppgave.OppgaveResponse import no.nav.familie.kontrakter.felles.oppgave.Oppgavetype import no.nav.familie.prosessering.domene.Task import no.nav.familie.tilbake.behandling.BehandlingRepository @@ -58,10 +56,13 @@ internal class OppdaterAnsvarligSaksbehandlerTaskTest { @Test fun `doTask skal oppdatere oppgave når saksbehandler endret`() { - val oppgave = Oppgave(tilordnetRessurs = "TIDLIGERE saksbehandler", prioritet = OppgavePrioritet.NORM) + val oppgave = Oppgave(tilordnetRessurs = behandling.ansvarligSaksbehandler, prioritet = OppgavePrioritet.NORM) + every { oppgavePrioritetService.utledOppgaveprioritet(any(), any()) } returns OppgavePrioritet.NORM every { mockOppgaveService.finnOppgaveForBehandlingUtenOppgaveType(behandling.id) } returns oppgave + every { mockOppgaveService.patchOppgave(match { it.tilordnetRessurs == behandling.ansvarligSaksbehandler && it.prioritet == OppgavePrioritet.NORM }) } returns OppgaveResponse(oppgave.id ?: 1) + oppdaterAnsvarligSaksbehandlerTask.doTask(lagTask()) verify(atLeast = 1) { @@ -74,16 +75,27 @@ internal class OppdaterAnsvarligSaksbehandlerTaskTest { } } + @Test - fun `Skal ikke oppdatere oppgave når ingenting er endret`() { + fun `Skal kalle patchOppgave med oppdatert prioritet når unntak kastes`() { val oppgave = Oppgave(tilordnetRessurs = behandling.ansvarligSaksbehandler, prioritet = OppgavePrioritet.NORM) + val oppgaveSomFeiler = Oppgave(prioritet = OppgavePrioritet.NORM) every { oppgavePrioritetService.utledOppgaveprioritet(any(), any()) } returns OppgavePrioritet.NORM - every { mockOppgaveService.finnOppgaveForBehandlingUtenOppgaveType(behandling.id) } returns oppgave + every { mockOppgaveService.finnOppgaveForBehandlingUtenOppgaveType(behandling.id) } returns oppgaveSomFeiler + + every { mockOppgaveService.patchOppgave(match { it.tilordnetRessurs == behandling.ansvarligSaksbehandler && it.prioritet == OppgavePrioritet.NORM }) } throws RuntimeException("Mock exception") + every { mockOppgaveService.patchOppgave(match { it.prioritet == OppgavePrioritet.NORM && it.tilordnetRessurs == null }) } returns OppgaveResponse(oppgave.id ?: 1) oppdaterAnsvarligSaksbehandlerTask.doTask(lagTask()) - verify(exactly = 0) { mockOppgaveService.patchOppgave(any()) } + verify(exactly = 2) { mockOppgaveService.patchOppgave(any()) } + verify { + mockOppgaveService.patchOppgave(match { + it.prioritet == OppgavePrioritet.NORM + }) + } + } private fun lagTask(opprettetAv: String? = null): Task = @@ -99,4 +111,9 @@ internal class OppdaterAnsvarligSaksbehandlerTaskTest { } }, ) + + + } + + From 451a02638e1d38de759c3eee918493fa272538ce Mon Sep 17 00:00:00 2001 From: amiraliw Date: Wed, 11 Dec 2024 08:30:43 +0100 Subject: [PATCH 06/11] fjerner wildcard from import --- .../OppdaterAnsvarligSaksbehandlerTask.kt | 12 +++++------ .../tilbake/oppgave/OppdaterOppgaveTask.kt | 2 +- .../OppdaterAnsvarligSaksbehandlerTaskTest.kt | 20 +++++++++---------- 3 files changed, 16 insertions(+), 18 deletions(-) diff --git a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt index 0930beacc..5a436b9c3 100644 --- a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt +++ b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt @@ -32,12 +32,12 @@ class OppdaterAnsvarligSaksbehandlerTask( val prioritet = oppgavePrioritetService.utledOppgaveprioritet(behandlingId, oppgave) oppgave.tildeltEnhetsnr - try{ - oppgaveService.patchOppgave(oppgave.copy(tilordnetRessurs = behandling.ansvarligSaksbehandler, prioritet = prioritet)) - }catch (e:Exception) { - oppgaveService.patchOppgave(oppgave.copy(prioritet = prioritet)) - log.info("Eneht på oppgaven og enhet hos saksbehandleren er IKKE det samme!") - } + try { + oppgaveService.patchOppgave(oppgave.copy(tilordnetRessurs = behandling.ansvarligSaksbehandler, prioritet = prioritet)) + } catch (e: Exception) { + oppgaveService.patchOppgave(oppgave.copy(prioritet = prioritet)) + log.info("Eneht på oppgaven og enhet hos saksbehandleren er IKKE det samme!") + } } companion object { diff --git a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterOppgaveTask.kt b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterOppgaveTask.kt index 05f31a68a..dd4f7f298 100644 --- a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterOppgaveTask.kt +++ b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterOppgaveTask.kt @@ -60,7 +60,7 @@ class OppdaterOppgaveTask( beskrivelse + System.lineSeparator() + (oppgave?.beskrivelse ?: "") val prioritet = oppgavePrioritetService.utledOppgaveprioritet(behandlingId, oppgave) - + var patchetOppgave = oppgave.copy( fristFerdigstillelse = frist, diff --git a/src/test/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTaskTest.kt b/src/test/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTaskTest.kt index 04bf50b5e..7653bd398 100644 --- a/src/test/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTaskTest.kt +++ b/src/test/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTaskTest.kt @@ -1,6 +1,9 @@ package no.nav.familie.tilbake.oppgave -import io.mockk.* +import io.mockk.clearMocks +import io.mockk.every +import io.mockk.mockk +import io.mockk.verify import no.nav.familie.kontrakter.felles.oppgave.Oppgave import no.nav.familie.kontrakter.felles.oppgave.OppgavePrioritet import no.nav.familie.kontrakter.felles.oppgave.OppgaveResponse @@ -75,7 +78,6 @@ internal class OppdaterAnsvarligSaksbehandlerTaskTest { } } - @Test fun `Skal kalle patchOppgave med oppdatert prioritet når unntak kastes`() { val oppgave = Oppgave(tilordnetRessurs = behandling.ansvarligSaksbehandler, prioritet = OppgavePrioritet.NORM) @@ -91,11 +93,12 @@ internal class OppdaterAnsvarligSaksbehandlerTaskTest { verify(exactly = 2) { mockOppgaveService.patchOppgave(any()) } verify { - mockOppgaveService.patchOppgave(match { - it.prioritet == OppgavePrioritet.NORM - }) + mockOppgaveService.patchOppgave( + match { + it.prioritet == OppgavePrioritet.NORM + }, + ) } - } private fun lagTask(opprettetAv: String? = null): Task = @@ -111,9 +114,4 @@ internal class OppdaterAnsvarligSaksbehandlerTaskTest { } }, ) - - - } - - From c77bba81b70dd20dce01b7b0a687164123882e9a Mon Sep 17 00:00:00 2001 From: amiraliw Date: Wed, 11 Dec 2024 10:42:29 +0100 Subject: [PATCH 07/11] la til mer forklarende info log --- .../tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt index 5a436b9c3..0ff050e45 100644 --- a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt +++ b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt @@ -36,7 +36,7 @@ class OppdaterAnsvarligSaksbehandlerTask( oppgaveService.patchOppgave(oppgave.copy(tilordnetRessurs = behandling.ansvarligSaksbehandler, prioritet = prioritet)) } catch (e: Exception) { oppgaveService.patchOppgave(oppgave.copy(prioritet = prioritet)) - log.info("Eneht på oppgaven og enhet hos saksbehandleren er IKKE det samme!") + log.info("Kunne ikke oppdatere tilordnetRessurs. Mulig årsak kan være at eneht på oppgaven og enhet hos saksbehandleren er IKKE det samme!") } } From 3ce56aa2a64bcc1bf6a8850d44debbfe48238953 Mon Sep 17 00:00:00 2001 From: amiraliw Date: Wed, 11 Dec 2024 11:02:12 +0100 Subject: [PATCH 08/11] fjernet ekstra kode --- .../tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt | 1 - 1 file changed, 1 deletion(-) diff --git a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt index 0ff050e45..33694c906 100644 --- a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt +++ b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt @@ -30,7 +30,6 @@ class OppdaterAnsvarligSaksbehandlerTask( val behandling = behandlingRepository.findByIdOrThrow(behandlingId) val oppgave = oppgaveService.finnOppgaveForBehandlingUtenOppgaveType(behandlingId) val prioritet = oppgavePrioritetService.utledOppgaveprioritet(behandlingId, oppgave) - oppgave.tildeltEnhetsnr try { oppgaveService.patchOppgave(oppgave.copy(tilordnetRessurs = behandling.ansvarligSaksbehandler, prioritet = prioritet)) From 88203818ce66821a697fc50a538f7c9996054f49 Mon Sep 17 00:00:00 2001 From: amiraliw Date: Wed, 11 Dec 2024 13:40:01 +0100 Subject: [PATCH 09/11] La if-statement tilbake ettersom ingenting skal oppdateres hvis det ikke er noe endring --- .../oppgave/OppdaterAnsvarligSaksbehandlerTask.kt | 12 +++++++----- .../OppdaterAnsvarligSaksbehandlerTaskTest.kt | 14 +++++++++++++- 2 files changed, 20 insertions(+), 6 deletions(-) diff --git a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt index 33694c906..3b85b0763 100644 --- a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt +++ b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt @@ -31,11 +31,13 @@ class OppdaterAnsvarligSaksbehandlerTask( val oppgave = oppgaveService.finnOppgaveForBehandlingUtenOppgaveType(behandlingId) val prioritet = oppgavePrioritetService.utledOppgaveprioritet(behandlingId, oppgave) - try { - oppgaveService.patchOppgave(oppgave.copy(tilordnetRessurs = behandling.ansvarligSaksbehandler, prioritet = prioritet)) - } catch (e: Exception) { - oppgaveService.patchOppgave(oppgave.copy(prioritet = prioritet)) - log.info("Kunne ikke oppdatere tilordnetRessurs. Mulig årsak kan være at eneht på oppgaven og enhet hos saksbehandleren er IKKE det samme!") + if (oppgave.tilordnetRessurs != behandling.ansvarligSaksbehandler || oppgave.prioritet != prioritet) { + try { + oppgaveService.patchOppgave(oppgave.copy(tilordnetRessurs = behandling.ansvarligSaksbehandler, prioritet = prioritet)) + } catch (e: Exception) { + oppgaveService.patchOppgave(oppgave.copy(prioritet = prioritet)) + log.info("Kunne ikke oppdatere tilordnetRessurs. Mulig årsak kan være at eneht på oppgaven og enhet hos saksbehandleren er IKKE det samme!") + } } } diff --git a/src/test/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTaskTest.kt b/src/test/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTaskTest.kt index 7653bd398..cd0f47ca9 100644 --- a/src/test/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTaskTest.kt +++ b/src/test/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTaskTest.kt @@ -58,12 +58,24 @@ internal class OppdaterAnsvarligSaksbehandlerTaskTest { } @Test - fun `doTask skal oppdatere oppgave når saksbehandler endret`() { + fun `Skal ikke oppdatere oppgave når ingenting er endret`() { val oppgave = Oppgave(tilordnetRessurs = behandling.ansvarligSaksbehandler, prioritet = OppgavePrioritet.NORM) every { oppgavePrioritetService.utledOppgaveprioritet(any(), any()) } returns OppgavePrioritet.NORM every { mockOppgaveService.finnOppgaveForBehandlingUtenOppgaveType(behandling.id) } returns oppgave + oppdaterAnsvarligSaksbehandlerTask.doTask(lagTask()) + + verify(exactly = 0) { mockOppgaveService.patchOppgave(any()) } + } + + @Test + fun `doTask skal oppdatere oppgave når saksbehandler endret`() { + val oppgave = Oppgave(tilordnetRessurs = "Saksbehandler", prioritet = OppgavePrioritet.NORM) + + every { oppgavePrioritetService.utledOppgaveprioritet(any(), any()) } returns OppgavePrioritet.NORM + every { mockOppgaveService.finnOppgaveForBehandlingUtenOppgaveType(behandling.id) } returns oppgave + every { mockOppgaveService.patchOppgave(match { it.tilordnetRessurs == behandling.ansvarligSaksbehandler && it.prioritet == OppgavePrioritet.NORM }) } returns OppgaveResponse(oppgave.id ?: 1) oppdaterAnsvarligSaksbehandlerTask.doTask(lagTask()) From 2af37b40764146c9fea8d2abbfb5fd479dfdccc5 Mon Sep 17 00:00:00 2001 From: amiraliw Date: Tue, 17 Dec 2024 14:43:43 +0100 Subject: [PATCH 10/11] endret til secureLog.warn --- .../tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt index 3b85b0763..0013092bb 100644 --- a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt +++ b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt @@ -5,6 +5,7 @@ import no.nav.familie.prosessering.TaskStepBeskrivelse import no.nav.familie.prosessering.domene.Task import no.nav.familie.tilbake.behandling.BehandlingRepository import no.nav.familie.tilbake.common.repository.findByIdOrThrow +import no.nav.familie.tilbake.integration.pdl.internal.secureLogger import org.slf4j.LoggerFactory import org.springframework.stereotype.Service import java.util.UUID @@ -36,7 +37,7 @@ class OppdaterAnsvarligSaksbehandlerTask( oppgaveService.patchOppgave(oppgave.copy(tilordnetRessurs = behandling.ansvarligSaksbehandler, prioritet = prioritet)) } catch (e: Exception) { oppgaveService.patchOppgave(oppgave.copy(prioritet = prioritet)) - log.info("Kunne ikke oppdatere tilordnetRessurs. Mulig årsak kan være at eneht på oppgaven og enhet hos saksbehandleren er IKKE det samme!") + secureLogger.warn("Kunne ikke oppdatere tilordnetRessurs, ${behandling.ansvarligSaksbehandler}" ) } } } From f1f96b454331bb95ba98ef35d629adcf4f370a95 Mon Sep 17 00:00:00 2001 From: amiraliw Date: Tue, 17 Dec 2024 15:15:38 +0100 Subject: [PATCH 11/11] Fjernet space --- .../tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt index 0013092bb..f87cfb15d 100644 --- a/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt +++ b/src/main/kotlin/no/nav/familie/tilbake/oppgave/OppdaterAnsvarligSaksbehandlerTask.kt @@ -37,7 +37,7 @@ class OppdaterAnsvarligSaksbehandlerTask( oppgaveService.patchOppgave(oppgave.copy(tilordnetRessurs = behandling.ansvarligSaksbehandler, prioritet = prioritet)) } catch (e: Exception) { oppgaveService.patchOppgave(oppgave.copy(prioritet = prioritet)) - secureLogger.warn("Kunne ikke oppdatere tilordnetRessurs, ${behandling.ansvarligSaksbehandler}" ) + secureLogger.warn("Kunne ikke oppdatere tilordnetRessurs, ${behandling.ansvarligSaksbehandler}") } } }