Mailing-List: contact harmony-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: harmony-dev@incubator.apache.org
Received-SPF: pass (asf.osuosl.org: domain of george.c.harley@googlemail.com
 designates 64.233.162.196 as permitted sender)
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
        s=beta; d=googlemail.com;
        h=received:message-id:date:from:reply-to:user-agent:mime-version:to:subject:references:in-reply-to:content-type:content-transfer-encoding;
        b=P365Zz4Gq9Bx2WjhJ6EqXXASob5NVNwDK2WRUVOSK+xzB6T/FtQqBHacWUdMy9Bp1iLxNfLUMERG3EB6+9nUh3NryykmtfCeBQM3YFEg8VuGbP9FEUfsWGVWtrLt4BTW81dh96HdzESvC+97F+6ay9tTm9+G+Sm4A+uCiLdyUbQ=
Message-ID: <44644BBD.2060005@googlemail.com>
Date: Fri, 12 May 2006 09:47:57 +0100
From: George Harley <george.c.harley@googlemail.com>
Reply-To: harmony-dev@incubator.apache.org
User-Agent: Thunderbird 1.5.0.2 (Windows/20060308)
MIME-Version: 1.0
To: harmony-dev@incubator.apache.org
Subject: Re: [classlib] Layout of tests in beans module
References: <44622161.6040208@googlemail.com>
	 <906dd82e0605102103k4562ab7ya15d3ef87df5a2dd@mail.gmail.com>
	 <4462FDDF.9000801@googlemail.com>
	 <906dd82e0605110351o525e1094qda466ca96ab62a3d@mail.gmail.com>
	 <44631D1C.4000100@googlemail.com>
	 <906dd82e0605110515k45a179dfwd1a68f5e51a28e3c@mail.gmail.com>
 <6e47b64f0605112032j7bb90e93t45838536b8fc2a15@mail.gmail.com>
In-Reply-To: <6e47b64f0605112032j7bb90e93t45838536b8fc2a15@mail.gmail.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

Mikhail / Stepan,

Thanks for your comments. I will proceed with this when SVN is available 
again for us all.

Best regards,
George


