Merge pull request #501 from rundeck/fix/exit-code

RUN-1634: Fix exit codes on command failures

Author: gschueler

rd-cli-acl/src/main/java/org/rundeck/client/ext/acl/Acl.java

@@ -477,9 +477,9 @@ static class TestOptions
     }
 
     @CommandLine.Command(description = "Test ACL Policies")
-    public boolean test(@CommandLine.Mixin TestOptions opts)  {
+    public int test(@CommandLine.Mixin TestOptions opts)  {
         if (!applyArgValidate(opts)) {
-            return false;
+            return 2;
         }
         final RuleEvaluator authorization = createAuthorization(opts);
         AuthRequest authRequest = createAuthRequestFromArgs(opts);
@@ -562,7 +562,7 @@ public boolean test(@CommandLine.Mixin TestOptions opts)  {
             );
         }
         log("The test " + (testPassed ? "passed" : "failed"));
-        return testPassed;
+        return testPassed?0:2;
     }
 
     @CommandLine.Command(description = "Create ACL Policies")
@@ -580,11 +580,11 @@ public void create(@CommandLine.Mixin AclCreateOptions opts) throws IOException
     }
 
     @CommandLine.Command(description = "Validate ACL Policies")
-    public boolean validate(@CommandLine.Mixin AclOptions opts) {
+    public int validate(@CommandLine.Mixin AclOptions opts) {
         final Validation validation = validatePolicies(opts);
         reportValidation(validation);
         log("The validation " + (validation.isValid() ? "passed" : "failed"));
-        return validation.isValid();
+        return validation.isValid() ? 0 : 2;
     }
 
     private HashSet<Map<String, String>> resources(final Map<String, String> resourceMap) {

rd-cli-base/src/main/java/org/rundeck/client/tool/commands/repository/InstallPlugin.java

@@ -27,7 +27,7 @@
 import java.util.concurrent.Callable;
 
 @CommandLine.Command(description = "Install a plugin from your plugin repository into your Rundeck instance", name = "install")
-public class InstallPlugin implements RdCommandExtension, RdOutput, Callable<Boolean> {
+public class InstallPlugin implements RdCommandExtension, RdOutput, Callable<Integer> {
     @Setter
     private RdTool rdTool;
 
@@ -40,7 +40,7 @@ public class InstallPlugin implements RdCommandExtension, RdOutput, Callable<Boo
     @CommandLine.Option(names = {"--version", "-v"}, description = "(Optional) Specific version of the plugin you want to install")
     String version;
 
-    public Boolean call() throws InputError, IOException {
+    public Integer call() throws InputError, IOException {
         RepositoryResponseHandler.handle(
                 rdTool.apiWithErrorResponse(api -> {
                     if (version != null) {
@@ -49,7 +49,7 @@ public Boolean call() throws InputError, IOException {
                         return api.installPlugin(repoName, pluginId);
                     }
                 }), rdOutput);
-        return true;
+        return 0;
     }
 
 }

rd-cli-base/src/main/java/org/rundeck/client/tool/commands/repository/UninstallPlugin.java

@@ -27,7 +27,7 @@
 import java.util.concurrent.Callable;
 
 @CommandLine.Command(description = "Unistall a Rundeck plugin from your Rundeck instance", name = "uninstall")
-public class UninstallPlugin implements Callable<Boolean>, RdCommandExtension, RdOutput {
+public class UninstallPlugin implements Callable<Integer>, RdCommandExtension, RdOutput {
     @Setter
     private RdTool rdTool;
     @Setter
@@ -38,10 +38,10 @@ public class UninstallPlugin implements Callable<Boolean>, RdCommandExtension, R
     String pluginId;
 
 
-    public Boolean call() throws InputError, IOException {
+    public Integer call() throws InputError, IOException {
         RepositoryResponseHandler.handle(
                 rdTool.apiWithErrorResponse(api -> api.uninstallPlugin(pluginId)), rdOutput
         );
-        return true;
+        return 0;
     }
 }

rd-cli-base/src/main/java/org/rundeck/client/tool/commands/repository/UploadPlugin.java

@@ -26,7 +26,7 @@
 import java.util.concurrent.Callable;
 
 @CommandLine.Command(description = "Upload a Rundeck plugin to your plugin repository", name = "upload")
-public class UploadPlugin extends BaseCommand implements Callable<Boolean> {
+public class UploadPlugin extends BaseCommand implements Callable<Integer> {
 
     @CommandLine.Option(names = {"-r", "--repository"}, description = "Target name of repository to upload plugin into.", required = true)
     String repoName;
@@ -35,13 +35,13 @@ public class UploadPlugin extends BaseCommand implements Callable<Boolean> {
     String binaryPath;
 
 
-    public Boolean call() throws InputError, IOException {
+    public Integer call() throws InputError, IOException {
         File binary = new File(binaryPath);
         if (!binary.exists())
             throw new IOException(String.format("Unable to find specified file: %s", binaryPath));
         RequestBody fileUpload = RequestBody.create(binary, Client.MEDIA_TYPE_OCTET_STREAM);
 
         RepositoryResponseHandler.handle(getRdTool().apiWithErrorResponse(api -> api.uploadPlugin(repoName, fileUpload)), getRdOutput());
-        return true;
+        return 0;
     }
 }

rd-cli-base/src/test/groovy/org/rundeck/client/tool/commands/repository/InstallPluginTest.groovy

@@ -46,10 +46,11 @@ class InstallPluginTest extends Specification {
         installCmd.pluginId = 'bcf8885df1e8'
 
         when:
-        installCmd.call()
+        def result = installCmd.call()
 
         then:
         1 * api.installPlugin("private", 'bcf8885df1e8') >> Calls.response(new ArtifactActionMessage(msg: "Plugin Installed"))
         1 * out.output('Plugin Installed')
+        result == 0
     }
 }

rd-cli-base/src/test/groovy/org/rundeck/client/tool/commands/repository/UninstallPluginTest.groovy

@@ -43,10 +43,11 @@ class UninstallPluginTest extends Specification {
 
 
         when:
-        uninstallCmd.call()
+        def result=uninstallCmd.call()
 
         then:
         1 * api.uninstallPlugin(_) >> Calls.response(new ArtifactActionMessage(msg: "Plugin Uninstalled"))
         1 * out.output('Plugin Uninstalled')
+        result==0
     }
 }

rd-cli-base/src/test/groovy/org/rundeck/client/tool/commands/repository/UploadPluginTest.groovy

@@ -47,10 +47,11 @@ class UploadPluginTest extends Specification {
 
 
         when:
-        uploadCmd.call()
+        def result = uploadCmd.call()
 
         then:
         1 * api.uploadPlugin("private", _) >> Calls.response(new ArtifactActionMessage(msg: "Upload succeeded"))
         1 * out.output('Upload succeeded')
+        result == 0
     }
 }

rd-cli-enterprise/build.gradle

@@ -14,7 +14,7 @@ dependencies {
     implementation libs.picocli
     annotationProcessor libs.picocliCodegen
 
-    testImplementation project(":rd-cli-lib"), project(":rd-api-client")
+    testImplementation project(":rd-cli-lib"), project(":rd-api-client"), project(":rd-testing")
 
     testImplementation libs.retrofitMock
     testImplementation libs.okhttpMockwebserver

rd-cli-enterprise/src/main/java/org/rundeck/client/tool/commands/enterprise/cluster/Mode.java

@@ -28,14 +28,14 @@ static class Options {
     }
 
     @CommandLine.Command(description = "Set cluster member execution mode Active")
-    public boolean active(@CommandLine.Mixin Options options) throws IOException, InputError {
-        return changeMode(ExecutionMode.active, options, EnterpriseApi::executionModeEnable);
+    public int active(@CommandLine.Mixin Options options) throws IOException, InputError {
+        return changeMode(ExecutionMode.active, options, EnterpriseApi::executionModeEnable) ? 0 : 1;
     }
 
 
     @CommandLine.Command(description = "Set cluster member execution mode Passive")
-    public boolean passive(@CommandLine.Mixin Options options) throws IOException, InputError {
-        return changeMode(ExecutionMode.passive, options, EnterpriseApi::executionModeDisable);
+    public int passive(@CommandLine.Mixin Options options) throws IOException, InputError {
+        return changeMode(ExecutionMode.passive, options, EnterpriseApi::executionModeDisable) ? 0 : 1;
     }
 
     boolean changeMode(

rd-cli-enterprise/src/main/java/org/rundeck/client/tool/commands/enterprise/license/License.java

@@ -50,7 +50,7 @@ static class StatusOpts {
     }
 
     @CommandLine.Command(description = "license status")
-    public boolean status(@CommandLine.Mixin StatusOpts opts) throws InputError, IOException {
+    public int status(@CommandLine.Mixin StatusOpts opts) throws InputError, IOException {
         RdTool.apiVersionCheck("license status", 41, getClient().getApiVersion());
         LicenseResponse response = getClient().apiCall(EnterpriseApi::verifyLicense);
 
@@ -85,9 +85,9 @@ public boolean status(@CommandLine.Mixin StatusOpts opts) throws InputError, IOE
                         opts.getRemaining()
                 ));
             }
-            return false;
+            return 1;
         }
-        return !opts.isStatus() || response.isActive();
+        return (!opts.isStatus() || response.isActive()) ? 0 : 1;
     }