TB-140 Legg til nye endepunkter for arkivarisk historikk #218
Conversation
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.OpenSSF Scorecard
Scanned Files |
| @@ -128,6 +128,14 @@ class BygningRouteTest : TestApplicationWithDb() { | |||
| } | |||
| } | |||
| } | |||
|
|
|||
| @Test | |||
| fun `gitt et ugyldig dato query parameter svarer bygning bad request`() = testApplication { | |||
There was a problem hiding this comment.
Legger ved den eneste uløste kommentaren på den forrige PRen her @henriwi
Tenker vi bør ha en test som verifiserer at vi får tilbake korrekt data basert på registreringstidspunkt også (eventuelt om dette allerede er dekket av andre tester)
There was a problem hiding this comment.
Det kan absolutt fikses! La inn en test på det i BygningRepositoryTest som sjekker direkte på db funksjonene, men alltids fint å få en sjekk på selve rutene også. Siden det er to relativt like endepunkter /bygning vs /bruksenhet, er det ønskelig å kjøre tester på begge på rutenivå egentlig? 👍 Ser at jeg bare tok på en av de, men hvis det er selve ruta man tester kan det kanskje greit med begge 🤷♀️ @henriwi
| @@ -80,6 +82,51 @@ fun Route.bygningRouting( | |||
|
|
|||
| call.respond(status, body) | |||
| } | |||
|
|
|||
| get( | |||
| "arkiv", | |||
There was a problem hiding this comment.
Foreslår å ha denne på toppnivå og gjenbruke respons-typer og slikt. Selv om det blir noe "duplikate" paths, tenker jeg det gir mest mening fra et API-ståsted.
There was a problem hiding this comment.
Du får ta en ny titt og se om det ble på det nivået du så for deg nå 😄🫡
Legger til endepunkter for å hente ut egenregistrert data for henholdsvis en bygning og en bruksenhet for et gitt registreringstidspunkt. Åpner for muligheten til å se hva som var registrert på et gitt tidspunkt. TB-140
idagyl
force-pushed
the
TB-140-arkiv-endepunkter-for-bygning-bruksenhet
branch
from
dbc5659 to
b2b8060
Compare
Lukket den forrige PRen pga trøbbel med force pushing etc etter at branchen den bygde på ble merget: #214
Bakgrunn
Det må være mulig å hente ut tilstanden for egenregistrerte data for et gitt tidspunkt tilbake i tid. Dette er blant annet nødvendig for å støtte egenregistrert data inn i Historisk Matrikkelbrev (lovkrav).
Endringer 🚀
/arkiv?registreringstidspunkt=<datoher>Instant.now()- etter samtale med William. I databasen, rent teknisk, henter vi ut materialiseringer frem til og med oppgitt registreringstidspunkt og bruker den nyeste av de.<string>i swagger doc'en og heller legge inn date-time ellernoe i description🤷♀️ Ville ikke bruke for mye tid på det i denne oppgaven for å unngå blokking.Har ikke lagt til noe ekstra tilgangsstyring for nå, det får bli en egen oppgave når vi har fått litt mer oversikt over scopet.
Tips til testing 🧪