From: Dominik Stadler <centic@apache.org>
Date: Sun, 11 Dec 2016 21:15:31 +0000 (+0000)
Subject: Revert "Fail after running tests if halt.on.test.failure is set and tests are failing"
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=a5f60dbe27dc7ad14ee80e337ba82d357f11a489;p=poi.git

Revert "Fail after running tests if halt.on.test.failure is set and tests are failing"

This reverts commit 1fdfcd3d44e71a2b919f045bad849b9d38150aea.

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1773688 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/build.xml b/build.xml
index f979fa1ecf..340781a395 100644
--- a/build.xml
+++ b/build.xml
@@ -1276,8 +1276,6 @@ 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">
@@ -1387,8 +1385,6 @@ 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">
@@ -1432,9 +1428,6 @@ 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">
@@ -1456,8 +1449,6 @@ under the License.
                   </batchtest>
               </junit>
             </jacoco:coverage>
-        
-			<fail if="ooxml.xmlsec.test.failed" message="At least one test failed"/>
         </sequential>
     </macrodef>
 
@@ -1557,8 +1548,6 @@ 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">
@@ -1652,8 +1641,6 @@ 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 -->