-1

If you are running tests, then most likely you are developing new code. So, if the tests fail, I should be able to make corrections to my source code and then run the tests again without having to insert a build step.

-Adrian

On 4/23/2012 2:21 PM, Jacopo Cappellato wrote:
What do you think? In order to run most of the tests successfully you need to run 
load-data that already depends on "build"...

Here is the code change I would like to commit:

Index: build.xml
===================================================================
--- build.xml   (revision 1328357)
+++ build.xml   (working copy)
@@ -868,7 +868,7 @@
          <antcall target="load-admin-user-login"/>
      </target>

-<target name="run-tests" depends="build"
+<target name="run-tests"
            description="Run OFBiz default tests; you have to manually execute 'ant 
load-demo' before and see results in runtime/logs/test-results/html/all-tests.html.">
          <java jar="ofbiz.jar" fork="true" resultproperty="test.result">
              <jvmarg value="${memory.initial.param}"/>
@@ -910,7 +910,7 @@
              <env key="LC_ALL" value="C"/>
          </java>
      </target>
-<target name="run-test" depends="build"
+<target name="run-test"
            description="Run a single test, syntax eg: ant run-test 
-Dtest.component=service -Dtest.case=service-soap-tests">
          <fail unless="test.component">test.component is a required parameter: 
-Dtest.component=componentname</fail>
          <fail unless="test.case">test.case is a required parameter: 
-Dtest.case=testcasename</fail>
@@ -936,7 +936,7 @@
              </condition>
          </fail>
      </target>
-<target name="run-test-suite" depends="build"
+<target name="run-test-suite"
              description="Run a single test suite, syntax eg: ant run-test-suite 
-Dtest.component=mycomponent -Dtest.suiteName=mytests">
          <fail unless="test.component">test.component is a required parameter: 
-Dtest.component=componentname</fail>
          <fail unless="test.suiteName">test.suiteName is a required parameter: 
-Dtest.suiteName=testsuitename</fail>

Jacopo

Reply via email to