Include media type in Open Test Report XML output

Author: marcphilipp

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

@@ -377,7 +377,10 @@ public void fileEntryPublished(TestIdentifier testIdentifier, FileEntry entry) {
 		String id = inProgressIds.get(testIdentifier.getUniqueIdObject());
 		eventsFileWriter.append(reported(id, Instant.now()), //
 			reported -> reported.append(attachments(), attachments -> attachments.append(file(entry.getTimestamp()), //
-				file -> file.withPath(outputDir.relativize(entry.getPath()).toString()))));
+				file -> {
+					file.withPath(outputDir.relativize(entry.getPath()).toString());
+					entry.getMediaType().ifPresent(file::withMediaType);
+				})));
 	}
 
 	@Override

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

@@ -37,8 +37,10 @@
 import org.junit.jupiter.api.io.TempDir;
 import org.junit.jupiter.params.ParameterizedTest;
 import org.junit.jupiter.params.provider.ValueSource;
+import org.junit.platform.commons.util.ExceptionUtils;
 import org.junit.platform.engine.TestEngine;
 import org.junit.platform.engine.UniqueId;
+import org.junit.platform.engine.reporting.FileEntry;
 import org.junit.platform.engine.reporting.ReportEntry;
 import org.junit.platform.engine.support.hierarchical.DemoHierarchicalTestEngine;
 import org.junit.platform.tests.process.ProcessResult;
@@ -78,10 +80,17 @@ void restoreSystemPrintStreams() {
 	void writesValidXmlReport(@TempDir Path tempDirectory) throws Exception {
 		var engine = new DemoHierarchicalTestEngine("dummy");
 		engine.addTest("failingTest", "display<-->Name 😎", (context, descriptor) -> {
-			var listener = context.request.getEngineExecutionListener();
-			listener.reportingEntryPublished(descriptor, ReportEntry.from("key", "value"));
-			System.out.println("Hello, stdout!");
-			System.err.println("Hello, stderr!");
+			try {
+				var listener = context.request.getEngineExecutionListener();
+				listener.reportingEntryPublished(descriptor, ReportEntry.from("key", "value"));
+				listener.fileEntryPublished(descriptor, FileEntry.from(
+					Files.writeString(tempDirectory.resolve("test.txt"), "Hello, world!"), "text/plain"));
+				System.out.println("Hello, stdout!");
+				System.err.println("Hello, stderr!");
+			}
+			catch (Throwable t) {
+				throw ExceptionUtils.throwAsUncheckedException(t);
+			}
 			fail("failure message");
 		});
 
@@ -130,6 +139,11 @@ void writesValidXmlReport(@TempDir Path tempDirectory) throws Exception {
 				            </data>
 				        </attachments>
 				    </e:reported>
+				    <e:reported id="2" time="${xmlunit.isDateTime}">
+				        <attachments>
+				            <file time="${xmlunit.isDateTime}" path="../test.txt" mediaType="text/plain" />
+				        </attachments>
+				    </e:reported>
 				    <e:reported id="2" time="${xmlunit.isDateTime}">
 				        <attachments>
 				            <output time="${xmlunit.isDateTime}" source="stdout"><![CDATA[Hello, stdout!]]></output>