On Mon, Feb 1, 2010 at 11:17 AM, sebb <[email protected]> wrote:
> On 01/02/2010, [email protected] <[email protected]> wrote:
>> Author: niallp
>>  Date: Mon Feb  1 10:43:00 2010
>>  New Revision: 905240
>>
>>  URL: http://svn.apache.org/viewvc?rev=905240&view=rev
>>  Log:
>>  Configure surefire plugin to run *Test.java and not test suites (at least 
>> one test was missing because it wasn't in a suite)
>>
>>  Modified:
>>     commons/proper/lang/branches/LANG_2_X/pom.xml
>>
>>  Modified: commons/proper/lang/branches/LANG_2_X/pom.xml
>>  URL: 
>> http://svn.apache.org/viewvc/commons/proper/lang/branches/LANG_2_X/pom.xml?rev=905240&r1=905239&r2=905240&view=diff
>>  ==============================================================================
>>  --- commons/proper/lang/branches/LANG_2_X/pom.xml (original)
>>  +++ commons/proper/lang/branches/LANG_2_X/pom.xml Mon Feb  1 10:43:00 2010
>>  @@ -405,9 +405,10 @@
>>          <artifactId>maven-surefire-plugin</artifactId>
>>          <configuration>
>>            <includes>
>>  -            <include>**/*TestSuite.java</include>
>>  +            <include>**/*Test.java</include>
>>            </includes>
>>            <excludes>
>>  +            <exclude>**/*TestSuite.java</exclude>
>>              <exclude>**/AllLangTestSuite.java</exclude>
>
> Why not just delete all the TestSuite files? Are they still needed? [Why?]

Hi Sebb,

Its on my TODO list to port those changes back from trunk, but I was
going to do it in conjunction with removing the suite() methods and
changing the ant build

Niall

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to