Additional test (#15)

Author: csviri

src/main/java/com/csviri/jenvtest/binary/BinaryManager.java

@@ -27,7 +27,7 @@ public BinaryManager(APIServerConfig config) {
     }
 
     public void initAndDownloadIfRequired() {
-        Optional<File> maybeBinaryDir = findLatestBinariesAvailable();
+        Optional<File> maybeBinaryDir = findTargetBinariesIfAvailable();
         File binaryDir = maybeBinaryDir.orElse(null);
 
         if (maybeBinaryDir.isEmpty()) {
@@ -63,11 +63,16 @@ public Binaries binaries() {
         return binaries;
     }
 
-    private Optional<File> findLatestBinariesAvailable() {
+    private Optional<File> findTargetBinariesIfAvailable() {
         var platformSuffix = Utils.platformSuffix(osInfoProvider);
         if (config.getApiServerVersion().isPresent()) {
-            return Optional.of(new File(config.getJenvtestDir(), BINARY_LIST_DIR
-                    + File.separator + config.getApiServerVersion() + platformSuffix));
+            var targetVersionDir = new File(config.getJenvtestDir(), BINARY_LIST_DIR
+                    + File.separator + config.getApiServerVersion().get() + platformSuffix);
+            if (targetVersionDir.exists()){
+                return Optional.of(targetVersionDir);
+            } else {
+                return Optional.empty();
+            }
         }
         File binariesListDir = new File(config.getJenvtestDir(), BINARY_LIST_DIR);
         if (!binariesListDir.exists()) {

src/test/java/com/csviri/jenvtest/ApiServerTest.java

@@ -10,19 +10,29 @@
 class ApiServerTest {
 
     @Test
-    void sanityTest() {
-        var kubeApi = new APIServer();
-        try {
-            kubeApi.start();
-
-            var client = new KubernetesClientBuilder().build();
-            client.resource(testConfigMap()).createOrReplace();
-            var cm = client.resource(testConfigMap()).get();
-
-            assertThat(cm).isNotNull();
-        } finally {
-            kubeApi.stop();
-        }
+    void trivialCase() {
+        testWithAPIServer(new APIServer());
     }
 
+    @Test
+    void apiServerWithSpecificVersion() {
+        testWithAPIServer(new APIServer(APIServerConfigBuilder.anAPIServerConfig()
+                .withApiServerVersion("1.26.0")
+                .build()));
+    }
+
+
+    void testWithAPIServer(APIServer kubeApi) {
+        kubeApi.start();
+
+        var client = new KubernetesClientBuilder().build();
+        client.resource(testConfigMap()).create();
+        var cm = client.resource(testConfigMap()).get();
+
+        assertThat(cm).isNotNull();
+
+        kubeApi.stop();
+    }
+
+
 }

src/test/java/com/csviri/jenvtest/binary/BinaryManagerTest.java

@@ -0,0 +1,22 @@
+package com.csviri.jenvtest.binary;
+
+import com.csviri.jenvtest.APIServerConfig;
+import com.csviri.jenvtest.APIServerConfigBuilder;
+import com.csviri.jenvtest.JenvtestException;
+import org.junit.jupiter.api.Test;
+
+import static org.junit.jupiter.api.Assertions.*;
+
+class BinaryManagerTest {
+
+    @Test
+    void throwsExceptionIfBinaryNotPresentAndInOfflineMode() {
+        BinaryManager binaryManager = new BinaryManager(APIServerConfigBuilder.anAPIServerConfig()
+                .withDownloadBinaries(false)
+                .withApiServerVersion("1.0.1")
+                .build());
+
+        assertThrows(JenvtestException.class, binaryManager::initAndDownloadIfRequired);
+    }
+
+}