Setter ApplicationState initially til false, for å ikke få false posi…

[inavga]

…tives under oppstart

Vårt team har problemer med at pdfgen-oms, en av applikasjonene våre som bygger på pdfgen avviser forespørsler. Vi mistenker at det kan ha med å gjøre at Nais sender requests til den før den er klar. Det ser ikke ut til at ApplicationState.ready settes til false andre steder enn under shutdown fra før, og den bør jo være false før applikasjonen er ferdig med å starte opp.

[MikAoJk]

Gjerne lag ein unit test, for å verifisere at ApplicationStatene er false, og så blir true, når dei er klare

[MikAoJk]

Bør kanskje også ha denne slik:
src/main/kotlin/no/nav/pdfgen/plugins/LifecycleHooks.kt

fun Application.configureLifecycleHooks(applicationState: ApplicationState) {
    monitor.subscribe(ApplicationStarted) {
        applicationState.ready = true
        applicationState.alive = true}
    monitor.subscribe(ApplicationStopped) {
        applicationState.ready = false
        applicationState.alive = false
    }
}

[inavga]

Gjerne lag ein unit test, for å verifisere at ApplicationStatene er false, og så blir true, når dei er klare

Jeg antar jeg kan gjøre det fra ApplicationTest da? Hvordan kan jeg trigge ApplicationStarted-eventen?

[MikAoJk]

ApplicationTest

Slik kan du trigge det:
monitor.raise(ApplicationStopped, this)

@Test
    internal fun `Returns internal server error when readyness check fails`() {
        testApplication {
            application {
                routing {
                    val applicationState = ApplicationState()
                    applicationState.ready = false
                    applicationState.alive = false
                    registerNaisApi(applicationState)
                }
                monitor.raise(ApplicationStopped, this)
            }
            val response = client.get("/internal/is_ready")

            assertEquals(HttpStatusCode.InternalServerError, response.status)
            assertEquals("Please wait! I'm not ready :(", response.bodyAsText())
        }
    }