fixed test cases and bug

Author: Shubh Bapna

extended/src/main/java/io/kubernetes/client/extended/kubectl/KubectlLabel.java

@@ -38,7 +38,7 @@ public KubectlLabel<ApiType> addLabel(String key, String value) {
   }
 
   public KubectlLabel<ApiType> deleteLabel(String key) {
-    this.addingLabels.put(key, null);
+    this.addingLabels.put(key, "null");
     return this;
   }
 

extended/src/test/java/io/kubernetes/client/extended/kubectl/KubectlLabelTest.java

@@ -89,19 +89,14 @@ public void testKubectlDeleteLabelNamespacedResourceShouldWork() throws KubectlE
                                     .withBody("{\"metadata\":{\"name\":\"foo\",\"namespace\":\"default\"}}")));
     wireMockRule.stubFor(
             put(urlPathEqualTo("/api/v1/namespaces/default/pods/foo"))
+                    .withRequestBody(
+                            matchingJsonPath(
+                                    "$.metadata.labels", equalToJson("{ \"k1\": \"null\" }")))
                     .willReturn(
                             aResponse()
                                     .withStatus(200)
                                     .withBody("{\"metadata\":{\"name\":\"foo\",\"namespace\":\"default\"}}")));
 
-    Kubectl.label(V1Pod.class)
-            .apiClient(apiClient)
-            .skipDiscovery()
-            .namespace("default")
-            .name("foo")
-            .addLabel("k1", "v1")
-            .execute();
-
     V1Pod unlabelledPod =
             Kubectl.label(V1Pod.class)
                     .apiClient(apiClient)
@@ -111,8 +106,8 @@ public void testKubectlDeleteLabelNamespacedResourceShouldWork() throws KubectlE
                     .deleteLabel("k1")
                     .execute();
 
-    wireMockRule.verify(2, getRequestedFor(urlPathEqualTo("/api/v1/namespaces/default/pods/foo")));
-    wireMockRule.verify(2, putRequestedFor(urlPathEqualTo("/api/v1/namespaces/default/pods/foo")));
+    wireMockRule.verify(1, getRequestedFor(urlPathEqualTo("/api/v1/namespaces/default/pods/foo")));
+    wireMockRule.verify(1, putRequestedFor(urlPathEqualTo("/api/v1/namespaces/default/pods/foo")));
     assertNotNull(unlabelledPod);
   }
 
@@ -178,24 +173,20 @@ public void testKubectlDeleteLabelClusterResourceShouldWork() throws KubectlExce
                     .willReturn(aResponse().withStatus(200).withBody("{\"metadata\":{\"name\":\"foo\"}}")));
     wireMockRule.stubFor(
             put(urlPathEqualTo("/api/v1/nodes/foo"))
+                    .withRequestBody(
+                            matchingJsonPath(
+                                    "$.metadata.labels", equalToJson("{ \"k1\": \"null\" }")))
                     .willReturn(aResponse().withStatus(200).withBody("{\"metadata\":{\"name\":\"foo\"}}")));
 
-    Kubectl.label(V1Node.class)
-            .apiClient(apiClient)
-            .skipDiscovery()
-            .name("foo")
-            .addLabel("k1", "v1")
-            .execute();
-
     V1Node unlabelledNode =
             Kubectl.label(V1Node.class)
                     .apiClient(apiClient)
                     .skipDiscovery()
                     .name("foo")
                     .deleteLabel("k1")
                     .execute();
-    wireMockRule.verify(2, getRequestedFor(urlPathEqualTo("/api/v1/nodes/foo")));
-    wireMockRule.verify(2, putRequestedFor(urlPathEqualTo("/api/v1/nodes/foo")));
+    wireMockRule.verify(1, getRequestedFor(urlPathEqualTo("/api/v1/nodes/foo")));
+    wireMockRule.verify(1, putRequestedFor(urlPathEqualTo("/api/v1/nodes/foo")));
     assertNotNull(unlabelledNode);
   }