refactor: architecture

Signed-off-by: Alan Brault <alan.brault@visus.io>

src/main/kotlin/io/visus/demos/kotlinapi/controller/HealthController.kt

@@ -6,11 +6,7 @@ import io.swagger.v3.oas.annotations.media.Schema
 import io.swagger.v3.oas.annotations.responses.ApiResponse
 import io.swagger.v3.oas.annotations.tags.Tag
 import io.visus.demos.kotlinapi.api.dto.HealthResponse
-import io.visus.demos.kotlinapi.api.mappers.HealthResponseMapper
-import io.visus.demos.kotlinapi.domain.model.HealthStatus
-import io.visus.demos.kotlinapi.domain.model.Status
 import io.visus.demos.kotlinapi.domain.service.HealthService
-import org.springframework.http.HttpStatus
 import org.springframework.http.MediaType
 import org.springframework.http.ResponseEntity
 import org.springframework.web.bind.annotation.GetMapping
@@ -19,7 +15,7 @@ import org.springframework.web.bind.annotation.RestController
 @RestController
 @Tag(name = "Health", description = "Health check endpoints")
 class HealthController(
-    private val healthCheckServices: List<HealthService>,
+    private val healthCheckService: HealthService,
 ) {
     @GetMapping("/health", produces = [MediaType.APPLICATION_JSON_VALUE])
     @Operation(
@@ -48,17 +44,5 @@ class HealthController(
             ),
         ],
     )
-    fun health(): ResponseEntity<HealthResponse> {
+    fun health(): ResponseEntity<HealthResponse> = healthCheckService.check()
-        val componentHealths = healthCheckServices.map { it.check() }
-        val healthStatus = HealthStatus.fromComponents(componentHealths)
-        val response = HealthResponseMapper.map(healthStatus)
-
-        val httpStatus =
-            when (healthStatus.status) {
-                Status.UP -> HttpStatus.OK
-                Status.DOWN, Status.DEGRADED -> HttpStatus.SERVICE_UNAVAILABLE
-            }
-
-        return ResponseEntity.status(httpStatus).body(response)
-    }
 }

src/main/kotlin/io/visus/demos/kotlinapi/domain/port/HealthIndicator.kt

@@ -0,0 +1,7 @@
+package io.visus.demos.kotlinapi.domain.port
+
+import io.visus.demos.kotlinapi.domain.model.ComponentHealth
+
+interface HealthIndicator {
+    fun check(): ComponentHealth
+}

src/main/kotlin/io/visus/demos/kotlinapi/domain/service/HealthService.kt

@@ -1,7 +1,29 @@
 package io.visus.demos.kotlinapi.domain.service
 
-import io.visus.demos.kotlinapi.domain.model.ComponentHealth
+import io.visus.demos.kotlinapi.api.dto.HealthResponse
+import io.visus.demos.kotlinapi.api.mappers.HealthResponseMapper
+import io.visus.demos.kotlinapi.domain.port.HealthIndicator
+import io.visus.demos.kotlinapi.domain.model.HealthStatus
+import io.visus.demos.kotlinapi.domain.model.Status
+import org.springframework.http.HttpStatus
+import org.springframework.http.ResponseEntity
+import org.springframework.stereotype.Service
 
-interface HealthService {
+@Service
-    fun check(): ComponentHealth
+class HealthService(
+    private val healthIndicators: List<HealthIndicator>,
+) {
+    fun check(): ResponseEntity<HealthResponse> {
+        val componentHealths = healthIndicators.map { it.check() }
+        val healthStatus = HealthStatus.fromComponents(componentHealths)
+        val response = HealthResponseMapper.map(healthStatus)
+
+        val httpStatus =
+            when (healthStatus.status) {
+                Status.UP -> HttpStatus.OK
+                Status.DOWN, Status.DEGRADED -> HttpStatus.SERVICE_UNAVAILABLE
+            }
+
+        return ResponseEntity.status(httpStatus).body(response)
+    }
 }

src/main/kotlin/io/visus/demos/kotlinapi/infrastructure/health/MongoHealthService.kt

@@ -3,14 +3,14 @@ package io.visus.demos.kotlinapi.infrastructure.health
 import com.mongodb.client.MongoClient
 import io.visus.demos.kotlinapi.domain.model.ComponentHealth
 import io.visus.demos.kotlinapi.domain.model.ComponentStatus
-import io.visus.demos.kotlinapi.domain.service.HealthService
+import io.visus.demos.kotlinapi.domain.port.HealthIndicator
 import org.bson.Document
 import org.springframework.stereotype.Service
 
 @Service
 class MongoHealthService(
     private val mongoClient: MongoClient,
-) : HealthService {
+) : HealthIndicator {
     override fun check(): ComponentHealth =
         try {
             val command = Document("ping", 1)