From edad691d59acabb7c361ab72665ea4afdf77fb01 Mon Sep 17 00:00:00 2001
From: Matthew Jason Benson <mbenson@apache.org>
Date: Fri, 22 Sep 2006 21:11:30 +0000
Subject: [PATCH] disallow mixing of old-style "testcase" property with either
 of antunit.testcase, junit.testcase, or both.  Additionally, allow files
 simply named test.xml in addition to *-test.xml .

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

diff --git a/build.xml b/build.xml
index b3678947a..793f8049d 100644
--- a/build.xml
+++ b/build.xml
@@ -1592,6 +1592,18 @@
       </sequential>
     </macrodef>
 
+    <fail>"testcase" cannot be specified with "junit.testcase" or "antunit.testcase".
+      <condition>
+        <and>
+          <isset property="testcase" />
+          <or>
+            <isset property="antunit.testcase" />
+            <isset property="junit.testcase" />
+          </or>
+        </and>
+      </condition>
+    </fail>
+
     <condition property="antunit.testcase" value="${testcase}">
       <available file="${src.antunit}/${testcase}" />
     </condition>
@@ -1600,6 +1612,20 @@
       <available classname="${testcase}" classpathref="tests-classpath" />
     </condition>
 
+    <fail>Cannot locate test ${testcase}
+      <condition>
+        <and>
+          <isset property="testcase" />
+          <not>
+            <or>
+              <isset property="antunit.testcase" />
+              <isset property="junit.testcase" />
+            </or>
+          </not>
+        </and>
+      </condition>
+    </fail>
+
     <condition property="run.junit">
       <and>
         <not><equals arg1="${testcase}" arg2="${antunit.testcase}" /></not>
@@ -1830,7 +1856,7 @@
           if="run.antunit">
 
     <condition property="antunit.includes" value="${antunit.testcase}"
-               else="**/*-test.xml">
+               else="**/test.xml,*-test.xml">
       <isset property="${antunit.testcase}" />
     </condition>