♻️ Kaizen - diverse småendringer anbefalt av IntelliJ inspection

navikt · Aug 20, 2024 · 09eb8f9 · 09eb8f9
1 parent f341e53
commit 09eb8f9
Show file tree

Hide file tree

Showing 44 changed files with 146 additions and 733 deletions.
diff --git a/spesialist-api/src/main/kotlin/no/nav/helse/spesialist/api/feilhåndtering/Modellfeil.kt b/spesialist-api/src/main/kotlin/no/nav/helse/spesialist/api/feilhåndtering/Modellfeil.kt
@@ -25,28 +25,6 @@ abstract class Modellfeil protected constructor() : RuntimeException() {
     override val message: String get() = feilkode
 }
 
-class OppgaveAlleredeTildelt(val tildeling: TildelingApiDto) : Modellfeil() {
-    override val eksternKontekst: Map<String, Any> =
-        mapOf(
-            "tildeling" to tildeling,
-        )
-    override val httpkode = HttpStatusCode.Conflict
-    override val feilkode: String = "oppgave_er_allerede_tildelt"
-
-    override fun logger() {
-        logg.info(
-            "Returnerer {} for {}",
-            keyValue("httpkode", "${httpkode.value}"),
-            keyValue("feilkode", feilkode),
-        )
-        sikkerLogg.info(
-            "Returnerer {} for {}, tildelingsinfo=$eksternKontekst",
-            keyValue("httpkode", "${httpkode.value}"),
-            keyValue("feilkode", feilkode),
-        )
-    }
-}
-
 class OppgaveIkkeTildelt(private val oppgaveId: Long) : Modellfeil() {
     override val eksternKontekst: Map<String, Any> = mapOf("oppgaveId" to oppgaveId.toString())
     override val httpkode = HttpStatusCode.FailedDependency

diff --git a/spesialist-api/src/test/kotlin/no/nav/helse/spesialist/api/AbstractOverstyringApiTest.kt b/spesialist-api/src/test/kotlin/no/nav/helse/spesialist/api/AbstractOverstyringApiTest.kt
@@ -5,7 +5,7 @@ import no.nav.helse.spesialist.api.graphql.schema.ArbeidsforholdOverstyringHandl
 import no.nav.helse.spesialist.api.graphql.schema.InntektOgRefusjonOverstyring
 import no.nav.helse.spesialist.api.graphql.schema.TidslinjeOverstyring
 
-internal abstract class AbstractOverstyringApiTest() : AbstractGraphQLApiTest() {
+internal abstract class AbstractOverstyringApiTest : AbstractGraphQLApiTest() {
     fun overstyrTidslinje(overstyring: TidslinjeOverstyring): JsonNode =
         runQuery(
             """

diff --git a/spesialist-api/src/test/kotlin/no/nav/helse/spesialist/api/DatabaseIntegrationTest.kt b/spesialist-api/src/test/kotlin/no/nav/helse/spesialist/api/DatabaseIntegrationTest.kt
@@ -15,7 +15,6 @@ import no.nav.helse.spesialist.api.db.AbstractDatabaseTest
 import no.nav.helse.spesialist.api.egenAnsatt.EgenAnsattApiDao
 import no.nav.helse.spesialist.api.graphql.schema.NotatType
 import no.nav.helse.spesialist.api.notat.NotatDao
-import no.nav.helse.spesialist.api.notat.NotatRepository
 import no.nav.helse.spesialist.api.oppgave.OppgaveApiDao
 import no.nav.helse.spesialist.api.oppgave.Oppgavehåndterer
 import no.nav.helse.spesialist.api.oppgave.Oppgavestatus
@@ -72,7 +71,6 @@ import org.junit.jupiter.api.Assertions.assertEquals
 import java.time.LocalDate
 import java.time.LocalDateTime
 import java.util.UUID
-import kotlin.properties.Delegates
 
 internal abstract class DatabaseIntegrationTest : AbstractDatabaseTest() {
     private val NAVN = Navn(lagFornavn(), lagFornavn(), lagEtternavn())
@@ -111,11 +109,7 @@ internal abstract class DatabaseIntegrationTest : AbstractDatabaseTest() {
     protected val egenAnsattApiDao = mockk<EgenAnsattApiDao>(relaxed = true)
     protected val snapshotClient = mockk<SnapshotClient>(relaxed = true)
 
-    protected var sisteOppgaveId by Delegates.notNull<Long>()
-    protected var sisteCommandContextId by Delegates.notNull<UUID>()
-
     protected val snapshotService = SnapshotService(snapshotApiDao, snapshotClient)
-    protected val notatRepository = mockk<NotatRepository>(relaxed = true)
     protected val oppgavehåndterer = mockk<Oppgavehåndterer>(relaxed = true)
     protected val totrinnsvurderinghåndterer = mockk<Totrinnsvurderinghåndterer>(relaxed = true)
 
@@ -374,15 +368,6 @@ internal abstract class DatabaseIntegrationTest : AbstractDatabaseTest() {
         )
     }
 
-    protected fun vedtakId(vedtaksperiodeId: UUID = PERIODE.id) =
-        sessionOf(dataSource).use { session ->
-            @Language("PostgreSQL")
-            val statement = "SELECT id FROM vedtak WHERE vedtaksperiode_id = ?"
-            requireNotNull(session.run(queryOf(statement, vedtaksperiodeId).map { it.long("id") }.asSingle)) {
-                "Finner ikke vedtak i db for vedtaksperiodeId=$vedtaksperiodeId"
-            }
-        }
-
     protected fun opprettPerson(
         fødselsnummer: String = FØDSELSNUMMER,
         aktørId: String = AKTØRID,
@@ -560,21 +545,6 @@ internal abstract class DatabaseIntegrationTest : AbstractDatabaseTest() {
             requireNotNull(session.run(queryOf(statement).asUpdateAndReturnGeneratedKey))
         }
 
-    private fun opprettSnapshot() =
-        sessionOf(dataSource, returnGeneratedKey = true).use { session ->
-            @Language("PostgreSQL")
-            val statement = "INSERT INTO snapshot(data, versjon) VALUES(?::json, ?)"
-            requireNotNull(
-                session.run(
-                    queryOf(
-                        statement,
-                        objectMapper.writeValueAsString(snapshot()),
-                        1,
-                    ).asUpdateAndReturnGeneratedKey,
-                ),
-            )
-        }
-
     private fun opprettOppgave(
         status: Oppgavestatus = Oppgavestatus.AvventerSaksbehandler,
         vedtakRef: Long,
@@ -600,10 +570,7 @@ internal abstract class DatabaseIntegrationTest : AbstractDatabaseTest() {
                     ).asUpdateAndReturnGeneratedKey,
                 )
             },
-        ).also {
-            sisteOppgaveId = it
-            sisteCommandContextId = commandContextId
-        }
+        )
     }
 
     private fun opprettHendelse(
@@ -876,20 +843,6 @@ internal abstract class DatabaseIntegrationTest : AbstractDatabaseTest() {
         assertEquals(forventetAntall, antall)
     }
 
-    protected fun assertAvvisteVarsler(
-        generasjonRef: Long,
-        forventetAntall: Int,
-    ) {
-        @Language("PostgreSQL")
-        val query =
-            "SELECT COUNT(1) FROM selve_varsel sv WHERE sv.generasjon_ref = ? AND status = 'AVVIST'"
-        val antall =
-            sessionOf(dataSource).use { session ->
-                session.run(queryOf(query, generasjonRef).map { it.int(1) }.asSingle)
-            }
-        assertEquals(forventetAntall, antall)
-    }
-
     protected fun finnOppgaveIdFor(vedtaksperiodeId: UUID): Long =
         sessionOf(dataSource).use { session ->
             @Language("PostgreSQL")

diff --git a/spesialist-api/src/test/kotlin/no/nav/helse/spesialist/api/TestApplication.kt b/spesialist-api/src/test/kotlin/no/nav/helse/spesialist/api/TestApplication.kt