From ffbf62b5d0008c6f832c480d9587097e7db5b0fc Mon Sep 17 00:00:00 2001
From: ltheussl <ltheussl@13f79535-47bb-0310-9956-ffa450edef68>
Date: Fri, 24 Mar 2006 18:05:45 +0000
Subject: [PATCH] PR: MPTEST-36 Display a warning if some tests do not pass
 (and maven.test.failure.ignore=true).

git-svn-id: https://svn.apache.org/repos/asf/maven/maven-1/plugins/trunk@388592 13f79535-47bb-0310-9956-ffa450edef68
---
 test/plugin.jelly      | 33 +++++++++++++++++++++++++++++++++
 test/xdocs/changes.xml |  1 +
 2 files changed, 34 insertions(+)

diff --git a/test/plugin.jelly b/test/plugin.jelly
index 241ba76a..14339fe0 100644
--- a/test/plugin.jelly
+++ b/test/plugin.jelly
@@ -142,6 +142,7 @@
       <junit printSummary="${maven.junit.printSummary}"
              haltonfailure="${_haltonfailure}"
              haltonerror="${_haltonerror}"
+             failureProperty="_testfailure"
              fork="${maven.junit.fork}"
              forkmode="${maven.junit.forkmode}">
 
@@ -193,6 +194,16 @@
         </batchtest>
       </junit>
 
+        <j:if test="${_testfailure}">
+           <echo>
+           ==========================================================
+
+             WARNING:  There were test failures!
+
+           ==========================================================
+           </echo>
+         </j:if>
+
     </j:if>
     
     <j:if test="${!unitTestSourcesPresent}">
@@ -274,6 +285,7 @@
           <junit printSummary="${maven.junit.printSummary}"
                  haltonfailure="${_haltonfailure}"
                  haltonerror="${_haltonerror}"
+                 failureProperty="_testfailure"
                  fork="${maven.junit.fork}"
                  dir="${maven.junit.dir}"
                  forkmode="${maven.junit.forkmode}">
@@ -315,6 +327,16 @@
 
           </junit>
 
+          <j:if test="${_testfailure}">
+             <echo>
+             ==========================================================
+
+               WARNING:  There were test failures!
+
+             ==========================================================
+             </echo>
+           </j:if>
+
         </j:otherwise>
       </j:choose>
     </j:if>
@@ -367,6 +389,7 @@
           <junit printSummary="${maven.junit.printSummary}"
                  haltonfailure="${_haltonfailure}"
                  haltonerror="${_haltonerror}"
+                 failureProperty="_testfailure"
                  fork="${maven.junit.fork}"
                  dir="${maven.junit.dir}"
                  forkmode="${maven.junit.forkmode}">
@@ -426,6 +449,16 @@
 
           </junit>
 
+          <j:if test="${_testfailure}">
+             <echo>
+             ==========================================================
+
+               WARNING:  There were test failures!
+
+             ==========================================================
+             </echo>
+           </j:if>
+
         </j:otherwise>
       </j:choose>
     </j:if>
diff --git a/test/xdocs/changes.xml b/test/xdocs/changes.xml
index f018d433..c996a37f 100644
--- a/test/xdocs/changes.xml
+++ b/test/xdocs/changes.xml
@@ -24,6 +24,7 @@
   </properties>
   <body>
     <release version="1.8-SNAPSHOT" date="in SVN">
+      <action dev="ltheussl" type="add" issue="MPTEST-36">Display a warning if some tests do not pass (and <code>maven.test.failure.ignore=true</code>).</action>
       <action dev="ltheussl" type="add" issue="MPTEST-10">Add timeouts to the test plugin, new property <code>maven.junit.timeout</code>.</action>
       <action dev="ltheussl" type="add" issue="MPTEST-58" due-to="Mauro Botelho">Add ability to fail the build on test errors, new property <code>maven.test.error.ignore</code>.</action>
       <action dev="ltheussl" type="fix" issue="MPTEST-46">Do not execute <code>test:test-resources</code> and <code>test:compile</code> unless tests will be run.</action>