Adding a negative unit test

sarveshkaushal · sarveshkaushal · commit b7041d02cb56 · 2020-07-24T10:47:20.000-07:00
diff --git a/util/src/main/java/io/kubernetes/client/util/version/Version.java b/util/src/main/java/io/kubernetes/client/util/version/Version.java
@@ -38,8 +38,8 @@ public VersionInfo getVersion() throws ApiException, IOException {
     Response response = null;
     try {
       response = call.execute();
-    } catch (IOException ex) {
-      throw new ApiException(ex);
+    } catch (IOException e) {
+      throw new ApiException(e);
     }
 
     if (!response.isSuccessful()) {
diff --git a/util/src/test/java/io/kubernetes/client/util/version/VersionTest.java b/util/src/test/java/io/kubernetes/client/util/version/VersionTest.java
@@ -18,6 +18,7 @@
 import static com.github.tomakehurst.wiremock.client.WireMock.getRequestedFor;
 import static com.github.tomakehurst.wiremock.client.WireMock.urlPathEqualTo;
 import static com.github.tomakehurst.wiremock.client.WireMock.verify;
+import static org.junit.Assert.assertEquals;
 
 import com.github.tomakehurst.wiremock.junit.WireMockRule;
 import io.kubernetes.client.openapi.ApiClient;
@@ -61,6 +62,34 @@ public void testUrl() throws InterruptedException, IOException, ApiException {
 
     verify(
         getRequestedFor(urlPathEqualTo("/version/"))
-            .withHeader("Content-Type", equalTo("application/json")));
+            .withHeader("Content-Type", equalTo("application/json"))
+            .withHeader("Accept", equalTo("application/json")));
+  }
+
+  @Test
+  public void testFailure() throws InterruptedException, IOException, ApiException {
+
+    wireMockRule.stubFor(
+        get(urlPathEqualTo("/version/"))
+            .willReturn(
+                aResponse()
+                    .withStatus(401)
+                    .withHeader("Content-Type", "application/json")
+                    .withBody("{}")));
+
+    Version versionUtil = new Version(client);
+    boolean thrown = false;
+    try {
+      VersionInfo versionInfo = versionUtil.getVersion();
+    } catch (ApiException ex) {
+      assertEquals(401, ex.getCode());
+      thrown = true;
+    }
+    assertEquals(thrown, true);
+
+    verify(
+        getRequestedFor(urlPathEqualTo("/version/"))
+            .withHeader("Content-Type", equalTo("application/json"))
+            .withHeader("Accept", equalTo("application/json")));
   }
 }
