On 8/15/06, Paulex Yang wrote:
I agree it's not right time to re-layout test case right now, but actually LUNI is a different case, whose test layout is a little....too confusing...
I agree with you and Nathan that the layout is confusing. But how it should be changed? Currently we have only one complete proposal [1]. But the change doesn't follow it. Is this a new apporach for luni tests? Thanks, Stepan. [1] * http://incubator.apache.org/harmony/subcomponents/classlibrary/testing.html*<http://incubator.apache.org/harmony/subcomponents/classlibrary/testing.html> For example, there is package name o.a.h.luni.tests.java.lang as well as
o.a.h.tests.java.lang, IMHO, what ever methods we take at last, it's hard to understand what difference between them...further, there are another several packages named as tests.api.java.lang(it's worse before Nathan removed another two source folders), I myself always failed to find any test class at first try even in Eclipse(and it's nightmare if in explorer). So I believe some merge work is necessary, after all we must understand what's going on in that LUNI labyrinth before we apply any layout/metadata approach to it. Nathan Beyer wrote: > >> -----Original Message----- >> From: Stepan Mishura [mailto:[EMAIL PROTECTED] >> Sent: Monday, August 14, 2006 6:36 AM >> To: harmony-dev >> Subject: Re: svn commit: r431219 - in >> /incubator/harmony/enhanced/classlib/trunk/modules/luni/src/test: api/ >> impl/ java/org/apache/harmony/luni/tests/java/lang/ >> >> Hi Nathan, >> >> I saw that you've changed tests layout (consolidated) tests in luni >> module. >> But I don't understand what the purpose for the change. Is this a first >> step >> for migrating to TestNG? (May be I missed something important - sorry for >> stupid question then.) >> > > No, I'm just trying to put all of the tests into what I would consider is a > defacto standard "src/test/java" and "src/test/resources". The actual goal > of this consolidation is to pull together all of the duplicated tests and > similar tests. This is reducing the test run time for luni and making patch > creation/application much easier. At the same time, I've been cleaning up > the tests so that those that don't run can be run and anything that doesn't > run on a RI is fixed, so that it does. > > >> BTW did we agree on that we won't follow directory-based tests layout [1]? >> IMHO, not yet. IIRC we agreed that directory-based tests layout is not >> quite >> convenient for our needs and we'll try TestNG. But for me it doesn't mean >> that we'll migrate to TestNG for sure. Also I assume that mixed approach >> (directory-based layout + annotations) is possible. So it is not time for >> doing massive tests moves and merges. >> > > I've been watching the test layout/approach threads, but I haven't seen > anything substantial fall out. I'm still waiting to see a simple and clear > suggestion. In any case, the current state is, frankly, a bit of a mess and > any cleanup can't hurt. > > Additionally, I was trying the separated directory layout to see what it's > like and I can tell you first hand it's just not that great. It just adds > complexity in the repository and the build script. This may be needed for > some modules, but I don't see it as the norm. Also, I've been experimenting > with just using plain-old JUnit TestSuites to separate pure-API tests from > implementation tests and it works very well. It works equally well for > creating a failing-tests test suite. It also seems to make the tests run > much faster, but I'm not sure why; maybe it's because Ant doesn't search for > all tests, it just runs the single test that's an aggregate. > > >> And the minor comment to the commit log. It says: "remove unused folders". >> But there are a lot of other unrelated changes. Was it possible to make >> separate commits? >> > > This was a mistake. I accidentally committed extra files with that revision. > > >> Thanks, >> Stepan. >> >> [1] >> http://incubator.apache.org/harmony/subcomponents/classlibrary/testing.htm >> l >> >> -----Original Message----- >> From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] >> Sent: Monday, August 14, 2006 2:56 AM >> To: [EMAIL PROTECTED] >> Subject: svn commit: r431219 - in >> /incubator/harmony/enhanced/classlib/trunk/modules/luni/src/test: api/ >> impl/ >> java/org/apache/harmony/luni/tests/java/lang/ >> >> Author: ndbeyer >> Date: Sun Aug 13 12:55:54 2006 >> New Revision: 431219 >> >> URL: http://svn.apache.org/viewvc?rev=431219&view=rev >> Log: >> remove unused folders >> >> Removed: >> incubator/harmony/enhanced/classlib/trunk/modules/luni/src/test/api/ >> incubator/harmony/enhanced/classlib/trunk/modules/luni/src/test/impl/ >> Modified: >> >> incubator/harmony/enhanced/classlib/trunk/modules/luni/src/test/java/org/a >> pache/harmony/luni/tests/java/lang/DoubleTest.java >> >> incubator/harmony/enhanced/classlib/trunk/modules/luni/src/test/java/org/a >> pache/harmony/luni/tests/java/lang/SecurityManagerTest.java >> >> incubator/harmony/enhanced/classlib/trunk/modules/luni/src/test/java/org/a >> pache/harmony/luni/tests/java/lang/ThreadGroupTest.java >> >> Modified: >> incubator/harmony/enhanced/classlib/trunk/modules/luni/src/test/java/org/a >> pache/harmony/luni/tests/java/lang/DoubleTest.java >> URL: >> http://svn.apache.org/viewvc/incubator/harmony/enhanced/classlib/trunk/mod >> ules/luni/src/test/java/org/apache/harmony/luni/tests/java/lang/DoubleTest >> .java?rev=431219&r1=431218&r2=431219&view=diff >> ========================================================================== >> ==== >> >> >> --- >> incubator/harmony/enhanced/classlib/trunk/modules/luni/src/test/java/org/a >> pache/harmony/luni/tests/java/lang/DoubleTest.java >> (original) >> +++ >> incubator/harmony/enhanced/classlib/trunk/modules/luni/src/test/java/org/a >> pache/harmony/luni/tests/java/lang/DoubleTest.java >> Sun Aug 13 12:55:54 2006 >> @@ -441,7 +441,7 @@ >> */ >> public void test_parseDoubleLjava_lang_String() { >> assertEquals("Incorrect double returned, expected zero.", 0.0, >> Double >> - .parseDouble("2.4703282292062327208828439643411e-324 ")); >> + .parseDouble("2.4703282292062327208828439643411e-324 "), >> 0.0 >> ); >> assertTrue("Incorrect double returned, expected minimum double.", >> Double >> .parseDouble("2.4703282292062327208828439643412e-324") == >> Double.MIN_VALUE); >> >> >> Modified: >> incubator/harmony/enhanced/classlib/trunk/modules/luni/src/test/java/org/a >> pache/harmony/luni/tests/java/lang/SecurityManagerTest.java >> <SNIP> >> >> >> ------------------------------------------------------ >> Terms of use : http://incubator.apache.org/harmony/mailing.html >> To unsubscribe, e-mail: [EMAIL PROTECTED] >> For additional commands, e-mail: [EMAIL PROTECTED] >> > > > --------------------------------------------------------------------- > Terms of use : http://incubator.apache.org/harmony/mailing.html > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > > -- Paulex Yang China Software Development Lab IBM --------------------------------------------------------------------- Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
-- Thanks, Stepan Mishura Intel Middleware Products Division ------------------------------------------------------ Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
