[jboss-as7-dev] Planed testsuite restructure
Thomas Diesler
thomas.diesler at jboss.com
Wed Nov 23 03:01:59 EST 2011
Doing things 'pragmatically' would mean you put the sources in the
modules where they are used. Otherwise this just creates entropy and
confusion.
-thomas
On 11/22/2011 07:23 PM, Ondřej Žižka wrote:
> Hi,
>
> If the IDE was correctly programmed, it would honour
> <testSourceDirectory>.
> Since maven convention is not described anywhere, it's subjective, so
> we can say it only breaks Eclipse's integration - In NetBeans it works
> pretty fine :)
>
> Also, I'm quite sure we will never ship testsuite source artifacts -
> there's no sense in that.
>
> But anyway, taken pragmatically - if it causes real troubles, I'll
> move the sources.
> But first we need to merge PR 697, I will base all further changes on it.
>
>
> Ondra
>
>
>
>
>
> Thomas Diesler píše v Pá 18. 11. 2011 v 17:04 +0100:
>> Done. However I don't think we can use your testsuite structure as it
>> is now. You keep the test sources in testsuite/integration/src but
>> introduce a number of new maven modules that refer to those sources.
>> This breaks maven convention and the IDE gets confused. The maven
>> source artefacts would also not contain the expected sources. If
>> possible please move the test sources to their respective modules.
>>
>> If test sources really need to get shared between maven modules, you
>> could have a look at the demos. I believe those sources are reused by
>> the testsuite (at least in the past). Hence you could have a module
>> 'testsuite/shared' or something similar.
>>
>> cheers
>> -thomas
>>
>> On 11/18/2011 09:46 AM, Thomas Diesler wrote:
>>> yes, I can do this today.
>>>
>>> On 11/18/2011 09:47 AM, Ondřej Žižka wrote:
>>>> Hi Thomas,
>>>>
>>>> could you please update the OSGi tests to use the
>>>> jboss.ts.integ.dir property instead of assuming it runs in
>>>> testsuite/integration?
>>>> The working dir will change.
>>>> I'd change it myself but it's outside of AS project.
>>>>
>>>> Also, I need it get fixed to get the updated testsuite merged ASAP,
>>>> so I'd be very happy if you had a minute for this before monday.
>>>>
>>>> Thanks,
>>>> Ondra
>>>
>>> --
>>> xxxxxxxxxxxxxxxxxxxxxxxxxxxx
>>> Thomas Diesler
>>> JBoss OSGi Lead
>>> JBoss, a division of Red Hat
>>> xxxxxxxxxxxxxxxxxxxxxxxxxxxx
>>
>> --
>> xxxxxxxxxxxxxxxxxxxxxxxxxxxx
>> Thomas Diesler
>> JBoss OSGi Lead
>> JBoss, a division of Red Hat
>> xxxxxxxxxxxxxxxxxxxxxxxxxxxx
--
xxxxxxxxxxxxxxxxxxxxxxxxxxxx
Thomas Diesler
JBoss OSGi Lead
JBoss, a division of Red Hat
xxxxxxxxxxxxxxxxxxxxxxxxxxxx
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/jboss-as7-dev/attachments/20111123/abd780df/attachment.html
More information about the jboss-as7-dev
mailing list