POC Cucumber #1211
POC Cucumber #1211
Conversation
pom.xml
Outdated
|
|
||
| <dependency> | ||
| <groupId>io.cucumber</groupId> | ||
| <artifactId>cucumber-junit</artifactId> |
There was a problem hiding this comment.
Hvis vi bruker cucumber-junit-platform-engine så bruker vi junit5 i stedet for 4
There was a problem hiding this comment.
Tror cucumber-java støtter junit5 direkte, tror ikke vi burde bruke java8 her? cucumber/cucumber-jvm#2174
There was a problem hiding this comment.
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 👍
| import java.time.format.DateTimeFormatter | ||
|
|
||
| abstract class BasisDomeneParser { | ||
| companion object { |
There was a problem hiding this comment.
Trenger disse metodene å være inne i en abstract class? Eller kan de bare vare inne i et Object?
There was a problem hiding this comment.
Trenger ikke være i abstract class. Jeg fjernet det og companion object, så nå er det statiske metoder i en fil i stedet.
| | BehandlingId | Endringstype | Endret i behandlingId | | ||
| | 1 | ERSTATTET | 2 | |
There was a problem hiding this comment.
Her ville jeg kanskje forventet å vise til at inntekten i andelshistorikken er forskjellig - siden det er endringen i tilstand? 🤔
There was a problem hiding this comment.
Ja, var litt frem og tilbake når jeg lagde testen. Tenkte at testene strengt talt ikke sjekket på inntekt, men hva historikken ble. Siden du nevnte det så tok jeg med inntekt i test.
- Byttet oppsett til nyere java - Sjekker at inntekt er som forventet i scenario
# Conflicts: # pom.xml
|
Kudos, SonarCloud Quality Gate passed!
|
No description provided.