From ca6cea18634eae814a1aed479548de5751859fd5 Mon Sep 17 00:00:00 2001
From: Dominik Stadler <centic@apache.org>
Date: Sun, 11 Dec 2016 20:58:16 +0000
Subject: [PATCH] Fail after running tests if halt.on.test.failure is set and
 tests are failing

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

diff --git a/build.xml b/build.xml
index 340781a39..f979fa1ec 100644
--- a/build.xml
+++ b/build.xml
@@ -1276,6 +1276,8 @@ under the License.
         </jacoco:coverage>
         <delete file="${main.testokfile}"/>
         <antcall target="-test-main-write-testfile"/>
+        
+        <fail if="main.test.failed" message="At least one test failed"/>
     </target>
 
     <target name="test-report" depends="init">
@@ -1385,6 +1387,8 @@ under the License.
         </jacoco:coverage>
         <delete file="${scratchpad.testokfile}"/>
         <antcall target="-test-scratchpad-write-testfile"/>
+        
+        <fail if="scratchpad.test.failed" message="At least one test failed"/>
     </target>
 
     <target name="-test-scratchpad-write-testfile" unless="scratchpad.test.failed">
@@ -1428,6 +1432,9 @@ under the License.
                   </batchtest>
               </junit>
             </jacoco:coverage>
+        
+			<fail if="ooxml.test.failed" message="At least one test failed"/>
+            
             <jacoco:coverage enabled="${coverage.enabled}" excludes="${coverage.excludes}" destfile="build/jacoco-@{type}-xmlsec.exec">
               <junit printsummary="yes" fork="yes" forkmode="once" haltonfailure="${halt.on.test.failure}"
                      failureproperty="ooxml.xmlsec.test.failed">
@@ -1449,6 +1456,8 @@ under the License.
                   </batchtest>
               </junit>
             </jacoco:coverage>
+        
+			<fail if="ooxml.xmlsec.test.failed" message="At least one test failed"/>
         </sequential>
     </macrodef>
 
@@ -1548,6 +1557,8 @@ under the License.
         </jacoco:coverage>
         <delete file="${integration.testokfile}"/>
         <antcall target="-test-integration-write-testfile"/>
+        
+        <fail if="integration.test.failed" message="At least one test failed"/>
     </target>
 
     <target name="-test-integration-write-testfile" unless="integration.test.failed">
@@ -1641,6 +1652,8 @@ under the License.
         </jacoco:coverage>
         <delete file="${excelant.testokfile}"/>
         <antcall target="-test-excelant-write-testfile"/>
+        
+        <fail if="excelant.test.failed" message="At least one test failed"/>
     </target>
 
     <!-- GENERATE DOCUMENTATION -->