commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Niall Pemberton <niall.pember...@gmail.com>
Subject Re: svn commit: r905240 - /commons/proper/lang/branches/LANG_2_X/pom.xml
Date Mon, 01 Feb 2010 11:26:05 GMT
On Mon, Feb 1, 2010 at 11:17 AM, sebb <sebbaz@gmail.com> wrote:
> On 01/02/2010, niallp@apache.org <niallp@apache.org> 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: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Mime
View raw message