junitlauncher - Support useFile attribute for listeners

apache · Nov 16, 2021 · 372bc1a · 372bc1a
1 parent c8bc470
commit 372bc1a
Show file tree

Hide file tree

Showing 3 changed files with 32 additions and 5 deletions.
diff --git a/manual/Tasks/junitlauncher.html b/manual/Tasks/junitlauncher.html
@@ -240,7 +240,7 @@ <h4 id="nested-classpath">classpath</h4>
             &lt;/classpath&gt;
             &lt;testclasses outputdir="${output.dir}"&gt;
                 &lt;fileset dir="${build.classes.dir}"/&gt;
-                &lt;listener type="legacy-brief" sendSysOut="true"/&gt;
+                &lt;listener type="legacy-brief" sendSysOut="true" useFile="false"/&gt;
                 &lt;listener type="legacy-xml" sendSysErr="true" sendSysOut="true"/&gt;
 
             &lt;/testclasses&gt;
@@ -380,6 +380,15 @@ <h5>Test result formatter</h5>
         </td>
         <td>No</td>
     </tr>
+    <tr>
+        <td>useFile</td>
+        <td>If set to <q>true</q> then the listener's output will be saved to a file. Otherwise, the output will be sent
+            to <code>stdout</code> and <code>outputDir</code>, <code>resultFile</code>, <code>extension</code>
+            attributes will be ignored.
+            <p><em>Since Ant 1.10.13</em></p>
+        </td>
+        <td>No; defaults to <q>true</q></td>
+    </tr>
     <tr>
         <td>sendSysOut</td>
         <td>If set to <q>true</q> then the listener will be passed the <code>stdout</code> content

diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/LauncherSupport.java b/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/LauncherSupport.java
@@ -256,10 +256,14 @@ private void setupResultFormatter(final TestRequest testRequest, final ListenerD
         // set the destination output stream for writing out the formatted result
         final java.nio.file.Path resultOutputFile = getListenerOutputFile(testRequest, formatterDefinition);
         try {
-            final OutputStream resultOutputStream = Files.newOutputStream(resultOutputFile);
-            // enroll the output stream to be closed when the execution of the TestRequest completes
-            testRequest.closeUponCompletion(resultOutputStream);
-            resultFormatter.setDestination(new KeepAliveOutputStream(resultOutputStream));
+            if (formatterDefinition.shouldUseFile()) {
+                final OutputStream resultOutputStream = Files.newOutputStream(resultOutputFile);
+                // enroll the output stream to be closed when the execution of the TestRequest completes
+                testRequest.closeUponCompletion(resultOutputStream);
+                resultFormatter.setDestination(new KeepAliveOutputStream(resultOutputStream));
+            } else {
+                resultFormatter.setDestination(new KeepAliveOutputStream(System.out));
+            }
         } catch (IOException e) {
             throw new BuildException(e);
         }

diff --git a/...ain/org/apache/tools/ant/taskdefs/optional/junitlauncher/confined/ListenerDefinition.java b/...ain/org/apache/tools/ant/taskdefs/optional/junitlauncher/confined/ListenerDefinition.java
@@ -50,6 +50,7 @@ public class ListenerDefinition {
     private String className;
     private String resultFile;
     private String extension = "txt";
+    private boolean useFile = true;
     private boolean sendSysOut;
     private boolean sendSysErr;
     private String outputDir;
@@ -123,6 +124,19 @@ public String getExtension() {
         return extension;
     }
 
+    /**
+     * Sets whether the formatter should log to a file.
+     * @param useFile if true use a file, if false send to standard out.
+     * @since Ant 1.10.13
+     */
+    public void setUseFile(boolean useFile) {
+        this.useFile = useFile;
+    }
+
+    public boolean shouldUseFile() {
+        return useFile;
+    }
+
     public void setSendSysOut(final boolean sendSysOut) {
         this.sendSysOut = sendSysOut;
     }