Added Test Cases for CommandExecutor

Updated kotlin_lint.yml
Fixed Lint Errors

.github/workflows/kotlin_lint.yml

@@ -2,7 +2,7 @@ name: kotlin_lint
 
 on:
   push:
-    branches:
+    branches-ignore:
       - "*"
   pull_request:
     paths:

src/jvmMain/kotlin/command/CommandBuilder.kt

@@ -29,11 +29,11 @@ class CommandBuilder {
     fun keyAlias(alias: String) = apply { this.keyAlias = alias }
     fun keyPassword(password: String) = apply { this.keyPassword = password }
 
-    fun verifyAdbPath(value: Boolean, path: String) = apply { this.adbVerifyCommandExecute = Pair(value,path) }
+    fun verifyAdbPath(value: Boolean, path: String) = apply { this.adbVerifyCommandExecute = Pair(value, path) }
 
     fun getAdbVerifyCommand(): String {
-        val (forVerify,path) = adbVerifyCommandExecute
-        if (forVerify){
+        val (forVerify, path) = adbVerifyCommandExecute
+        if (forVerify) {
             return "\"${path}\" version"
         }
         return ""

src/jvmMain/kotlin/command/CommandExecutor.kt

@@ -14,7 +14,7 @@ class CommandExecutor {
         onSuccess: (String) -> Unit,
         onFailure: (Throwable) -> Unit
     ) {
-        coroutineScope.launch(Dispatchers.IO){
+        coroutineScope.launch(Dispatchers.IO) {
             try {
                 val runtime = Runtime.getRuntime()
                 val startTime = System.currentTimeMillis()
@@ -51,5 +51,4 @@ class CommandExecutor {
             }
         }
     }
-
 }

src/jvmMain/kotlin/local/FileStorageHelper.kt

@@ -17,6 +17,7 @@ class FileStorageHelper {
     init {
         initializeDir()
     }
+
     private fun getKryo(): Kryo {
         return if (this::kryo.isInitialized) {
             kryo
@@ -71,5 +72,4 @@ class FileStorageHelper {
             false
         }
     }
-
 }

src/jvmMain/kotlin/local/KiteDbException.kt

@@ -5,5 +5,4 @@ class KiteDbException : RuntimeException {
     constructor(detailMessage: String?) : super(detailMessage)
 
     constructor(detailMessage: String?, throwable: Throwable?) : super(detailMessage, throwable)
-
 }

src/jvmMain/kotlin/ui/Typography.kt

@@ -9,9 +9,9 @@ import androidx.compose.ui.unit.sp
 
 val codeFontFamily = FontFamily(
     Font(resource = "fonts/sans_regular.ttf", weight = FontWeight.Light),
-    Font( resource = "fonts/sans_thin.ttf", weight = FontWeight.Normal),
-    Font( resource = "fonts/sans_medium.ttf", weight = FontWeight.Medium),
-    Font( resource = "fonts/sans_bold.ttf", weight = FontWeight.Bold)
+    Font(resource = "fonts/sans_thin.ttf", weight = FontWeight.Normal),
+    Font(resource = "fonts/sans_medium.ttf", weight = FontWeight.Medium),
+    Font(resource = "fonts/sans_bold.ttf", weight = FontWeight.Bold)
 )
 
 val typography = Typography(

src/jvmMain/kotlin/ui/components/LoadingDialog.kt

@@ -2,7 +2,12 @@ package ui.components
 
 import androidx.compose.foundation.BorderStroke
 import androidx.compose.foundation.border
-import androidx.compose.foundation.layout.*
+import androidx.compose.foundation.layout.fillMaxHeight
+import androidx.compose.foundation.layout.fillMaxWidth
+import androidx.compose.foundation.layout.padding
+import androidx.compose.foundation.layout.Arrangement
+import androidx.compose.foundation.layout.Box
+import androidx.compose.foundation.layout.Column
 import androidx.compose.foundation.shape.RoundedCornerShape
 import androidx.compose.material.CircularProgressIndicator
 import androidx.compose.material.Surface
@@ -45,13 +50,11 @@ fun LoadingDialog(text: String) {
                     Text(
                         text = text,
                         style = Styles.TextStyleBold(20.sp),
-                        modifier = Modifier.padding(start = 8.dp,end = 8.dp, bottom = 8.dp)
+                        modifier = Modifier.padding(start = 8.dp, end = 8.dp, bottom = 8.dp)
                     )
                     CircularProgressIndicator()
                 }
             }
         }
-
     }
-
 }

src/jvmMain/kotlin/utils/Constant.kt

@@ -22,5 +22,4 @@ object FileDialogType {
 object SigningMode {
     const val DEBUG = 1
     const val RELEASE = 2
-}
-
+}

src/jvmMain/kotlin/utils/FileHelper.kt

@@ -12,7 +12,10 @@ object FileHelper {
             Log.i("CHANGED NAME\n Unzipping...")
             try {
                 FileUtils.unzip(newFile, directory)
-                fileStatus.invoke(Constant.SUCCESS, "Rename and Unzip Successful\nFile Saved at ${directory.removeSuffix("\\")}.")
+                fileStatus.invoke(
+                    Constant.SUCCESS,
+                    "Rename and Unzip Successful\nFile Saved at ${directory.removeSuffix("\\")}."
+                )
                 Log.i("TRYING DELETING FILE")
                 val value = FileUtils.deleteFile(newFile)
                 Log.i("DELETE STATUS : $value")
@@ -25,5 +28,4 @@ object FileHelper {
             fileStatus.invoke(Constant.FAILURE, "FAILED RENAMING THE FILE!!")
         }
     }
-
 }

src/jvmMain/kotlin/utils/FileUtils.kt

@@ -1,6 +1,10 @@
 package utils
 
-import java.io.*
+import java.io.BufferedOutputStream
+import java.io.File
+import java.io.IOException
+import java.io.InputStream
+import java.io.FileOutputStream
 import java.util.zip.ZipFile
 
 object FileUtils {
@@ -13,7 +17,6 @@ object FileUtils {
     fun deleteFile(file: File) =
         file.delete()
 
-
     /**
      * @param oldFile
      * @param newFile
@@ -72,5 +75,4 @@ object FileUtils {
         }
         bos.close()
     }
-
 }

src/jvmMain/kotlin/utils/Utils.kt

@@ -1,11 +1,10 @@
 package utils
 
-import java.util.*
+import java.util.Locale
 
 object Utils {
 
-    fun isWindowsOS(): Boolean{
+    fun isWindowsOS(): Boolean {
         return System.getProperty("os.name").lowercase(Locale.getDefault()).contains("windows")
     }
-
 }

src/jvmTest/kotlin/command/CommandExecutorTest.kt

@@ -0,0 +1,55 @@
+package command
+
+import junit.framework.TestCase.assertTrue
+import kotlinx.coroutines.runBlocking
+import org.hamcrest.MatcherAssert.assertThat
+import org.junit.Before
+import org.junit.Test
+
+class CommandExecutorTest {
+    private lateinit var commandExecutor: CommandExecutor
+
+    @Before
+    fun setUp() {
+        commandExecutor = CommandExecutor()
+    }
+
+    // These test cases may not work for MAC and Linux need to verify.
+    @Test
+    fun `execute valid command successfully`() {
+        val expectedOutput = "openjdk 11.0.18 2023-01-17"
+        val cmd = "java --version"
+
+        runBlocking {
+            commandExecutor.executeCommand(
+                cmd,
+                this,
+                onSuccess = {
+                    println("SUCCESS -> $it")
+                    assertThat(expectedOutput, it.contains(expectedOutput))
+                },
+                onFailure = {
+                    println("ERROR -> ${it.message}")
+                    assertThat(expectedOutput, it.message?.contains(expectedOutput) ?: false)
+                }
+            )
+        }
+    }
+
+    @Test
+    fun `execute invalid command and handle failure`() {
+        val cmd = "invalid_command"
+        runBlocking {
+            commandExecutor.executeCommand(
+                cmd,
+                this,
+                onSuccess = {
+                    assertTrue(it.isEmpty())
+                },
+                onFailure = {
+                    assertTrue(it.message?.contains("The system cannot find the file specified") ?: false)
+                }
+            )
+        }
+    }
+}

src/jvmTest/kotlin/ui/components/CheckboxWithTextTest.kt

@@ -4,8 +4,12 @@ import androidx.compose.runtime.getValue
 import androidx.compose.runtime.mutableStateOf
 import androidx.compose.runtime.remember
 import androidx.compose.runtime.setValue
-import androidx.compose.ui.test.*
+import androidx.compose.ui.test.assertIsOff
+import androidx.compose.ui.test.assertIsOn
+import androidx.compose.ui.test.assertTextEquals
+import androidx.compose.ui.test.performClick
 import androidx.compose.ui.test.junit4.createComposeRule
+import androidx.compose.ui.test.onNodeWithTag
 import org.junit.Rule
 import org.junit.Test
 import utils.TestTags
@@ -57,5 +61,4 @@ class CheckboxWithTextTest {
         checkbox.performClick()
         checkbox.assertIsOff()
     }
-
 }