Coverage

DanielSchiavini · Jul 2, 2024 · d279b27 · d279b27
1 parent 5c92a76
commit d279b27
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 5 deletions.
diff --git a/src/main/kotlin/org/vyperlang/plugin/compile/VyperCompiler.kt b/src/main/kotlin/org/vyperlang/plugin/compile/VyperCompiler.kt
@@ -30,7 +30,7 @@ object VyperCompiler {
     private const val COMPILATION_EMPTY = "Compilation empty"
     private const val COMPILATION_SUCCESS = "Compilation succeed"
 
-    fun compile(params: VyperParameters, indicator: ProgressIndicator?) =
+    fun compile(params: VyperParameters, indicator: ProgressIndicator? = null) =
         try {
             params.files.map {
                 val compiler =

diff --git a/src/main/kotlin/org/vyperlang/plugin/docker/VyperCompilerDocker.kt b/src/main/kotlin/org/vyperlang/plugin/docker/VyperCompilerDocker.kt
@@ -133,8 +133,7 @@ private class VyperFrameStreamAdapter : ResultCallback.Adapter<Frame>() {
     }
 }
 
-private class VyperPullImageAdapter(private val indicator: ProgressIndicator?) :
-    ResultCallback.Adapter<PullResponseItem>() {
+private class VyperPullImageAdapter(private val indicator: ProgressIndicator?) : ResultCallback.Adapter<PullResponseItem>() {
     /**
      * Map of layer id to the `current` and `total` progress of that layer.
      * As the layers get known, the total progress can go backwards a bit, but it's still quite helpful to have.

diff --git a/src/test/kotlin/org/vyperlang/plugin/parser/TestCompilerAnnotator.kt b/src/test/kotlin/org/vyperlang/plugin/parser/TestCompilerAnnotator.kt
@@ -11,7 +11,6 @@ import io.mockk.mockk
 import org.vyperlang.plugin.annotators.CompilerAnnotator
 import org.vyperlang.plugin.annotators.CompilerError
 import org.vyperlang.plugin.annotators.FileInfo
-import kotlin.math.pow
 
 data class ExpectedError(val message: String, val range: Pair<Int, Int>, val tooltip: String="")
 
@@ -90,6 +89,6 @@ class TestCompilerAnnotator : BasePlatformTestCase() {
 
     private fun fileInfo(fileName: String): FileInfo {
         val file = LocalFileSystem.getInstance().findFileByPath("${testDataPath}/$fileName")!!
-        return FileInfo(project, file)
+        return FileInfo(project, file, mockk(relaxed = true))
     }
 }
diff --git a/src/test/kotlin/org/vyperlang/plugin/parser/TestVyperCompiler.kt b/src/test/kotlin/org/vyperlang/plugin/parser/TestVyperCompiler.kt
@@ -6,10 +6,13 @@ import com.intellij.openapi.vfs.LocalFileSystem
 import com.intellij.openapi.wm.RegisterToolWindowTask
 import com.intellij.openapi.wm.ToolWindowManager
 import com.intellij.testFramework.fixtures.BasePlatformTestCase
+import io.mockk.every
+import io.mockk.mockk
 import org.junit.Assert.assertTrue
 import org.vyperlang.plugin.VyperIcons
 import org.vyperlang.plugin.compile.VyperCompiler
 import org.vyperlang.plugin.compile.VyperParameters
+import org.vyperlang.plugin.docker.CompilerMissingError
 import org.vyperlang.plugin.docker.StatusDocker
 import org.vyperlang.plugin.docker.ToolResult
 import org.vyperlang.plugin.settings.VyperSettings
@@ -28,6 +31,14 @@ class TestVyperCompiler : BasePlatformTestCase() {
         )
     }
 
+    fun testMissingDocker() {
+        val params = mockk<VyperParameters> {
+            every { files } throws CompilerMissingError(Exception("Docker is not installed"))
+            every { project } returns myFixture.project
+        }
+        assertNull(VyperCompiler.compile(params))
+    }
+
     fun testCompile() {
         val result = compile("example.vy")
         assertEmpty(result.stderr)