Bugfix Q1 (#562)

* Remove Kontroll 115 and 120 from quarterly reporting

.gitignore

@@ -26,6 +26,8 @@ hs_err_pid*
 # Intellij
 .idea*
 *.iml
+/web-frontend/dist/
+/web-frontend/node_modules/
 
 # Node
 dist/

kontroller/src/main/kotlin/no/ssb/kostra/area/regnskap/RegnskapValidator.kt

@@ -64,8 +64,6 @@ abstract class RegnskapValidator(
         Rule080KombinasjonBevilgningFunksjonArt(),
         Rule100SummeringDriftUtgiftsposteringer(),
         Rule105SummeringDriftInntektsposteringer(),
-        Rule115SummeringBalanseAktiva(),
-        Rule120SummeringBalansePassiva(),
         Rule130SkatteInntekter(),
         Rule135Rammetilskudd(),
     )
@@ -75,6 +73,8 @@ abstract class RegnskapValidator(
         Rule090SummeringInvesteringInntektsposteringer(),
         Rule095SummeringInvesteringDifferanse(),
         Rule110SummeringDriftDifferanse(),
+        Rule115SummeringBalanseAktiva(),
+        Rule120SummeringBalansePassiva(),
         Rule125SummeringBalanseDifferanse(),
         Rule126SummeringDriftOsloInternDifferanse(),
         Rule127SummeringInvesteringOsloInternDifferanse(),

kontroller/src/main/kotlin/no/ssb/kostra/validation/rule/regnskap/kostra/Rule115SummeringBalanseAktiva.kt

@@ -20,8 +20,7 @@ class Rule115SummeringBalanseAktiva : AbstractRule<List<KostraRecord>>(
         ?.let { sumAktiva ->
             createSingleReportEntryList(
                 messageText = "Korrigér slik at fila inneholder registrering av aktiva/eiendeler " +
-                        "($sumAktiva), sum kapittel 10-29 i balanse.",
-                severity = if (arguments.kvartal.first() == ' ') Severity.ERROR else Severity.WARNING
+                        "($sumAktiva), sum kapittel 10-29 i balanse."
             )
         }
 }

kontroller/src/main/kotlin/no/ssb/kostra/validation/rule/regnskap/kostra/Rule120SummeringBalansePassiva.kt

@@ -20,8 +20,7 @@ class Rule120SummeringBalansePassiva : AbstractRule<List<KostraRecord>>(
         ?.let { sumPassiva ->
             createSingleReportEntryList(
                 messageText = "Korrigér slik at fila inneholder registrering av passiva/gjeld og egenkapital " +
-                        "($sumPassiva), sum kapittel 31-5990 i balanse.",
-                severity = if (arguments.kvartal.first() == ' ') Severity.ERROR else Severity.WARNING
+                        "($sumPassiva), sum kapittel 31-5990 i balanse."
             )
         }
 }

kontroller/src/test/kotlin/no/ssb/kostra/area/regnskap/kostra/KvartalKostraMainTest.kt

@@ -50,9 +50,9 @@ class KvartalKostraMainTest : BehaviorSpec({
                     }).plus(
                     mapOf(
                         "0AK1" to 1,
-                        "0BK1" to 3,
+                        "0BK1" to 2,
                         "0CK1" to 2,
-                        "0DK1" to 3
+                        "0DK1" to 2
                     ).map { (skjema, expectedNumberOfControls) ->
                         row(
                             "skjema = $skjema, region = $region -> validating a valid record string",
@@ -69,9 +69,9 @@ class KvartalKostraMainTest : BehaviorSpec({
                 ).plus(
                     mapOf(
                         "0AK1" to 2,
-                        "0BK1" to 4,
+                        "0BK1" to 3,
                         "0CK1" to 3,
-                        "0DK1" to 4
+                        "0DK1" to 3
                     ).map { (skjema, expectedNumberOfControls) ->
                         row(
                             "skjema = $skjema, region = $region -> validating a valid record string with invalid data",
@@ -105,7 +105,7 @@ class KvartalKostraMainTest : BehaviorSpec({
     companion object {
         private val regions = setOf("123400", "030100", "040000")
         private val validSkjemaTyper = setOf("0AK1", "0BK1", "0CK1", "0DK1")
-        private const val NUMBER_OF_VALIDATIONS = 34
+        private const val NUMBER_OF_VALIDATIONS = 32
 
         private fun argumentsInTest(
             argumentsVersion: String = RuleTestData.argumentsInTest.aargang,

kontroller/src/test/kotlin/no/ssb/kostra/validation/rule/regnskap/kostra/Rule115SummeringBalanseAktivaTest.kt

@@ -33,14 +33,6 @@ class Rule115SummeringBalanseAktivaTest : BehaviorSpec({
                 description = "0 < belop",
                 context = kostraRecordsInTest("420400", "0B", 2, 10, 1),
             ),
-            ForAllRowItem(
-                description = "isAktiva = false for quarterly reporting",
-                context = kostraRecordsInTest("420400", "0B", 2, 10, 0),
-                arguments = kostraArguments("1"),
-                expectedErrorMessage = "Korrigér slik at fila inneholder registrering av aktiva/eiendeler " +
-                        "(0), sum kapittel 10-29 i balanse.",
-                expectedSeverity = Severity.WARNING
-            ),
         )
     )
 }) {
@@ -58,7 +50,5 @@ class Rule115SummeringBalanseAktivaTest : BehaviorSpec({
             RegnskapConstants.FIELD_KAPITTEL to "$kapittel",
             RegnskapConstants.FIELD_BELOP to "$belop"
         ).toKostraRecord(1, fieldDefinitions).asList()
-
-        private fun kostraArguments(kvartal: String) = argumentsInTest.copy(kvartal = kvartal)
     }
 }

kontroller/src/test/kotlin/no/ssb/kostra/validation/rule/regnskap/kostra/Rule120SummeringBalansePassivaTest.kt

@@ -35,14 +35,6 @@ class Rule120SummeringBalansePassivaTest : BehaviorSpec({
                 description = "0 < belop",
                 context = kostraRecordsInTest("420400", "0B", 2, 31, -1),
             ),
-            ForAllRowItem(
-                description = "isPassiva = false for quarterly reporting",
-                context = kostraRecordsInTest("420400", "0B", 2, 31, 0),
-                arguments = kostraArguments("1"),
-                expectedErrorMessage = "Korrigér slik at fila inneholder registrering av passiva/gjeld og egenkapital " +
-                        "(0), sum kapittel 31-5990 i balanse.",
-                expectedSeverity = Severity.WARNING
-            ),
         )
     )
 }) {
@@ -60,36 +52,5 @@ class Rule120SummeringBalansePassivaTest : BehaviorSpec({
             RegnskapConstants.FIELD_KAPITTEL to "$kapittel",
             FIELD_BELOP to "$belop"
         ).toKostraRecord(1, fieldDefinitions).asList()
-
-        private fun kostraArguments(kvartal: String) = argumentsInTest.copy(kvartal = kvartal)
     }
 }
-
-
-//    Given("context") {
-//        val sut = Rule120SummeringBalansePassiva()
-//
-//        forAll(
-//            row("XX", "31  ", "1", false), // feil skjema
-//            row("0B", "10  ", "1", false), // feil kapittel
-//            row("0B", "31  ", "0", true), // feil belop
-//            row("0B", "31  ", "-1", false), // ok
-//        ) { skjema, kapittel, belop, expectError ->
-//            val kostraRecordList = mapOf(
-//                FIELD_SKJEMA to skjema,
-//                FIELD_FUNKSJON to kapittel,
-//                FIELD_BELOP to belop
-//            ).toKostraRecord(1, fieldDefinitions).asList()
-//
-//            When("Activa is zero for $skjema, $kapittel, $belop") {
-//                verifyValidationResult(
-//                    validationReportEntries = sut.validate(kostraRecordList, argumentsInTest),
-//                    expectError = expectError,
-//                    expectedSeverity = Severity.ERROR,
-//                    "Korrigér slik at fila inneholder registrering av passiva/gjeld og egenkapital " +
-//                            "($belop), sum kapittel 31-5990 i balanse."
-//                )
-//            }
-//        }
-//    }
-//})