[tomcat8] 10/10: Test failures are no longer ignored and now stop the build

[Emmanuel Bourg]

This is an automated email from the git hooks/post-receive script.

ebourg-guest pushed a commit to branch jessie
in repository tomcat8.

commit ef5ce6251a42ad98affde274a1b7c22d9b9fe0de
Author: Emmanuel Bourg <ebourg at apache.org>
Date:   Wed Nov 23 19:48:33 2016 +0100

    Test failures are no longer ignored and now stop the build
---
 debian/changelog                             |  1 +
 debian/patches/0005-skip-test-failures.patch | 13 +------------
 2 files changed, 2 insertions(+), 12 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index 95a7a32..6343228 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -25,6 +25,7 @@ tomcat8 (8.0.14-1+deb8u5) UNRELEASED; urgency=medium
   * Added asm-all.jar to the test classpath to fix TestWebappServiceLoader
   * Fixed a test failure in the new TestNamingContext test added with the fix
     for CVE-2016-6797
+  * Test failures are no longer ignored and now stop the build
 
  -- Emmanuel Bourg <ebourg at apache.org>  Tue, 22 Nov 2016 23:21:56 +0100
 
diff --git a/debian/patches/0005-skip-test-failures.patch b/debian/patches/0005-skip-test-failures.patch
index 4324b72..5bc9262 100644
--- a/debian/patches/0005-skip-test-failures.patch
+++ b/debian/patches/0005-skip-test-failures.patch
@@ -1,19 +1,8 @@
-Description: Ignore the failing tests and print only the summary
+Description: Print only the summary of the tests
 Author: Emmanuel Bourg <ebourg at apache.org>
 Forwarded: not-needed
 --- a/build.xml
 +++ b/build.xml
-@@ -1339,8 +1339,10 @@
- 
-   <target name="test" description="Runs the JUnit test cases"
-           depends="test-bio,test-nio,test-nio2,test-apr,cobertura-report" >
-+    <!--
-     <fail if="test.result.error" message='Some tests completed with an Error. See ${tomcat.build}/logs for details, search for "FAILED".' />
-     <fail if="test.result.failure" message='Some tests completed with a Failure. See ${tomcat.build}/logs for details, search for "FAILED".' />
-+    -->
-   </target>
- 
-   <target name="test-bio" description="Runs the JUnit test cases for BIO. Does not stop on errors."
 @@ -1399,7 +1401,7 @@
  
      <sequential>

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-java/tomcat8.git