Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

POC Cucumber #1211

Merged
merged 5 commits into from
Feb 22, 2022
Merged
Show file tree
Hide file tree
Changes from 3 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 24 additions & 2 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,8 @@
<prosessering.version>1.20211125150647_a7c6f64</prosessering.version>
<start-class>no.nav.familie.ef.sak.ApplicationKt</start-class>
<kontrakter.version>2.0_20220217093059_3870509</kontrakter.version>

<cucumber.version>7.2.3</cucumber.version>
<junit-vintage-engine.version>5.8.2</junit-vintage-engine.version>

<!--suppress UnresolvedMavenProperty Ligger som secret i github-->
<sonar.projectKey>${SONAR_PROJECTKEY}</sonar.projectKey>
Expand Down Expand Up @@ -222,6 +223,27 @@
<version>${unleash.version}</version>
</dependency>

<dependency>
<groupId>io.cucumber</groupId>
<artifactId>cucumber-java8</artifactId>
<version>${cucumber.version}</version>
<scope>test</scope>
</dependency>

<dependency>
<groupId>io.cucumber</groupId>
<artifactId>cucumber-junit</artifactId>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hvis vi bruker cucumber-junit-platform-engine så bruker vi junit5 i stedet for 4

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tror cucumber-java støtter junit5 direkte, tror ikke vi burde bruke java8 her? cucumber/cucumber-jvm#2174

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Godt poeng. Jeg gikk for cucumber-java8 fordi det var så lite dokumentasjon på cucumber-java. Men så issuet du linket til og skjønte det skulle fases ut og gi lite support, så jeg gjorde en ekstra innsats for å få det til med nytt oppsett til tross for lite dokumentasjon. Har fått det til å virke nå, og det er sjekket inn 👍

<version>${cucumber.version}</version>
<scope>test</scope>
</dependency>

<dependency>
<groupId>org.junit.vintage</groupId>
<artifactId>junit-vintage-engine</artifactId>
<version>${junit-vintage-engine.version}</version>
<scope>runtime</scope>
</dependency>

</dependencies>

<profiles>
Expand Down Expand Up @@ -306,7 +328,7 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-surefire-plugin</artifactId>
<version>2.22.2</version>
<version>3.0.0-M5</version>
</plugin>
</plugins>
</build>
Expand Down
14 changes: 14 additions & 0 deletions src/test/kotlin/no/nav/familie/ef/sak/RunCucumberTest.kt
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
package no.nav.familie.ef.sak.no.nav.familie.ef.sak

import io.cucumber.junit.Cucumber
import io.cucumber.junit.CucumberOptions
import org.junit.runner.RunWith

@RunWith(Cucumber::class)
@CucumberOptions(
features = ["src/test/resources/features"],
plugin = ["pretty", "html:build/cucumber.html"],
tags = "not @ignored",
monochrome = false
)
class RunCucumberTest
Original file line number Diff line number Diff line change
@@ -0,0 +1,143 @@
package no.nav.familie.ef.sak.no.nav.familie.ef.sak.cucumber.domeneparser

import io.cucumber.datatable.DataTable
import no.nav.familie.ef.sak.vedtak.EndringType
import no.nav.familie.ef.sak.vedtak.dto.ResultatType
import java.time.LocalDate
import java.time.format.DateTimeFormatter

