Utvidet CPA-tabellen med last_used

[terjenilssen]

• Utvidet CPA-tabellen med last_used.
• Lagt til funksjoner for å:
  • oppdatere last_used.
  • hente last_used for alle CPA'er.
• Lagt inn manglende testing av eksisterende timestamp-funksjoner fra cpa-repo. Har samtidig refaktorisert navn på andre timestamp-funksjoner i CPARepository.kt.
  • Spørsmålet er om vi også skal refaktorisere i Routes.kt og dermed også REST-endepunktene?

[terjenilssen]

Jeg la inn emottak-monitor under spec > accessPolicy > inbound > rules i yaml-filer. Er dette korrekt/tilstrekkelig for å gi emottak-monitor tilgang til å kalle på endepunkter fra cpa-repo? Eller må man legge til/endre flere steder?

[terjenilssen]

TODO: Burde kanskje lage en test på hva som skjer med last_used når en CPA blir oppdatert?
(test av POST-endepunktet /cpa (Route.postCpa()) som kaller på CPARepository.updateOrInsert())

Jeg tror den blir satt til null. Er det i såfall greit? For eksempel:

• CPA brukt i forrige uke.
• CPA oppdatert igår.
• last_used er null idag fordi ingen foreløpig har brukt den etter oppdateringen.

[thburnett]

Hvis du fjerner denne linja, unngår man vel at den blir oppdatert til null ved en oppdatering? Dersom det er en insert, vil den bli null automatisk, mens dersom det er en update vil den beholde eksisterende verdi. Eller vil det feile om man ikke eksplisitt angir alle kolonnene?

[terjenilssen]

Jeg committet nettopp tester hvor jeg tester og forventer at last_used er null etter en update av CPA.

Og så leste jeg denne kommentaren først nå. Jeg testet å fjerne denne linja, men resultatet blir faktisk identisk som før: last_used blir satt til null etter en update.

På en måte tenker jeg at det er greit, fordi da vet vi at den oppdaterte CPA'en ikke har vært i bruk enda. Men vi mister informasjon om når CPA'en sist var i bruk før oppdateringen da.

Om vi ønsker endret oppførsel på dette, må nok vi kjøre en SELECT for å hente ut last_used antar jeg. Eller undersøke upsert nærmere om hvordan man eventuelt kan få den til å beholde felter som ikke er angitt. Altså: Ved insert settes til null, mens ved update beholde eksisterende verdi.

[terjenilssen]

Jeg fikk til at last_used ikke blir nullstilt:

• Oppdaterte til nyeste versjon av exposed (versjon 1.0.0-rc-3)
• Endret upsert med en nytt argument som har blitt gjort tilgjengelig: onUpdateExclude (ved oppdatering: Utelat følgende felter)

CPA.upsert(
    CPA.id,
    onUpdateExclude = listOf(CPA.lastUsed)
) {
    it[id] = cpa.id
    it[CPA.cpa] = cpa.cpa
    it[entryCreated] = cpa.createdDate
    it[updated_date] = cpa.updatedDate
    it[herId] = cpa.herId
    it[lastUsed] = null
}