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>
