Add unit tests

Signed-off-by: doxiao <[email protected]>

Author: doxiao

kubernetes/internal/create-weblogic-domain.sh

@@ -608,10 +608,10 @@ function createYamlFiles {
     apacheConfigFileName="custom_mod_wl_apache.conf"
     if [ ! -z "${loadBalancerVolumePath}" ]; then
       if [ ! -d ${loadBalancerVolumePath} ]; then
-        echo -e "\nERROR: \nThe specified loadBalancerVolumePath $loadBalancerVolumePath does not exist! \n"
+        echo -e "\nERROR - The specified loadBalancerVolumePath $loadBalancerVolumePath does not exist! \n"
         fail "Exiting due to a validation error"
       elif [ ! -f ${loadBalancerVolumePath}/${apacheConfigFileName} ]; then
-        echo -e "\nERROR: \nThe required file ${apacheConfigFileName} does not exist under the specified loadBalancerVolumePath $loadBalancerVolumePath! \n"
+        echo -e "\nERROR - The required file ${apacheConfigFileName} does not exist under the specified loadBalancerVolumePath $loadBalancerVolumePath! \n"
         fail "Exiting due to a validation error"
       else
         enableLoadBalancerVolumePathPrefix="${enabledPrefix}"

operator/src/test/java/oracle/kubernetes/operator/create/CreateDomainInputsValidationTest.java

@@ -58,6 +58,7 @@ public class CreateDomainInputsValidationTest {
   private static final String PARAM_LOAD_BALANCER = "loadBalancer";
   private static final String PARAM_LOAD_BALANCER_WEB_PORT = "loadBalancerWebPort";
   private static final String PARAM_LOAD_BALANCER_DASHBOARD_PORT = "loadBalancerDashboardPort";
+  private static final String PARAM_LOAD_BALANCER_VOLUME_PATH = "loadBalancerVolumePath";
   private static final String PARAM_JAVA_OPTIONS = "javaOptions";
   private static final String PARAM_VERSION = "version";
 
@@ -554,6 +555,26 @@ public void createDomain_with_invalidLoadBalancerDashboardPort_failsAndReturnsEr
         failsAndPrints(invalidIntegerParamValueError(PARAM_LOAD_BALANCER_DASHBOARD_PORT, val)));
   }
 
+  @Test
+  public void createDomain_with_invalidLoadBalancerVolumePath_failsAndReturnsError()
+      throws Exception {
+    String val = "invalid-load-balancer-volume-path";
+    assertThat(
+        execCreateDomain(
+            newInputs().loadBalancer(LOAD_BALANCER_APACHE).loadBalancerVolumePath(val)),
+        failsAndPrints(missingDirectoryError(PARAM_LOAD_BALANCER_VOLUME_PATH, val)));
+  }
+
+  public void createDomain_with_invalidLoadBalancerVolumePath_missingFile_failsAndReturnsError()
+      throws Exception {
+    String val = "/";
+    assertThat(
+        execCreateDomain(
+            newInputs().loadBalancer(LOAD_BALANCER_APACHE).loadBalancerVolumePath(val)),
+        failsAndPrints(
+            missingFileError(PARAM_LOAD_BALANCER_VOLUME_PATH, val, "custom-mod-wl-apache.conf")));
+  }
+
   // TBD - shouldn't we allow empty java options?
   @Test
   public void createDomain_with_missingJavaOptions_failsAndReturnsError() throws Exception {
@@ -613,6 +634,14 @@ private String invalidRelatedParamValueError(
     return errorRegexp("Invalid.*" + param + ".*" + val + " with " + param2 + ".*" + val2);
   }
 
+  private String missingDirectoryError(String param, String val) {
+    return errorRegexp(param + ".*" + val + ".*" + "does not exist!");
+  }
+
+  private String missingFileError(String param, String val, String dir) {
+    return errorRegexp(param + ".*" + val + ".*" + "does not exist under" + ".*" + dir + ".*");
+  }
+
   private String paramMissingError(String param) {
     return errorRegexp(param + ".*missing");
   }