Stepan Mishura wrote:
> On 5/11/06, Mikhail Loenko wrote:
>>
>> 2006/5/11, George Harley :
>> > Mikhail Loenko wrote:
>> > > OK I see.
>> > >
>> > > I had some thoughts on merging the tests. Are we going to merge
>> > > several tests
>> > > into a single file (thus losing their histories) or we will add a
>> > > digit to the names and
>> > > put into the same folder?
>> > >
>> > > For example we have math tests from at least 3 sources and currently
>> > > some of the tests are of several megabytes long. Are we going to mix
>> > > all them
>> > > into a single file?
>> > >
>> > > Thanks,
>> > > Mikhail
>> >
>> > Hi Mikhail,
>> >
>> > The approach I have taken in other "tidied" modules has been to merge
>> > all test methods for a given type into the one file containing a JUnit
>> > testcase dedicated to that type. It sounds like it will be more
>> > challenging to do this in maths for the reasons you mention.
>> >
>> > Beyond getting the related tests into a single package structure I am
>> > pretty neutral really on how we manage the contents of individual 
>> files.
>> > If in some cases a merge looks set to create a single gigantic file
>> > containing a very large number of test methods for a single type then
>> > splitting the test case class into FooTest.java, FooTest2.java etc
>> > sounds fine. Is that what you were suggesting ?
>>
>> That is what I was talking about. At this point I can't say which way
>> looks better, I just have a concern about uniting some types of
>> files, like man-made and generated tests.
>
>
> I'd say that it is OK as short-term solution to adjust tests layout. And
> further tests for one class should be merged into one file (even big 
> one) -
> to avoid tests duplication and resolve ambiguities like "which unit test
> class should contain the changes".
>
> Thanks,
> Stepan.
>
> Thanks,
>> Mikhail
>>
>> >
>> > Best regards,
>> > George
>> >
>> > >
>> > > 2006/5/11, George Harley <george.c.harley@googlemail.com>:
>> > >> Mikhail Loenko wrote:
>> > >> > George
>> > >> >
>> > >> > Is there any reason to make this changes now rather then complete
>> > >> > discussion
>> > >> > around the tests layout?
>> > >> >
>> > >> > Thanks,
>> > >> > Mikhail
>> > >>
>> > >> Hi Mikhail,
>> > >>
>> > >> Good point, I should have referenced that discussion. My proposed
>> > >> changes are not intended to be the last word on the tests layout 
>> but
>> > >> more an attempt to tidy up the tests into a smaller set of packages
>> so
>> > >> that it is hopefully easier to make the agreed changes in the 
>> future.
>> > >> For instance, we presently have a number of test classes in 
>> different
>> > >> packages (but with the same base name) that are testing the same 
>> API
>> > >> types - I think there is value in merging these scattered tests
>> together
>> > >> so that we can all see precisely where the tests for a given public
>> API
>> > >> type should go. This will hopefully make it easier for folks to
>> supply
>> > >> patches that include unit test code as it removes any possible
>> > >> ambiguities about which unit test class should contain the changes.
>> It
>> > >> would remove something I have seen a few times where separate JIRA
>> > >> reports fixing bugs in a particular type supply unit test 
>> changes to
>> > >> different test classes.
>> > >>
>> > >> Best regards,
>> > >> George
>> > >>
>> > >> >
>> > >> > 2006/5/11, George Harley <george.c.harley@googlemail.com>:
>> > >> >> Hi,
>> > >> >>
>> > >> >> Just a heads up to you all (and especially anyone working in 
>> beans
>> > >> just
>> > >> >> now) that I would like to make some changes to src/tests in the
>> beans
>> > >> >> module in the next day or so. Proposed changes are similar to 
>> the
>> > >> tidy
>> > >> >> ups that have taken place in archive, jndi, rmi, logging, prefs
>> > >> etc over
>> > >> >> the last few weeks and consist of ...
>> > >> >>
>> > >> >> * under the src/test/java source folder, merge all of the API
>> > >> tests into
>> > >> >> one package structure under org.apache.harmony.beans.tests. 
>> So we
>> get
>> > >> >> o.a.h.b.t.java.beans and o.a.h.b.t.java.beans.beancontext etc.
>> > >> >>
>> > >> >> * move the relatively small number of tests that rely on running
>> > >> on the
>> > >> >> bootclasspath to under src/test/java-internal which get compiled
>> to
>> > >> >> bin/test-internal.
>> > >> >>
>> > >> >> * in the run.tests target of make/common/build.xml have a junit
>> task
>> > >> >> that runs the internal tests on the bootclasspath and another
>> > >> junit test
>> > >> >> that runs the API/non-internal tests on the classpath.
>> > >> >>
>> > >> >>
>> > >> >> Please holler if you are about to make any changes in this area.
>> > >> >> Otherwise I will proceed when/if SVN becomes available for me
>> again.
>> > >> >>
>> > >> >>
>> > >> >> Best regards,
>> > >> >> George
>> > >> >>
>> > >> >>
>> > >> >>
>> > >> >>
>> > >> >>
>> > >> >>
>> > >> >>
>> > >> >>
>> > >> >>
>> ---------------------------------------------------------------------
>> > >> >> Terms of use : http://incubator.apache.org/harmony/mailing.html
>> > >> >> To unsubscribe, e-mail:
>> harmony-dev-unsubscribe@incubator.apache.org
>> > >> >> For additional commands, e-mail:
>> > >> harmony-dev-help@incubator.apache.org
>> > >> >>
>> > >> >>
>> > >> >
>> > >> >
>> ---------------------------------------------------------------------
>> > >> > Terms of use : http://incubator.apache.org/harmony/mailing.html
>> > >> > To unsubscribe, e-mail:
>> harmony-dev-unsubscribe@incubator.apache.org
>> > >> > For additional commands, e-mail:
>> harmony-dev-help@incubator.apache.org
>> > >> >
>> > >> >
>> > >>
>> > >>
>> > >> 
>> ---------------------------------------------------------------------
>> > >> Terms of use : http://incubator.apache.org/harmony/mailing.html
>> > >> To unsubscribe, e-mail: 
>> harmony-dev-unsubscribe@incubator.apache.org
>> > >> For additional commands, e-mail:
>> harmony-dev-help@incubator.apache.org
>> > >>
>> > >>
>> > >
>> > > 
>> ---------------------------------------------------------------------
>> > > Terms of use : http://incubator.apache.org/harmony/mailing.html
>> > > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
>> > > For additional commands, e-mail: 
>> harmony-dev-help@incubator.apache.org
>> > >
>> > >
>> >
>> >
>> > ---------------------------------------------------------------------
>> > Terms of use : http://incubator.apache.org/harmony/mailing.html
>> > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
>> > For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>> >
>> >
>>
>> ---------------------------------------------------------------------
>> Terms of use : http://incubator.apache.org/harmony/mailing.html
>> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
>> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>>
>>
>
>


---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org