feat(kotoutumiskoulutus): add audit log event entries when viewing ko…

…to suoritus
Opetushallitus · Jan 10, 2025 · 72a2377 · 72a2377
1 parent 278627c
commit 72a2377
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 6 deletions.
diff --git a/server/src/main/kotlin/fi/oph/kitu/kotoutumiskoulutus/KielitestiViewController.kt b/server/src/main/kotlin/fi/oph/kitu/kotoutumiskoulutus/KielitestiViewController.kt
@@ -8,14 +8,12 @@ import org.springframework.web.servlet.ModelAndView
 @Controller
 @RequestMapping("koto-kielitesti", produces = ["text/html"])
 class KielitestiViewController(
-    private val suoritusRepository: KielitestiSuoritusRepository,
+    private val suoritusService: KoealustaService,
 ) {
     @GetMapping("/suoritukset")
     fun suorituksetView(): ModelAndView {
-        val suoritukset: List<KielitestiSuoritus> = suoritusRepository.findAll().toList()
-
         val modelAndView = ModelAndView("koto-kielitesti-suoritukset")
-        modelAndView.addObject("suoritukset", suoritukset)
+        modelAndView.addObject("suoritukset", suoritusService.getSuoritukset())
 
         return modelAndView
     }

diff --git a/server/src/main/kotlin/fi/oph/kitu/kotoutumiskoulutus/KoealustaService.kt b/server/src/main/kotlin/fi/oph/kitu/kotoutumiskoulutus/KoealustaService.kt
@@ -4,6 +4,7 @@ import fi.oph.kitu.PeerService
 import fi.oph.kitu.logging.Logging
 import fi.oph.kitu.logging.add
 import fi.oph.kitu.logging.addHttpResponse
+import fi.oph.kitu.logging.addUser
 import fi.oph.kitu.logging.withEventAndPerformanceCheck
 import fi.oph.kitu.oppijanumero.addValidationExceptions
 import org.slf4j.LoggerFactory
@@ -31,6 +32,18 @@ class KoealustaService(
 
     private val restClient by lazy { restClientBuilder.baseUrl(koealustaBaseUrl).build() }
 
+    fun getSuoritukset() =
+        kielitestiSuoritusRepository.findAll().toList().also {
+            for (suoritus in it) {
+                auditLogger
+                    .atInfo()
+                    .addUser()
+                    .add(
+                        "suoritus.id" to suoritus.id,
+                    ).log("Kielitesti suoritus viewed")
+            }
+        }
+
     fun importSuoritukset(from: Instant) =
         logger
             .atInfo()

diff --git a/server/src/test/kotlin/fi/oph/kitu/kotoutumiskoulutus/KoealustaServiceTests.kt b/server/src/test/kotlin/fi/oph/kitu/kotoutumiskoulutus/KoealustaServiceTests.kt
@@ -98,8 +98,11 @@ class KoealustaServiceTests {
             KoealustaService(
                 restClientBuilder = mockRestClientBuilder,
                 kielitestiSuoritusRepository = kielitestiSuoritusRepository,
-                jacksonObjectMapper = objectMapper,
-                mappingService = KoealustaMappingService(OppijanumeroServiceMock("123")),
+                mappingService =
+                    KoealustaMappingService(
+                        objectMapper,
+                        OppijanumeroServiceMock("123"),
+                    ),
             )
 
         koealustaService.koealustaToken = "token"