Fix(inspect-code): getRetCode was always inverted

Apicurio · Nov 22, 2024 · 2b59596 · 2b59596
1 parent 311a580
commit 2b59596
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/src/main/java/io/apicurio/registry/systemtests/executor/Exec.java b/src/main/java/io/apicurio/registry/systemtests/executor/Exec.java
@@ -292,21 +292,21 @@ public static ExecutionResultData executeAndCheck(String input, int timeout, Str
                 input
         );
 
-        if (!results.getRetCode()) {
-            throw new IllegalStateException(results.getStdErr());
+        if (results.getRetCode()) {
+            return results;
         }
 
-        return results;
+        throw new IllegalStateException(results.getStdErr());
     }
 
     public static ExecutionResultData executeAndCheck(
             List<String> command, int timeout, boolean logToOutput, boolean appendLineSeparator, Map<String, String> env
     ) {
         ExecutionResultData results = execute(command, timeout, logToOutput, appendLineSeparator, env, null);
-        if (!results.getRetCode()) {
-            throw new IllegalStateException(results.getStdErr());
+        if (results.getRetCode()) {
+            return results;
         }
-        return results;
+        throw new IllegalStateException(results.getStdErr());
     }
 
     public static ExecutionResultData execute(List<String> command) {