Repository navigation
Conversation
|
Ser også at bygget feiler når testene kjører via |
Ja, det er da pga missmatch mellom keycloakversjoner i build.gradle og dockerfile |
| .assertThat().statusCode( | ||
| Matchers.anyOf( | ||
| Matchers.equalTo(401), | ||
| Matchers.equalTo(500)) |
There was a problem hiding this comment.
Denne bør vel alltid gi samme responskode? Så enten 401 eller 500 (vil tro 401 høres mest riktig ut).
Testen bør kun teste for en av statuskodene og feile i andre tilfeller.
There was a problem hiding this comment.
Den gir 500 dessverre..
There was a problem hiding this comment.
Hm, ok. Litt rart da. Klarer du å se noe i loggene på noe vis? Gir den også en 500 hvis du trigger endepunktet i kc-test/betatest?
There was a problem hiding this comment.
ja, kan se ut som det har noe med hvordan exeptions blir håndtert. Skal titte litt mer på det...
There was a problem hiding this comment.
Litt småpirk og noen spørsmål om vi trenger alle testene.
There was a problem hiding this comment.
Et par småpirk, eller tenker jeg dette bare er å merge 👍 Bra jobba!
* Tester for kartverk-opprettede endepunkter i extention
TODO: fyllut mer