On Wed, 2003-11-19 at 12:30, robert burrell donkin wrote:
> hi simon
> 
> just to let you know that i would have committed your patch but there's 
> a broken lock in cvs :(

Thanks Robert.

Since the new rule is considered acceptable, here's a patch that adds
the tests to build.xml. Hope that CVS issue doesn't cause you too much
headache..

Cheers,

Simon
Index: build.xml
===================================================================
RCS file: /home/cvspublic/jakarta-commons/digester/build.xml,v
retrieving revision 1.48
diff -u -r1.48 build.xml
--- build.xml	18 Oct 2003 13:40:46 -0000	1.48
+++ build.xml	19 Nov 2003 03:09:15 -0000
@@ -272,6 +272,7 @@
                                test.factory,
                                test.regex,
                                test.wdrules,
+                               test.setnestedprops,
                                test.plugins
                               "
    description="Run all unit test cases">
@@ -477,6 +478,20 @@
       <classpath refid="test.classpath"/>
     </java>
   </target>
+
+  <target name="test.setnestedprops" depends="compile.tests"
+          description="Run SetNestedPropertiesRule tests ...">
+    <echo message="Running SetNestedPropertiesRule tests ..."/>
+    <condition property="logopt" value="${log.factoryopt}=${log.class}">
+      <istrue value="${suppressLogOutputDuringTests}"/>
+    </condition>
+    <java classname="${test.runner}" fork="yes"
+        failonerror="${test.failonerror}">
+      <jvmarg value="-D${logopt}"/>
+      <arg value="org.apache.commons.digester.SetNestedPropertiesRuleTestCase"/>
+      <classpath refid="test.classpath"/>
+    </java>
+  </target> 
 
   <target name="test.plugins" depends="compile.tests"
           description="Run Plugins tests ...">

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to