Bugfix Famvern controls (#1187)

Author: jonolehagemo

kontroller/src/main/kotlin/no/ssb/kostra/validation/rule/famvern/famvern52a/Rule022OmraaderArbeidet.kt

@@ -20,10 +20,7 @@ import no.ssb.kostra.area.famvern.famvern52a.Familievern52aColumnNames.TEMA_SAMS
 import no.ssb.kostra.area.famvern.famvern52a.Familievern52aColumnNames.TEMA_SYKD_A_COL_NAME
 import no.ssb.kostra.area.famvern.famvern52a.Familievern52aColumnNames.TEMA_TVANG_A_COL_NAME
 import no.ssb.kostra.area.famvern.famvern52a.Familievern52aColumnNames.TEMA_VOLD_A_COL_NAME
-import no.ssb.kostra.area.famvern.famvern52a.Familievern52aFieldDefinitions.fieldDefinitions
 import no.ssb.kostra.program.KostraRecord
-import no.ssb.kostra.program.extension.byColumnName
-import no.ssb.kostra.program.extension.codeExists
 import no.ssb.kostra.validation.report.Severity
 import no.ssb.kostra.validation.rule.AbstractNoArgsRule
 
@@ -34,15 +31,13 @@ class Rule022OmraaderArbeidet :
     ) {
     override fun validate(context: List<KostraRecord>) =
         context
-            .filterNot {
-                tema.any { tema ->
-                    fieldDefinitions.byColumnName(tema).codeExists(it[tema])
-                }
+            .filter {
+                temaer.none { tema -> it[tema] == "1" }
             }.map {
                 createValidationReportEntry(
                     messageText =
                         "Det er ikke fylt ut hvilke områder det har vært arbeidet med siden saken ble opprettet. " +
-                            "Feltet er obligatorisk å fylle ut, og kan inneholde mer enn ett område.",
+                                "Feltet er obligatorisk å fylle ut, og kan inneholde mer enn ett område.",
                     lineNumbers = listOf(it.lineNumber),
                 ).copy(
                     caseworker = it[KONTOR_NR_A_COL_NAME],
@@ -51,7 +46,7 @@ class Rule022OmraaderArbeidet :
             }.ifEmpty { null }
 
     companion object {
-        private val tema =
+        private val temaer =
             listOf(
                 TEMA_PARREL_A_COL_NAME,
                 TEMA_AVKLAR_A_COL_NAME,

kontroller/src/main/kotlin/no/ssb/kostra/validation/rule/famvern/famvern52a/Rule024DeltagelseBehandlingssamtaler.kt

@@ -22,12 +22,12 @@ class Rule024DeltagelseBehandlingssamtaler :
     override fun validate(context: List<KostraRecord>) =
         context
             .filter {
-                deltakere.none { deltaker -> it[deltaker] == JA }
+                fields.none { deltaker -> it[deltaker] == JA }
             }.map {
                 createValidationReportEntry(
                     messageText =
                         "Det er ikke krysset av for om andre deltakere i saken har deltatt i samtaler " +
-                            "med primærklienten i løpet av rapporteringsåret. Feltene er obligatorisk å fylle ut.",
+                                "med primærklienten i løpet av rapporteringsåret. Feltene er obligatorisk å fylle ut.",
                     lineNumbers = listOf(it.lineNumber),
                 ).copy(
                     caseworker = it[KONTOR_NR_A_COL_NAME],
@@ -37,7 +37,7 @@ class Rule024DeltagelseBehandlingssamtaler :
 
     companion object {
         private const val JA = "1"
-        private val deltakere =
+        private val fields =
             listOf(
                 DELT_PARTNER_A_COL_NAME,
                 DELT_EKSPART_A_COL_NAME,

kontroller/src/main/kotlin/no/ssb/kostra/validation/rule/famvern/famvern52a/Rule032SamarbeidMedAndreInstanserSidenOpprettelsen.kt

@@ -13,10 +13,7 @@ import no.ssb.kostra.area.famvern.famvern52a.Familievern52aColumnNames.SAMARB_PS
 import no.ssb.kostra.area.famvern.famvern52a.Familievern52aColumnNames.SAMARB_SKOLE_A_COL_NAME
 import no.ssb.kostra.area.famvern.famvern52a.Familievern52aColumnNames.SAMARB_SOS_A_COL_NAME
 import no.ssb.kostra.area.famvern.famvern52a.Familievern52aColumnNames.SAMARB_STATB_A_COL_NAME
-import no.ssb.kostra.area.famvern.famvern52a.Familievern52aFieldDefinitions.fieldDefinitions
 import no.ssb.kostra.program.KostraRecord
-import no.ssb.kostra.program.extension.byColumnName
-import no.ssb.kostra.program.extension.codeExists
 import no.ssb.kostra.validation.report.Severity
 import no.ssb.kostra.validation.rule.AbstractNoArgsRule
 
@@ -27,15 +24,14 @@ class Rule032SamarbeidMedAndreInstanserSidenOpprettelsen :
     ) {
     override fun validate(context: List<KostraRecord>) =
         context
-            .filterNot {
-                fields.any { field ->
-                    fieldDefinitions.byColumnName(field).codeExists(it[field])
-                }
+            .filter {
+                (it[SAMARB_INGEN_A_COL_NAME] == "1" && fields.any { field -> it[field] == "1" }) ||
+                        (it[SAMARB_INGEN_A_COL_NAME] == " " && fields.none { field -> it[field] == "1" })
             }.map {
                 createValidationReportEntry(
                     messageText =
                         "Det er ikke krysset av for om det har vært samarbeid med andre " +
-                            "instanser siden saken ble opprettet. Feltet er obligatorisk å fylle ut.",
+                                "instanser siden saken ble opprettet. Feltet er obligatorisk å fylle ut.",
                     lineNumbers = listOf(it.lineNumber),
                 ).copy(
                     caseworker = it[KONTOR_NR_A_COL_NAME],
@@ -46,7 +42,6 @@ class Rule032SamarbeidMedAndreInstanserSidenOpprettelsen :
     companion object {
         private val fields =
             listOf(
-                SAMARB_INGEN_A_COL_NAME,
                 SAMARB_LEGE_A_COL_NAME,
                 SAMARB_HELSE_A_COL_NAME,
                 SAMARB_PSYKH_A_COL_NAME,

kontroller/src/test/kotlin/no/ssb/kostra/validation/rule/famvern/FileDescriptionOutputTest.kt

@@ -27,7 +27,7 @@ class FileDescriptionOutputTest : BehaviorSpec({
                     .enable(YAMLGenerator.Feature.INDENT_ARRAYS_WITH_INDICATOR)
                     .enable(YAMLGenerator.Feature.MINIMIZE_QUOTES)
                     .disable(YAMLGenerator.Feature.WRITE_DOC_START_MARKER)
-                val mapper = ObjectMapper(factory).setSerializationInclusion(JsonInclude.Include.NON_NULL)
+                val mapper = ObjectMapper(factory)
                 val fileName = "src/main/resources/file_description_$id.yaml"
                 val fileDescription = FileDescription(
                     title = title,

kontroller/src/test/kotlin/no/ssb/kostra/validation/rule/famvern/famvern52a/Rule032SamarbeidMedAndreInstanserSidenOpprettelsenTest.kt

@@ -12,29 +12,37 @@ class Rule032SamarbeidMedAndreInstanserSidenOpprettelsenTest : BehaviorSpec({
             sut = Rule032SamarbeidMedAndreInstanserSidenOpprettelsen(),
             expectedSeverity = Severity.WARNING,
             ForAllRowItem(
-                "valid count",
-                kostraRecordInTest("1"),
+                "No cooperation is checked, other is unchecked",
+                kostraRecordInTest("1", " "),
             ),
             ForAllRowItem(
-                "missing count",
-                kostraRecordInTest(" "),
+                "No cooperation is checked, but other is also checked",
+                kostraRecordInTest("1", "1"),
                 expectedErrorMessage = "Det er ikke krysset av for om det har vært samarbeid med andre " +
                         "instanser siden saken ble opprettet. Feltet er obligatorisk å fylle ut.",
             ),
             ForAllRowItem(
-                "invalid count, illegal characters",
-                kostraRecordInTest("X"),
+                "No cooperation is unchecked, other is checked",
+                kostraRecordInTest(" ", "1"),
+            ),
+            ForAllRowItem(
+                "No cooperation is unchecked, but other is also unchecked",
+                kostraRecordInTest(" ", " "),
                 expectedErrorMessage = "Det er ikke krysset av for om det har vært samarbeid med andre " +
                         "instanser siden saken ble opprettet. Feltet er obligatorisk å fylle ut.",
             ),
         )
     )
 }) {
     companion object {
-        private fun kostraRecordInTest(count: String) = listOf(
-            Familievern52aTestUtils.familievernRecordInTest(
-                mapOf(Familievern52aColumnNames.SAMARB_INGEN_A_COL_NAME to count)
+        private fun kostraRecordInTest(cooperation: String, other: String) =
+            listOf(
+                Familievern52aTestUtils.familievernRecordInTest(
+                    mapOf(
+                        Familievern52aColumnNames.SAMARB_INGEN_A_COL_NAME to cooperation,
+                        Familievern52aColumnNames.SAMARB_LEGE_A_COL_NAME to other
+                    )
+                )
             )
-        )
     }
 }