Make test independent of local Git repo config

Author: marcphilipp

junit-platform-reporting/src/main/java/org/junit/platform/reporting/open/xml/OpenTestReportGeneratingListener.java

@@ -58,6 +58,7 @@
 import java.net.UnknownHostException;
 import java.nio.charset.Charset;
 import java.nio.file.Path;
+import java.nio.file.Paths;
 import java.time.Instant;
 import java.util.Map;
 import java.util.Optional;
@@ -108,9 +109,16 @@ public class OpenTestReportGeneratingListener implements TestExecutionListener {
 	private final AtomicInteger idCounter = new AtomicInteger();
 	private final Map<UniqueId, String> inProgressIds = new ConcurrentHashMap<>();
 	private DocumentWriter<Events> eventsFileWriter = DocumentWriter.noop();
+	private final Path workingDir;
 	private Path outputDir;
 
+	@SuppressWarnings("unused") // Used via ServiceLoader
 	public OpenTestReportGeneratingListener() {
+		this(Paths.get(".").toAbsolutePath());
+	}
+
+	OpenTestReportGeneratingListener(Path workingDir) {
+		this.workingDir = workingDir;
 	}
 
 	@Override
@@ -160,7 +168,7 @@ private void reportInfrastructure() {
 		});
 	}
 
-	private static void addGitInfo(Infrastructure infrastructure) {
+	private void addGitInfo(Infrastructure infrastructure) {
 		boolean gitInstalled = exec("git", "--version").isPresent();
 		if (gitInstalled) {
 			exec("git", "config", "--get", "remote.origin.url") //
@@ -179,7 +187,7 @@ private static void addGitInfo(Infrastructure infrastructure) {
 		}
 	}
 
-	static Optional<String> exec(String... args) {
+	Optional<String> exec(String... args) {
 
 		Process process = startProcess(args);
 
@@ -211,10 +219,10 @@ private static BufferedReader newBufferedReader(InputStream stream) {
 		return new BufferedReader(new InputStreamReader(stream, Charset.defaultCharset()));
 	}
 
-	private static Process startProcess(String[] args) {
+	private Process startProcess(String[] command) {
 		Process process;
 		try {
-			process = Runtime.getRuntime().exec(args);
+			process = new ProcessBuilder().directory(workingDir.toFile()).command(command).start();
 		}
 		catch (IOException e) {
 			throw new UncheckedIOException("Failed to start process", e);

platform-tests/src/test/java/org/junit/platform/reporting/open/xml/OpenTestReportGeneratingListenerTests.java

@@ -79,10 +79,6 @@ void writesValidXmlReport() throws Exception {
 				        <java:javaVersion>${xmlunit.ignore}</java:javaVersion>
 				        <java:fileEncoding>${xmlunit.ignore}</java:fileEncoding>
 				        <java:heapSize max="${xmlunit.isNumber}"/>
-				        <git:repository originUrl="${xmlunit.matchesRegex#(git@|https://).+#}"/>
-				        <git:branch>${xmlunit.ignore}</git:branch>
-				        <git:commit>${xmlunit.matchesRegex#[0-9a-f]{40}#}</git:commit>
-				        <git:status clean="${xmlunit.ignore}">${xmlunit.ignore}</git:status>
 				    </infrastructure>
 				    <e:started id="1" name="dummy" time="${xmlunit.isDateTime}">
 				        <metadata>
@@ -137,7 +133,7 @@ private void executeTests(TestEngine engine) {
 				.configurationParameter(OUTPUT_DIR_PROPERTY_NAME,
 					tempDirectory.resolve("junit-" + OUTPUT_DIR_UNIQUE_NUMBER_PLACEHOLDER).toString()) //
 				.build();
-		createLauncher(engine).execute(build, new OpenTestReportGeneratingListener());
+		createLauncher(engine).execute(build, new OpenTestReportGeneratingListener(tempDirectory));
 	}
 
 }