Then I will add that again :)
Or would it be better to add the if/unless to <patternset>, I think so?

Jan

> -----Original Message-----
> From: Peter Reilly [mailto:[EMAIL PROTECTED]
> Sent: Friday, February 27, 2004 1:15 PM
> To: Ant Developers List
> Subject: Re: Ant build.xml: run "teststhatfail"
> 
> 
> [EMAIL PROTECTED] wrote:
> 
> >The target "run-tests" excludes the patternset 
> "teststhatfail" from being 
> >executed. The comment sais, that I can force running them by 
> setting the
> >property "run.failing.tests". But I havenīt found any 
> reference to that
> >name inside the build file.
> >
> >Is there something build-in in the <junit> task or is there something
> >missing in
> >the buildfile (e.g. the unless-clause in the <exclude>s on 
> line 330/331)?
> >  
> >
> Looks that that should be the case. Looking at the history of 
> build.xml,
> at one stage the testthatfailed reference had this unless class in the
> excludes,
> http://cvs.apache.org/viewcvs.cgi/*checkout*/ant/build.xml?con
tent-type=text%2Fplain&rev=1.290
 and at one stage there was no tests that failed, soon afterwards,
one was added back in (the beanshell one) but this did not have the excludes
clause.

Peter

>
>Jan
>
>
>
> 329:  <patternset id="teststhatfail">
> 330:    <exclude name="${optional.package}/BeanShellScriptTest.java"/>
> 331:    <exclude name="${optional.package}/jdepend/JDependTest.java"/>
> 332:  </patternset>
>
>1414:   <target name="run-tests" ...
>1426:     <junit ...
>1440:      <batchtest>
>1441:        <fileset dir="${tests.dir}">
>1442:          <include name="**/*Test*"/>
>1468:          <!-- failing tests excluded unless run.failing.tests is set
>-->
>1469:          <patternset refid="teststhatfail"/>
>
>
>
>
>---------------------------------------------------------------------
>To unsubscribe, e-mail: [EMAIL PROTECTED]
>For additional commands, e-mail: [EMAIL PROTECTED]
>
>
>
>  
>


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

Reply via email to