abstract class BasisDomeneParser {
companion object {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trenger disse metodene å være inne i en abstract class? Eller kan de bare vare inne i et Object?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trenger ikke være i abstract class. Jeg fjernet det og companion object, så nå er det statiske metoder i en fil i stedet.

val norskDatoFormatter = DateTimeFormatter.ofPattern("dd.MM.yyyy")
val isoDatoFormatter = DateTimeFormatter.ISO_LOCAL_DATE

fun parseDato(domenebegrep: Domenenøkkel, rad: Map<String, String>): LocalDate {
return parseDato(domenebegrep.nøkkel(), rad)
}

fun parseValgfriDato(domenebegrep: Domenenøkkel, rad: Map<String, String?>): LocalDate? {
return parseValgfriDato(domenebegrep.nøkkel(), rad)
}

fun parseString(domenebegrep: Domenenøkkel, rad: Map<String, String>): String {
return verdi(domenebegrep.nøkkel(), rad)
}

fun parseValgfriString(domenebegrep: Domenenøkkel, rad: Map<String, String>): String? {
return valgfriVerdi(domenebegrep.nøkkel(), rad)
}

fun parseBooleanMedBooleanVerdi(domenebegrep: Domenenøkkel, rad: Map<String, String>): Boolean {
val verdi = verdi(domenebegrep.nøkkel(), rad)

return when (verdi) {
"true" -> true
else -> false
}
}

fun parseBoolean(domenebegrep: Domenenøkkel, rad: Map<String, String>): Boolean {
val verdi = verdi(domenebegrep.nøkkel(), rad)

return when (verdi) {
"Ja" -> true
else -> false
}
}

fun parseValgfriBoolean(domenebegrep: String, rad: Map<String, String?>): Boolean? {

if (rad.get(domenebegrep) == null || rad.get(domenebegrep) == "") {
return null
}

return when (rad.get(domenebegrep)) {
"Ja" -> true
"Nei" -> false
else -> null
}
}

fun parseDato(domenebegrep: String, rad: Map<String, String>): LocalDate {
val dato = rad.get(domenebegrep)!!

return if (dato.contains(".")) {
LocalDate.parse(dato, norskDatoFormatter)
} else {
LocalDate.parse(dato, isoDatoFormatter)
}
}

fun parseValgfriDato(domenebegrep: String, rad: Map<String, String?>): LocalDate? {
if (rad.get(domenebegrep) == null || rad.get(domenebegrep) == "") {
return null
}
val dato = rad.get(domenebegrep)!!

return if (dato.contains(".")) {
LocalDate.parse(dato, norskDatoFormatter)
} else {
LocalDate.parse(dato, isoDatoFormatter)
}
}

fun verdi(nøkkel: String, rad: Map<String, String>): String {
val verdi = rad.get(nøkkel)

if (verdi == null || verdi == "") {
throw java.lang.RuntimeException("Fant ingen verdi for $nøkkel")
}

return verdi
}

fun valgfriVerdi(nøkkel: String, rad: Map<String, String>): String? {
val verdi = rad.get(nøkkel)

return verdi
}

fun parseInt(domenebegrep: Domenenøkkel, rad: Map<String, String>): Int {
val verdi = verdi(domenebegrep.nøkkel(), rad)

return Integer.parseInt(verdi)
}

fun parseDouble(domenebegrep: Domenenøkkel, rad: Map<String, String>): Double {
val verdi = verdi(domenebegrep.nøkkel(), rad)
return verdi.toDouble()
}

fun parseValgfriDouble(domenebegrep: Domenenøkkel, rad: Map<String, String>): Double? {
return valgfriVerdi(domenebegrep.nøkkel(), rad)?.toDouble() ?: return null
}

fun parseValgfriInt(domenebegrep: Domenenøkkel, rad: Map<String, String>): Int? {
val verdi = valgfriVerdi(domenebegrep.nøkkel(), rad)
if (verdi == null) {
return null
}

return parseInt(domenebegrep, rad)
}

fun parseResultatType(rad: Map<String, String>): ResultatType {
val verdi = verdi(VedtakDomenebegrep.RESULTAT_TYPE.nøkkel, rad)
return ResultatType.valueOf(verdi)
}

fun parseEndringType(rad: Map<String, String>): EndringType? {
val verdi = valgfriVerdi(VedtakDomenebegrep.ENDRING_TYPE.nøkkel, rad) ?: return null
return EndringType.valueOf(verdi)
}


fun <T> mapDataTable(dataTable: DataTable, radMapper: RadMapper<T>): List<T> {
return dataTable.asMaps().map { radMapper.mapRad(it) }
}
}
}

