Return-Path: 
 <commits-return-4781-apmail-maven-commits-archive=maven.apache.org@maven.apache.org>
Delivered-To: apmail-maven-commits-archive@www.apache.org
Received: (qmail 86486 invoked from network); 24 Mar 2006 18:06:10 -0000
Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199)
  by minotaur.apache.org with SMTP; 24 Mar 2006 18:06:10 -0000
Received: (qmail 63967 invoked by uid 500); 24 Mar 2006 18:06:09 -0000
Delivered-To: apmail-maven-commits-archive@maven.apache.org
Received: (qmail 63937 invoked by uid 500); 24 Mar 2006 18:06:08 -0000
Mailing-List: contact commits-help@maven.apache.org; run by ezmlm
Precedence: bulk
List-Help: <mailto:commits-help@maven.apache.org>
List-Unsubscribe: <mailto:commits-unsubscribe@maven.apache.org>
List-Post: <mailto:commits@maven.apache.org>
List-Id: <commits.maven.apache.org>
Reply-To: dev@maven.apache.org
Delivered-To: mailing list commits@maven.apache.org
Received: (qmail 63926 invoked by uid 99); 24 Mar 2006 18:06:08 -0000
Received: from asf.osuosl.org (HELO asf.osuosl.org) (140.211.166.49)
    by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 24 Mar 2006 10:06:08 -0800
X-ASF-Spam-Status: No, hits=-9.4 required=10.0
	tests=ALL_TRUSTED,NO_REAL_NAME
X-Spam-Check-By: apache.org
Received: from [209.237.227.194] (HELO minotaur.apache.org) (209.237.227.194)
    by apache.org (qpsmtpd/0.29) with SMTP; Fri, 24 Mar 2006 10:06:08 -0800
Received: (qmail 86343 invoked by uid 65534); 24 Mar 2006 18:05:47 -0000
Message-ID: <20060324180547.86342.qmail@minotaur.apache.org>
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: svn commit: r388592 - in /maven/maven-1/plugins/trunk/test:
 plugin.jelly
 xdocs/changes.xml
Date: Fri, 24 Mar 2006 18:05:47 -0000
To: commits@maven.apache.org
From: ltheussl@apache.org
X-Mailer: svnmailer-1.0.7
X-Virus-Checked: Checked by ClamAV on apache.org
X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N

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>