From b4f39b0f732b0ec1d120f41d113feb9266e9280d Mon Sep 17 00:00:00 2001
From: kkolinko <kkolinko@13f79535-47bb-0310-9956-ffa450edef68>
Date: Tue, 18 May 2010 01:07:10 +0000
Subject: [PATCH] Do not abort testing after the first failure, but report
 about existence of failures after all the tests were run.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@945446 13f79535-47bb-0310-9956-ffa450edef68
---
 build.xml | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/build.xml b/build.xml
index bf7df1a06..c188c5e19 100644
--- a/build.xml
+++ b/build.xml
@@ -919,7 +919,9 @@
 
       <!-- If test.entry is defined, run a single test, otherwise run all valid tests -->
       <test todir="${tomcat.build}/logs" name="${test.entry}" if="test.entry"/>
-      <batchtest todir="${tomcat.build}/logs" haltonerror="true" haltonfailure="true" unless="test.entry">
+      <batchtest todir="${tomcat.build}/logs" unless="test.entry"
+        errorproperty="test.result.error" failureproperty="test.result.failure" 
+        >
         <fileset dir="test" >
           <!-- Include all by default -->
           <include name="**/Test*.java" />
@@ -930,6 +932,9 @@
         </fileset>
       </batchtest>
     </junit>
+
+    <fail if="test.result.error" message="Some tests completed with an Error. See ${tomcat.build}/logs for details." />
+    <fail if="test.result.failure" message="Some tests completed with a Failure. See ${tomcat.build}/logs for details." />
   </target>
 
   <target name="extras-prepare" >
-- 
2.11.0