From ca55545c820373ac6383b595ced31b84988aef82 Mon Sep 17 00:00:00 2001
From: Matthew Jason Benson <mbenson@apache.org>
Date: Tue, 26 Sep 2006 22:25:00 +0000
Subject: [PATCH] a little more work

git-svn-id: https://svn.apache.org/repos/asf/ant/core/trunk@450238 13f79535-47bb-0310-9956-ffa450edef68
---
 build.xml | 28 +++++++++++++---------------
 1 file changed, 13 insertions(+), 15 deletions(-)

diff --git a/build.xml b/build.xml
index b392f78eb..02e35d13c 100644
--- a/build.xml
+++ b/build.xml
@@ -1549,24 +1549,22 @@
        ===================================================================
   -->
 
-  <target name="check-failed">
-    <condition property="tests.failed">
-      <or>
-        <isset property="junit.failed" />
-        <isset property="antunit.failed" />
-      </or>
-    </condition>
+  <target name="tests-failed">
+    <fail>Unit tests failed; see ${build.junit.reports} / ${antunit.reports}
+      <condition>
+        <or>
+          <isset property="junit.failed" />
+          <isset property="antunit.failed" />
+        </or>
+      </condition>
+    </fail>
   </target>
 
-  <target name="test" depends="junit-report,antunit-report,check-failed"
-          description="--> run unit tests / reports">
-    <fail if="tests.failed">Unit tests failed; see ${build.junit.reports} / ${antunit.reports}</fail>
-  </target>
+  <target name="test" depends="junit-report,antunit-report,tests-failed"
+          description="--> run unit tests and reports" />
 
-  <target name="run-tests" depends="junit-tests,antunit-tests,check-failed"
-          description="--> run unit tests">
-    <fail if="tests.failed">Unit tests failed; see ${build.junit.reports} / ${antunit.reports}</fail>
-  </target>
+  <target name="run-tests" depends="junit-tests,antunit-tests,tests-failed"
+          description="--> run unit tests without reports" />
 
   <target name="test-init" depends="probe-offline,check_for_optional_packages">
     <macrodef name="test-junit">