creadur-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "P. Ottlinger" <pottlin...@aiki-it.de>
Subject Re: svn commit: r1617205 - /creadur/rat/trunk/apache-rat-plugin/src/test/java/org/apache/rat/mp/RatCheckMojoTest.java
Date Mon, 11 Aug 2014 21:42:35 GMT
Hi Sebb, hi Dennis,

thanks for your help in identifying that RAT's testing can be improved.

Am 11.08.2014 um 14:15 schrieb sebb:
>> Let me have a look at the ITs in general, and adding one that catches
>> > whatever is causing problems. I've created a decent amount of Maven
>> > Plugin ITs in my day.
> Great - maybe we can a README to the IT tree to explain how to add new tests?
> Also point to the Maven docs that describe the layout and processing.
> 
> For example, I can see that there are test/ and test/invoker
> sub-folders that relate to each other.
> However, it is not clear how the shared files under test/ and
> test/invoker should be used.
> In particular, the files under java - are these used for all it tests?
> If so. how does one allow for different test conditions?
> 
>>> >> I may give it a try, but no promises!
> I added it4_RAT-168
> 
> But note that it2 and it3 don't seem to be run currently - RAT-169
> It would be good to fix that.

Following your discussion I've extracted some static testHelper methods
in order to be able to see tests more clearly.

Hope this is okay and we can continue to create a new 0.11 RC.

What do you think?

Phil


Mime
View raw message