🐛 Tåler at arbeidsforhold ikke er registrert i aareg

Co-authored-by: Jakob Havstein Eriksen <[email protected]>
navikt · Feb 16, 2024 · 911a5a9 · 911a5a9
1 parent add2bdf
commit 911a5a9
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 13 deletions.
diff --git a/spesialist-selve/src/main/kotlin/no/nav/helse/modell/arbeidsforhold/ArbeidsforholdDao.kt b/spesialist-selve/src/main/kotlin/no/nav/helse/modell/arbeidsforhold/ArbeidsforholdDao.kt
@@ -1,11 +1,11 @@
 package no.nav.helse.modell.arbeidsforhold
 
+import java.time.LocalDate
+import javax.sql.DataSource
 import kotliquery.Session
 import kotliquery.queryOf
 import kotliquery.sessionOf
 import org.intellij.lang.annotations.Language
-import java.time.LocalDate
-import javax.sql.DataSource
 
 class ArbeidsforholdDao(private val dataSource: DataSource) {
     internal fun insertArbeidsforhold(
@@ -118,17 +118,15 @@ class ArbeidsforholdDao(private val dataSource: DataSource) {
             WHERE person_ref = (SELECT id FROM person WHERE fodselsnummer = :fodselsnummer)
               AND arbeidsgiver_ref = (SELECT id FROM arbeidsgiver WHERE orgnummer = :organisasjonsnummer);
         """
-        checkNotNull(
-            session.run(
-                queryOf(
-                    query, mapOf(
-                        "fodselsnummer" to fødselsnummer.toLong(),
-                        "organisasjonsnummer" to organisasjonsnummer.toLong()
-                    )
+        session.run(
+            queryOf(
+                query, mapOf(
+                    "fodselsnummer" to fødselsnummer.toLong(),
+                    "organisasjonsnummer" to organisasjonsnummer.toLong()
                 )
-                    .map { row -> row.sqlDate("oppdatert").toLocalDate() }
-                    .asSingle
             )
-        ) { "Forventer at OpprettArbeidsforholdCommand har lagt inn arbeidsforhold i databasen" }
+                .map { row -> row.sqlDate("oppdatert").toLocalDate() }
+                .asSingle
+        )
     }
 }
diff --git a/...c/main/kotlin/no/nav/helse/modell/arbeidsforhold/command/OppdaterArbeidsforholdCommand.kt b/...c/main/kotlin/no/nav/helse/modell/arbeidsforhold/command/OppdaterArbeidsforholdCommand.kt
@@ -20,6 +20,10 @@ internal class OppdaterArbeidsforholdCommand(
     override fun execute(context: CommandContext): Boolean {
         val sistOppdatert = arbeidsforholdDao.findArbeidsforholdSistOppdatert(fødselsnummer, organisasjonsnummer)
         return when {
+            sistOppdatert == null -> {
+                logg.warn("Oppdaterer ikke arbeidsforhold som ikke finnes i databasen")
+                true
+            }
             skalOppdateres(sistOppdatert) -> behandle(context)
             else -> {
                 logg.info("Arbeidsforhold er allerede oppdatert")

diff --git a/spesialist-selve/src/test/kotlin/no/nav/helse/modell/arbeidsforhold/ArbeidsforholdDaoTest.kt b/spesialist-selve/src/test/kotlin/no/nav/helse/modell/arbeidsforhold/ArbeidsforholdDaoTest.kt
@@ -100,7 +100,7 @@ internal class ArbeidsforholdDaoTest : DatabaseIntegrationTest() {
     fun `finner sist oppdatert`() {
         arbeidsforholdDao.insertArbeidsforhold(FNR, ORGNUMMER, STARTDATO, SLUTTDATO, STILLINGSTITTEL, STILLINGSPROSENT)
         arbeidsforholdDao.findArbeidsforholdSistOppdatert(FNR, ORGNUMMER).also {
-            assertTrue(it > LocalDate.now().minusDays(10))
+            assertTrue(it!! > LocalDate.now().minusDays(10))
         }
     }