Re: svn commit: r1133353 - /ofbiz/trunk/build.xml

[Adrian Crum-3]

"run-tests-with-corbertura" should be "run-tests-with-cobertura"

Spelling error.


-Adrian

On 6/8/2011 12:44 PM, [hidden email] wrote:

> Author: erwan
> Date: Wed Jun  8 11:44:03 2011
> New Revision: 1133353
>
> URL: http://svn.apache.org/viewvc?rev=1133353&view=rev
> Log:
> Making run-tests not using cobertura, and creating a new separate target if you want code coverage: run-tests-with-cobertura
>
> Modified:
>      ofbiz/trunk/build.xml
>
> Modified: ofbiz/trunk/build.xml
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/build.xml?rev=1133353&r1=1133352&r2=1133353&view=diff
> ==============================================================================
> --- ofbiz/trunk/build.xml (original)
> +++ ofbiz/trunk/build.xml Wed Jun  8 11:44:03 2011
> @@ -75,6 +75,7 @@ under the License.
>           <antcall target="clean-catalina"/>
>           <antcall target="clean-cache"/>
>           <antcall target="clean-tempfiles"/>
> +<antcall target="clean-downloads"/>
>           <antcall target="clean"/>
>       </target>
>
> @@ -679,8 +680,8 @@ under the License.
>               <jvmarg value="-Xrunjdwp:transport=dt_socket,server=y,suspend=n,address=8091"/>
>           </java>
>       </target>
> -<target name="run-tests" depends="download-cobertura, build"
> -          description="Run OFBiz default tests, execute ant run-install before and see results in runtime/logs/test-results/html/all-tests.html. You will need a valid Internet connection to download cobertura">
> +<target name="run-tests" depends="build"
> +          description="Run OFBiz default tests, execute ant run-install 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}"/>
>               <jvmarg value="${memory.max.param}"/>
> @@ -704,6 +705,9 @@ under the License.
>           </fail>
>       </target>
>
> +<target name="run-tests-with-corbertura" depends="download-cobertura, build, run-tests"
> +        description="Same as run-tests but downloads cobertura and performs code coverage. You will need a valid Internet connection to download cobertura"/>
> +
>       <target name="_check-separated-tests-already-setup">
>           <available file="runtime/test-list-build.xml" property="_separated-tests-already-setup"/>
>       </target>
>
>

[Erwan de FERRIERES]

Le 08/06/2011 13:45, Adrian Crum a écrit :
> "run-tests-with-corbertura" should be "run-tests-with-cobertura"
>
> Spelling error.
>
>
> -Adrian
>
Nice catch, thanks, I'm correcting right now !

--
Erwan de FERRIERES
www.nereide.biz