maven-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ltheu...@apache.org
Subject svn commit: r388592 - in /maven/maven-1/plugins/trunk/test: plugin.jelly xdocs/changes.xml
Date Fri, 24 Mar 2006 18:05:47 GMT
Author: ltheussl
Date: Fri Mar 24 10:05:45 2006
New Revision: 388592

URL: http://svn.apache.org/viewcvs?rev=388592&view=rev
Log:
PR: MPTEST-36
Display a warning if some tests do not pass
(and maven.test.failure.ignore=true).

Modified:
    maven/maven-1/plugins/trunk/test/plugin.jelly
    maven/maven-1/plugins/trunk/test/xdocs/changes.xml

Modified: maven/maven-1/plugins/trunk/test/plugin.jelly
URL: http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/test/plugin.jelly?rev=388592&r1=388591&r2=388592&view=diff
==============================================================================
--- maven/maven-1/plugins/trunk/test/plugin.jelly (original)
+++ maven/maven-1/plugins/trunk/test/plugin.jelly Fri Mar 24 10:05:45 2006
@@ -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}">
@@ -425,6 +448,16 @@
             </batchtest>
 
           </junit>
+
+          <j:if test="${_testfailure}">
+             <echo>
+             ==========================================================
+
+               WARNING:  There were test failures!
+
+             ==========================================================
+             </echo>
+           </j:if>
 
         </j:otherwise>
       </j:choose>

Modified: maven/maven-1/plugins/trunk/test/xdocs/changes.xml
URL: http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/test/xdocs/changes.xml?rev=388592&r1=388591&r2=388592&view=diff
==============================================================================
--- maven/maven-1/plugins/trunk/test/xdocs/changes.xml (original)
+++ maven/maven-1/plugins/trunk/test/xdocs/changes.xml Fri Mar 24 10:05:45 2006
@@ -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>



Mime
View raw message