Skip to content

Commit

Permalink
🔥 Fjern funksjon som kun ble brukt i tester
Browse files Browse the repository at this point in the history
  • Loading branch information
chsko committed Oct 28, 2024
1 parent d191f40 commit 5445eb5
Show file tree
Hide file tree
Showing 7 changed files with 2 additions and 23 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,6 @@ import java.util.UUID
interface OppgaveDao {
fun finnOppgaveIdUansettStatus(fødselsnummer: String): Long

fun finnUtbetalingId(oppgaveId: Long): UUID?

fun finnGenerasjonId(oppgaveId: Long): UUID

fun finnOppgave(id: Long): OppgaveFraDatabase?
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,6 @@ class PgOppgaveDao(queryRunner: QueryRunner) : OppgaveDao, QueryRunner by queryR
constructor(dataSource: DataSource) : this(MedDataSource(dataSource))
constructor(session: Session) : this(MedSession(session))

override fun finnUtbetalingId(oppgaveId: Long): UUID? {
return asSQL("SELECT utbetaling_id FROM oppgave WHERE id = :oppgaveId", "oppgaveId" to oppgaveId)
.singleOrNull { it.uuid("utbetaling_id") }
}

override fun finnGenerasjonId(oppgaveId: Long): UUID {
return asSQL("SELECT generasjon_ref FROM oppgave WHERE id = :oppgaveId", "oppgaveId" to oppgaveId)
.single { it.uuid("generasjon_ref") }
Expand Down
3 changes: 2 additions & 1 deletion spesialist-selve/src/test/kotlin/AbstractE2ETest.kt
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,8 @@ internal abstract class AbstractE2ETest : AbstractDatabaseTest() {
spleisBehandlingId = behandlinger.getValue(VEDTAKSPERIODE_ID).last(),
)
private val avviksvurderingTestdata = AvviksvurderingTestdata()
private lateinit var utbetalingId: UUID
internal lateinit var utbetalingId: UUID
private set
internal val snapshotClient = mockk<SnapshotClient>()
private val testRapid = TestRapid()
internal val inspektør get() = testRapid.inspektør
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -213,15 +213,6 @@ class PgOppgaveDaoTest : DatabaseIntegrationTest() {
)
}

@Test
fun `finner utbetalingId`() {
opprettPerson()
opprettArbeidsgiver()
opprettVedtaksperiode()
opprettOppgave(utbetalingId = UTBETALING_ID)
assertEquals(UTBETALING_ID, pgOppgaveDao.finnUtbetalingId(oppgaveId))
}

@Test
fun `finner hendelseId`() {
opprettPerson()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -107,8 +107,6 @@ internal class GodkjenningServiceTest : AbstractIntegrationTest() {
settTotrinnsvurdering(opprinneligSaksbehandler, beslutter)

godkjenningService.hÃ¥ndter(godkjenningDto(), "[email protected]", beslutter)

val utbetalingId = pgOppgaveDao.finnUtbetalingId(1.oppgave(VEDTAKSPERIODE_ID)) ?: fail("Fant ikke utbetalingId")
assertPeriodehistorikk(utbetalingId)
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -113,8 +113,6 @@ class OppgavehenterTest {

override fun finnOppgaveIdUansettStatus(fødselsnummer: String): Long = throw UnsupportedOperationException()

override fun finnUtbetalingId(oppgaveId: Long): UUID = throw UnsupportedOperationException()

override fun finnGenerasjonId(oppgaveId: Long): UUID = throw UnsupportedOperationException()

override fun oppgaveDataForAutomatisering(oppgaveId: Long): OppgaveDataForAutomatisering = throw UnsupportedOperationException()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,11 +26,9 @@ class TotrinnsvurderingServiceTest {
fun `Sett er retur, oppdaterer status på totrinnsvurdering og oppdaterer periodehistorikk med vedtaksperiodeId`() {
val vedtaksperiodeId = UUID.randomUUID()
val oppgaveId = 42L
val utbetalingId = UUID.randomUUID()

every { pgOppgaveDao.finnIdForAktivOppgave(vedtaksperiodeId) } returns oppgaveId
every { pgOppgaveDao.finnIdForAktivOppgave(any()) } returns oppgaveId
every { pgOppgaveDao.finnUtbetalingId(any<Long>()) } returns utbetalingId

totrinnsvurderingService.settAutomatiskRetur(vedtaksperiodeId)

Expand Down

0 comments on commit 5445eb5

Please sign in to comment.