search

eclipse / microprofile-health

microprofile-health
Apache License 2.0
106 stars 61 forks source link

Add TCK for HealthChecks produced with CDI method producers for Startup #293

Closed xstefank closed 3 years ago

xstefank commented 3 years ago

@Emily-Jiang correctly pointed in https://github.com/eclipse/microprofile-health/pull/292#discussion_r636011181 that we don't have TCK for other than class level HealthChecks for Startup health checks. We should add some tests for at least CDI method producers as field and parameter injections (Qualifier targets) are more targeted towards use mostly in the implementations.

pgunapal commented 3 years ago

We do have a TCK test case called CDIProducedProceduresTest, which verifies the liveness and readiness health integration with CDI when the Health Check procedures that are defined with CDI Producers. The test uses the following CDIProducedProcedureCheck as a deployment test, which adds the @Liveness and @Readiness HealthChecks with CDI method producers. We probably need to update the test case and deployment to verify the Startup health integration with CDI as well.