interface RadMapper<T> {
fun mapRad(rad: Map<String, String>): T
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
package no.nav.familie.ef.sak.no.nav.familie.ef.sak.cucumber.domeneparser

object DomeneParser: BasisDomeneParser() {
}

enum class Domenebegrep(val nøkkel: String) : Domenenøkkel {
BEHANDLING_ID("BehandlingId"),
FRA_OG_MED_DATO("Fra og med dato"),
TIL_OG_MED_DATO("Til og med dato");

override fun nøkkel(): String {
return nøkkel
}
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
package no.nav.familie.ef.sak.no.nav.familie.ef.sak.cucumber.domeneparser

interface Domenenøkkel {

fun nøkkel(): String
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,131 @@
package no.nav.familie.ef.sak.no.nav.familie.ef.sak.cucumber.domeneparser

import io.cucumber.datatable.DataTable
import no.nav.familie.ef.sak.no.nav.familie.ef.sak.cucumber.domeneparser.BasisDomeneParser.Companion.parseEndringType
import no.nav.familie.ef.sak.no.nav.familie.ef.sak.cucumber.domeneparser.BasisDomeneParser.Companion.parseInt
import no.nav.familie.ef.sak.no.nav.familie.ef.sak.cucumber.domeneparser.BasisDomeneParser.Companion.parseResultatType
import no.nav.familie.ef.sak.no.nav.familie.ef.sak.cucumber.domeneparser.BasisDomeneParser.Companion.parseString
import no.nav.familie.ef.sak.no.nav.familie.ef.sak.cucumber.domeneparser.BasisDomeneParser.Companion.parseValgfriDato
import no.nav.familie.ef.sak.no.nav.familie.ef.sak.cucumber.domeneparser.BasisDomeneParser.Companion.parseValgfriInt
import no.nav.familie.ef.sak.no.nav.familie.ef.sak.cucumber.domeneparser.BasisDomeneParser.Companion.parseValgfriString
import no.nav.familie.ef.sak.tilkjentytelse.domain.AndelTilkjentYtelse
import no.nav.familie.ef.sak.tilkjentytelse.domain.TilkjentYtelse
import no.nav.familie.ef.sak.tilkjentytelse.domain.TilkjentYtelseType
import no.nav.familie.ef.sak.vedtak.HistorikkEndring
import no.nav.familie.ef.sak.vedtak.domain.AktivitetType
import no.nav.familie.ef.sak.vedtak.domain.PeriodeWrapper
import no.nav.familie.ef.sak.vedtak.domain.Vedtak
import no.nav.familie.ef.sak.vedtak.domain.Vedtaksperiode
import no.nav.familie.ef.sak.vedtak.domain.VedtaksperiodeType
import no.nav.familie.ef.sak.økonomi.lagTilkjentYtelse
import java.time.LocalDate
import java.time.LocalDateTime
import java.util.UUID

object VedtakDomeneParser {

val behandlingIdTilUUID = mapOf(1 to UUID.randomUUID(), 2 to UUID.randomUUID(), 3 to UUID.randomUUID())
val tilkjentYtelseIdNummerTilUUID = mapOf(1 to UUID.randomUUID(), 2 to UUID.randomUUID(), 3 to UUID.randomUUID())

fun mapVedtak(dataTable: DataTable): List<Vedtak> {
return dataTable.asMaps().map {
VedtakMapper().mapRad(it)
}
}

fun mapBehandlingForHistorikkEndring(dataTable: DataTable): List<Pair<UUID,HistorikkEndring?>> {
return dataTable.asMaps().map {
BehandlingForHistorikkEndringMapper().mapRad(it)
}
}


fun mapAndelTilkjentYtelse(dataTable: DataTable): List<AndelTilkjentYtelse> {
return dataTable.asMaps().map {
AndelTilkjentYtelseMapper().mapRad(it)
}
}


fun lagDefaultTilkjentYtelseFraAndel(dataTable: DataTable): List<TilkjentYtelse> {
return dataTable.asMaps().map {
val andel = AndelTilkjentYtelseMapper().mapRad(it)
lagTilkjentYtelse(listOf(andel), UUID.randomUUID(), andel.kildeBehandlingId)
}
}

class VedtakMapper {
fun mapRad(rad: Map<String, String>): Vedtak {
return Vedtak(
behandlingId = behandlingIdTilUUID[parseInt(VedtakDomenebegrep.BEHANDLING_ID, rad)]!!,
resultatType = parseResultatType(rad),
perioder = PeriodeWrapper(listOf(Vedtaksperiode(LocalDate.now(), LocalDate.now().plusYears(1), AktivitetType.BARN_UNDER_ETT_ÅR, VedtaksperiodeType.HOVEDPERIODE)))
)
}
}

class TilkjentYtelseMapper {

fun mapRad(rad: Map<String, String>): TilkjentYtelse {
return TilkjentYtelse(
id = tilkjentYtelseIdNummerTilUUID[parseInt(VedtakDomenebegrep.TILKJENT_YTELSE_ID, rad)]!!,
behandlingId = behandlingIdTilUUID[parseInt(VedtakDomenebegrep.BEHANDLING_ID, rad)]!!,
personident = parseString(VedtakDomenebegrep.PERSONIDENT, rad),
vedtakstidspunkt = LocalDateTime.now(),
type = TilkjentYtelseType.FØRSTEGANGSBEHANDLING,
andelerTilkjentYtelse = listOf()
)
}
}

class AndelTilkjentYtelseMapper {
fun mapRad(rad: Map<String, String>): AndelTilkjentYtelse {
return AndelTilkjentYtelse(
beløp = parseValgfriInt(VedtakDomenebegrep.BELØP, rad) ?: 0,
stønadFom = parseValgfriDato(VedtakDomenebegrep.FRA_OG_MED_DATO, rad) ?: LocalDate.now(),
stønadTom = parseValgfriDato(VedtakDomenebegrep.TIL_OG_MED_DATO, rad) ?: LocalDate.now().plusYears(1),
personIdent = parseValgfriString(VedtakDomenebegrep.PERSONIDENT, rad) ?: "1",
inntekt = parseValgfriInt(VedtakDomenebegrep.INNTEKT, rad) ?: 0,
inntektsreduksjon = parseValgfriInt(VedtakDomenebegrep.INNTEKTSREDUKSJON, rad) ?: 0,
samordningsfradrag = parseValgfriInt(VedtakDomenebegrep.SAMORDNINGSFRADRAG, rad) ?: 0,
kildeBehandlingId = behandlingIdTilUUID[parseInt(VedtakDomenebegrep.BEHANDLING_ID, rad)]!!
)
}
}

class BehandlingForHistorikkEndringMapper {
fun mapRad(rad: Map<String, String>): Pair<UUID, HistorikkEndring?> {
if (parseEndringType(rad) == null && parseValgfriInt(VedtakDomenebegrep.ENDRET_I_BEHANDLING_ID, rad) == null)
return behandlingIdTilUUID[parseInt(VedtakDomenebegrep.BEHANDLING_ID, rad)]!! to null

return behandlingIdTilUUID[parseInt(VedtakDomenebegrep.BEHANDLING_ID, rad)]!! to
HistorikkEndring(
type = parseEndringType(rad)!!,
behandlingId = behandlingIdTilUUID[parseInt(VedtakDomenebegrep.ENDRET_I_BEHANDLING_ID, rad)]!!,
vedtakstidspunkt = LocalDateTime.now()
)
}
}

}

enum class VedtakDomenebegrep(val nøkkel: String) : Domenenøkkel {
RESULTAT_TYPE("Vedtaksresultat"),
TILKJENT_YTELSE_ID("Tilkjent ytelse Id"),
PERSONIDENT("Personnummer"),
INNTEKT("Inntekt"),
INNTEKTSREDUKSJON("Inntektsreduksjon"),
SAMORDNINGSFRADRAG("Samordningsfradrag"),
BELØP("Beløp"),
FRA_OG_MED_DATO("Fra og med dato"),
TIL_OG_MED_DATO("Til og med dato"),
BEHANDLING_ID("BehandlingId"),
ENDRET_I_BEHANDLING_ID("Endret i behandlingId"),
ENDRING_TYPE("Endringstype"),
HISTORIKKENDRING("Historikkendring"),
;

override fun nøkkel(): String {
return nøkkel
}
}
Loading