Restore original I/O strategy for CliDriver

Author: dmlloyd

devtools/cli/src/test/java/io/quarkus/cli/CliDriver.java

@@ -8,6 +8,7 @@
 import java.io.PrintStream;
 import java.nio.file.Files;
 import java.nio.file.Path;
+import java.time.Duration;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collection;
@@ -153,23 +154,39 @@ public static void preserveLocalRepoSettings(Collection<String> args) {
     public static Result executeArbitraryCommand(Path startingDir, String... args) {
         System.out.println("$ " + String.join(" ", args));
 
-        StringBuilder errorBuilder = new StringBuilder();
         Result result = new Result();
 
-        String output = io.smallrye.common.process.ProcessBuilder.newBuilder(Path.of(args[0]))
-                .arguments(Arrays.copyOfRange(args, 1, args.length))
-                .exitCodeChecker(ec -> {
-                    result.exitCode = ec;
-                    return true;
-                })
-                .directory(startingDir)
-                .output().toSingleString(65536)
-                .error().consumeLinesWith(65536, line -> errorBuilder.append(line).append(System.lineSeparator()))
-                .run();
-        String error = errorBuilder.toString();
-
-        result.stdout = output;
-        result.stderr = error;
+        ByteArrayOutputStream out = new ByteArrayOutputStream();
+        PrintStream outPs = new PrintStream(out);
+        System.setOut(outPs);
+
+        ByteArrayOutputStream err = new ByteArrayOutputStream();
+        PrintStream errPs = new PrintStream(err);
+        System.setErr(errPs);
+
+        try {
+            io.smallrye.common.process.ProcessBuilder.newBuilder(Path.of(args[0]))
+                    .arguments(Arrays.copyOfRange(args, 1, args.length))
+                    .exitCodeChecker(ec -> {
+                        result.exitCode = ec;
+                        return true;
+                    })
+                    .directory(startingDir)
+                    // since there is no I/O, we need an explicit timeout
+                    .softExitTimeout(Duration.ofMinutes(5))
+                    .hardExitTimeout(Duration.ofMinutes(5))
+                    .output().inherited()
+                    .error().logOnSuccess(false).gatherOnFail(false).inherited()
+                    .run();
+        } finally {
+            outPs.flush();
+            errPs.flush();
+            System.setOut(stdout);
+            System.setErr(stderr);
+        }
+
+        result.stdout = out.toString();
+        result.stderr = err.toString();
         return result;
     }