Merge pull request #30 from Open-MBEE/feat/test-coverage-setup

JaCoCo setup and initial test function

build.gradle.kts

@@ -8,6 +8,7 @@ val kotlinx_json_version: String by project
 plugins {
     application
     kotlin("jvm") version "1.9.20"
+    jacoco
 }
 
 group = "org.openmbee.flexo.mms"
@@ -40,4 +41,21 @@ dependencies {
 
     testImplementation("io.ktor:ktor-server-tests:$ktor_version")
     testImplementation("org.jetbrains.kotlin:kotlin-test-junit:$kotlin_version")
+
+    val junitVersion = "5.10.1"
+    testImplementation("org.junit.jupiter:junit-jupiter-engine:$junitVersion")
+}
+
+tasks {
+    test {
+        useJUnitPlatform()
+    }
 }
+
+tasks.test {
+    finalizedBy(tasks.jacocoTestReport) // report is always generated after tests run
+}
+
+tasks.jacocoTestReport {
+    dependsOn(tasks.test) // tests are required to run before generating the report
+}

src/main/kotlin/org/openmbee/flexo/mms/auth/LdapDetails.kt

@@ -131,9 +131,9 @@ private fun ldapEscapeImpl(string: String, firstIndex: Int): String = buildStrin
     append(string, lastIndex, string.length)
 }
 
-private val ESCAPE_CHARACTERS = charArrayOf(' ', '"', '#', '+', ',', ';', '<', '=', '>', '\\')
+internal val ESCAPE_CHARACTERS = charArrayOf(' ', '"', '#', '+', ',', ';', '<', '=', '>', '\\')
 
-private fun Char.shouldEscape(): Boolean = this.code.let { codepoint ->
+internal fun Char.shouldEscape(): Boolean = this.code.let { codepoint ->
     when (codepoint) {
         in 0x3f..0x7e -> codepoint == 0x5c // the only forbidden character is backslash
         in 0x2d..0x3a -> false // minus, point, slash (allowed), digits + colon :

src/test/kotlin/org/openmbee/flexo/mms/CharExtensionsTest.kt

@@ -0,0 +1,15 @@
+package org.openmbee.flexo.mms
+
+import kotlin.test.assertTrue
+import org.junit.jupiter.api.Test
+import org.openmbee.flexo.mms.auth.shouldEscape
+import org.openmbee.flexo.mms.auth.ESCAPE_CHARACTERS
+
+class CharExtensionsTest {
+    @Test
+    fun testShouldEscape() {
+        ESCAPE_CHARACTERS.forEach { char ->
+            assertTrue(char.shouldEscape() as Boolean)
+        }
+    }
+}