Skip to content

Commit 13af1f4

Browse files
committed
Use Groovy convenience method to handle process output
1 parent 10eced1 commit 13af1f4

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

platform-tooling-support-tests/platform-tooling-support-tests.gradle.kts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,9 @@ dependencies {
2222
testImplementation("com.tngtech.archunit:archunit-junit5-api:${Versions.archunit}") {
2323
because("checking the architecture of JUnit 5")
2424
}
25+
testImplementation("org.codehaus.groovy:groovy-all:${Versions.groovy}") {
26+
because("it provides convenience methods to handle process output")
27+
}
2528
testRuntimeOnly("com.tngtech.archunit:archunit-junit5-engine:${Versions.archunit}") {
2629
because("contains the ArchUnit TestEngine implementation")
2730
}

platform-tooling-support-tests/src/test/java/platform/tooling/support/tests/ModularUserGuideTests.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@
2525
import java.util.List;
2626
import java.util.spi.ToolProvider;
2727

28+
import org.codehaus.groovy.runtime.ProcessGroovyMethods;
2829
import org.junit.jupiter.api.Test;
2930
import org.junit.jupiter.api.io.TempDir;
3031
import platform.tooling.support.Helper;
@@ -138,9 +139,10 @@ private static List<String> junit(Path temp, Writer out, Writer err) throws Exce
138139
// System.out.println("______________");
139140
// command.forEach(System.out::println);
140141

141-
var builder = new ProcessBuilder(command).directory(temp.toFile()).inheritIO();
142+
var builder = new ProcessBuilder(command).directory(temp.toFile());
142143
var java = builder.start();
143-
var code = java.waitFor();
144+
ProcessGroovyMethods.waitForProcessOutput(java, out, err);
145+
var code = java.exitValue();
144146

145147
if (code != 0) {
146148
System.out.println(out);

0 commit comments

Comments
 (0)