[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
Re: [stp-dev] Commit: build/ - Code commit guidelines
|
I agree and would add that you sometime have to write your tests in the
same java package in order to test package-protected members, methods
and classes.
alex
Beaurpere, David wrote:
>I just have one comment regarding the test packaging: I wouldn't add the
>extra "tests" package layer.
>
>Frankly, I simply find it more practical to keep the tests under the
>same packaging structure as the code they are testing. Since the tests
>resources will be hosted in a distinct CVS location anyway, I do not
>really see an actual benefit for the extra package.
>
>I know this suggestion doesn't match the eclipse convention.
>
>Rgds,
>D.
>
>
>
>
>
>
>
>>-----Original Message-----
>>From: stp-dev-bounces@xxxxxxxxxxx
>>[mailto:stp-dev-bounces@xxxxxxxxxxx] On Behalf Of Hurley, Oisin
>>Sent: 15 February 2006 17:48
>>To: STP Dev list
>>Subject: Re: [stp-dev] Commit: build/ - Code commit guidelines
>>
>>Ok, I'm making a collection of proposals for naming conventions:
>>
>>1. Differentiate between API and non-API classes by using a
>>package name element of 'internal'
>>
>>2. Differentiate between UI and non-UI classes by using a
>>package name element of 'ui'
>>
>>3. Differentiate between test and implementation classes by
>>using a package name element of 'test'
>>
>>4. Use org.eclipse.stp as the project base plugin name
>>
>>5. Use the subproject abbrevs in the plugin names e.g.
>> org.eclipse.stp.cf.module
>>
>>So, the CVS setup would look like (as per Naci's mail):
>>
>>/releng.stpbuilder
>> **
>>/releng
>> /maps
>> *.map
>>/components
>> /soas
>> /features
>> /org.eclipse.stp.soas ....
>> /sc
>> /features
>> /org.eclipse.stp.sc
>> /plugins
>> /org.eclipse.stp.sc.ui
>> /org.eclipse.stp.sc.core
>> ..
>> /tests
>> /org.eclipse.stp.sc.tests.ui
>> /org.eclipse.stp.sc.tests.core
>> /cf
>> ..
>> /b2j
>> ..
>> /bpmn
>> ..
>>
>> --oh
>>_______________________________________________
>>stp-dev mailing list
>>stp-dev@xxxxxxxxxxx
>>https://dev.eclipse.org/mailman/listinfo/stp-dev
>>
>>
>>
>_______________________________________________
>stp-dev mailing list
>stp-dev@xxxxxxxxxxx
>https://dev.eclipse.org/mailman/listinfo/stp-dev
>
>