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

@@ -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>

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>