Merge pull request #385 from gocd/trim-cert-to-null

Treat blank CA Cert data as not set

Author: chadlwilson

src/main/java/cd/go/contrib/elasticagent/PluginSettings.java

@@ -108,7 +108,7 @@ public String getClusterUrl() {
     }
 
     public String getCaCertData() {
-        return clusterCACertData;
+        return isBlank(clusterCACertData) ? null : clusterCACertData;
     }
 
     public Integer getClusterRequestTimeout() {

src/test/java/cd/go/contrib/elasticagent/PluginSettingsTest.java

@@ -98,4 +98,14 @@ public void shouldConsiderBlankStringAsNull() {
 
         assertThat(pluginSettings.getNamespace()).isEqualTo("default");
     }
+
+    @Test
+    public void shouldConsiderBlankCertAsNull() {
+        final Map<String, Object> pluginSettingsMap = new HashMap<>();
+        pluginSettingsMap.put("kubernetes_cluster_ca_cert", "   ");
+
+        PluginSettings pluginSettings = PluginSettings.fromJSON(new Gson().toJson(pluginSettingsMap));
+
+        assertThat(pluginSettings.getCaCertData()).isNull();
+    }
 }