From ac6688de5926587853c2a2bc9aad21fa4c9dacd2 Mon Sep 17 00:00:00 2001
From: "Thomas L. Kjeldsen" <tlk@closureconsulting.com>
Date: Fri, 31 Jan 2020 16:00:04 +0100
Subject: [PATCH] Treat JUnit errors as failures to increase visibility

Before this change `cd app; ant test` would return "BUILD SUCCESS"
even when unit tests were returning errors.
---
 app/build.xml | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/app/build.xml b/app/build.xml
index 411fe0f6494..a2926990222 100644
--- a/app/build.xml
+++ b/app/build.xml
@@ -109,7 +109,7 @@
       <fileset dir="test" includes="**/*.pac" />
     </copy>
 
-    <junit printsummary="yes" dir="${work.dir}" fork="true">
+    <junit printsummary="yes" dir="${work.dir}" fork="true" showoutput="yes" failureproperty="test.failed">
       <jvmarg value="-Djava.library.path=${java.additional.library.path}"/>
       <jvmarg value="-DWORK_DIR=."/>
       <jvmarg value="-ea"/>
@@ -131,6 +131,7 @@
       </batchtest>
     </junit>
 
+    <fail if="test.failed"/>
   </target>
 
   <target name="build" depends="compile" description="Build PDE">