Only validate user credentials if there are patches involved. (#410)

Author: ddsharpe

imagetool/src/main/java/com/oracle/weblogic/imagetool/cli/menu/CommonPatchingOptions.java

@@ -57,12 +57,14 @@ void initializeOptions() throws IOException, InvalidCredentialException, Invalid
         super.initializeOptions();
         password = Utils.getPasswordFromInputs(passwordStr, passwordFile, passwordEnv);
 
-        // if userid or password is provided, validate the pair of provided values
-        if ((userId != null || password != null) && !AruUtil.rest().checkCredentials(userId, password)) {
-            throw new InvalidCredentialException();
-        }
+        if (applyingPatches()) {
+            // if userid or password is provided, validate the pair of provided values
+            if ((userId != null || password != null) && !AruUtil.rest().checkCredentials(userId, password)) {
+                throw new InvalidCredentialException();
+            }
 
-        Utils.validatePatchIds(patches, false);
+            Utils.validatePatchIds(patches, false);
+        }
     }
 
     /**

imagetool/src/test/java/com/oracle/weblogic/imagetool/cli/menu/CommonPatchingOptions2Test.java

@@ -37,7 +37,7 @@ static void tearDown() throws NoSuchFieldException, IllegalAccessException {
     void noPassword() {
         // This test requires that ARUUtil instance NOT be overridden
         CreateImage createImage = new CreateImage();
-        new CommandLine(createImage).parseArgs("--tag", "tag:1", "--user", "derek");
+        new CommandLine(createImage).parseArgs("--tag", "tag:1", "--user", "derek", "--patches", "12345678");
         assertThrows(InvalidCredentialException.class, createImage::initializeOptions);
     }