And how would you split 1,2,4,5 between these two folders? Thanks, Mikhail
2006/4/27, Paulex Yang <[EMAIL PROTECTED]>: > Mikhail Loenko wrote: > > If it's separated, I'm OK with it > > > > Can you suggest dir names? > > > Great! > > Either boot or injected(as Mark said) sounds good keyword to me. so it > can be one of below: > 1. src/test.injected/ > 2. src/test/injected > 3. src/test.boot/ > 4. src/test/boot > > Mmm...seems injected is more self-descriptive, I prefer option 1. So > the layout would be: > src/main > src/test > src/test.injected > > ideas? > > Thanks, > > Mikhail > > > > 2006/4/27, Paulex Yang <[EMAIL PROTECTED]>: > > > >> Mikhail Loenko wrote: > >> > >>> The problem is the tests will be mixed like that: > >>> > >>> org.apache.harmony.security.asn1 -- tests in bootclasspath > >>> org.apache.harmony.security.cert -- tests in bootclasspath > >>> org.apache.harmony.security.tests -- tests in classpath > >>> org.apache.harmony.security.x509 -- tests in bootclasspath > >>> > >>> > >> Good catch! :) > >> > >> It's not so hard for script to separate, if the package name doesn't > >> contains "test", it belongs to bootclasspath > >> > >> And if you concern about human reading, we can separate src directory > >> for them, i.e. > >> src/main/java > >> src/test/java > >> src/boottest/java or src/test/boot/java > >> > >> comments? > >> > >>> Thanks, > >>> Mikhail > >>> > >>> 2006/4/27, Paulex Yang <[EMAIL PROTECTED]>: > >>> > >>> > >>>> Mikhail Loenko wrote: > >>>> > >>>> > >>>>> Paulex, > >>>>> > >>>>> we have at least 8 categories of tests: > >>>>> running from classpath or bootclasspath > >>>>> implementation specific or independent > >>>>> testing org.apache.harmony.* or java.* > >>>>> Could you please list how all the tests will be named > >>>>> > >>>>> > >>>>> > >>>> Mikhail, > >>>> > >>>> OK, let me try. I'm not sure if all 8 categories are necessary, > >>>> > >>>> 1. [classpath], [independent], [java.*] > >>>> o.a.h.<module name>.tests.api.<package name>, e.g., > >>>> o.a.h.luni.tests.api.java.util is test package for java.util > >>>> I consider the module name is optional and can be removed, so it can > >>>> be > >>>> o.a.h.tests.api.java.util for java.util > >>>> > >>>> 2. [classpath], [dependent], [o.a.h.*] > >>>> o.a.h.<module name>.tests.<package name without o.a.h...prefix>, > >>>> e.g., o.a.h.luni.tests.util is test package for o.a.h.luni.util > >>>> Again, the module name is optional, but if module name is not > >>>> prefix, it should be part of package name, so it is: > >>>> o.a.h.tests.luni.util for o.a.h.luni.util > >>>> > >>>> 3. [classpath], [independent], [o.a.h.*] > >>>> I don't expect these classes/tests exist. > >>>> > >>>> 4. [classpath], [dependent], [java.*] > >>>> if my understanding is correct, this category means the tests for > >>>> intended different behavior between Harmony and RI, I consider these > >>>> tests should be put in same package with API tests, and separate them by > >>>> running script instead of different package, because RI may fix bug in > >>>> the revision later(e.g, Sun JDK 1.5.1 maybe?), and IIRC in another > >>>> thread we have agreed that we always follow latest version of RI, so it > >>>> will be much easier to change the different behavior tests to API tests > >>>> if we use script to separate them, and vice versa. > >>>> > >>>> 5. [bootclasspath], > >>>> which covers 4 categories you mentioned, the test cases are in same > >>>> package with implementions > >>>> e.g. > >>>> java.util is test package for java.util > >>>> and > >>>> o.a.h.luni.util is test packag for o.a.h.luni.util > >>>> > >>>> As a summary, we'll have four kind of test packages: > >>>> > >>>> a. o.a.h.<module>.tests.api.java.*** for case 1,4 above > >>>> b. o.a.h.<module>.tests.*** for case 2 above > >>>> c. java.***, for case 5 above > >>>> d. o.a.h.***, for case 5 above > >>>> > >>>> comments? > >>>> > >>>> > >>>> > >>>> > >>>>> Thanks, > >>>>> Mikhail > >>>>> > >>>>> > >>>>> 2006/4/27, Paulex Yang <[EMAIL PROTECTED]>: > >>>>> > >>>>> > >>>>> > >>>>>> Mikhail Loenko wrote: > >>>>>> > >>>>>> > >>>>>> > >>>>>>> Geir, > >>>>>>> > >>>>>>> The problem is that no one yet suggested a consistent solution > >>>>>>> that would fit for all the tests and would not get into packages like > >>>>>>> > >>>>>>> org.apache.harmony.security.tests.org.apache.harmony.security.util > >>>>>>> > >>>>>>> > >>>>>>> Everybody seems to agree that SOME implementation specific tests are > >>>>>>> in the same package as implementation. > >>>>>>> > >>>>>>> But if we have ALL the tests in the same package as implementation, > >>>>>>> then we can't run e.g. tests against java.beans.Beans from classpath, > >>>>>>> so > >>>>>>> we can't run regression test for HARMONY-358 > >>>>>>> > >>>>>>> The question is where we put 'classpath' test. If we put them into > >>>>>>> o.a.h.module.tests.[package under test] we result in that ugly package > >>>>>>> names like > >>>>>>> o.a.h.security.tests.o.a.h.security.util > >>>>>>> > >>>>>>> > >>>>>>> > >>>>>>> > >>>>>> Mikhail, > >>>>>> > >>>>>> Why my prior proposal that o.a.h.security.tests.impl.util is > >>>>>> unacceptable? > >>>>>> > >>>>>> And I have no objection if "impl" considered redundant, too. > >>>>>> > >>>>>> > >>>>>> > >>>>>> > >>>>>>> Thanks, > >>>>>>> Mikhail > >>>>>>> > >>>>>>> > >>>>>>> 2006/4/27, Geir Magnusson Jr <[EMAIL PROTECTED]>: > >>>>>>> > >>>>>>> > >>>>>>> > >>>>>>> > >>>>>>>> Geir Magnusson Jr wrote: > >>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>>>> George Harley wrote: > >>>>>>>>> > >>>>>>>>> > >>>>>>>>> > >>>>>>>>> > >>>>>>>> >> > >>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>>>>> Of course, the text module has only "implementation-independent > >>>>>>>>>> tests > >>>>>>>>>> that designed to be run from classpath". For modules that have got > >>>>>>>>>> implementation-specific tests then I suppose we could use something > >>>>>>>>>> like "org.apache.harmony.[module].tests.impl.[package under test]" > >>>>>>>>>> or > >>>>>>>>>> "org.apache.harmony.[module].tests.internal.[package under test]" > >>>>>>>>>> etc. > >>>>>>>>>> I've got no preference. > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>> -1 > >>>>>>>>> > >>>>>>>>> Put them in the same package as the class being tested, please, if > >>>>>>>>> there > >>>>>>>>> is no problem being run that way. > >>>>>>>>> > >>>>>>>>> > >>>>>>>>> > >>>>>>>>> > >>>>>>>> Sorry to be so snippy. Been a long day. > >>>>>>>> > >>>>>>>> Why would we want this convoluted package name? Why wouldn't we > >>>>>>>> want to > >>>>>>>> let them be in the same package as the class being tested since it's > >>>>>>>> implementation testing? > >>>>>>>> > >>>>>>>> geir > >>>>>>>> > >>>>>>>> --------------------------------------------------------------------- > >>>>>>>> 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] > >>>>>> > >>>>>> > >>>>>> > >>>>>> > >>>>>> > >>>>> --------------------------------------------------------------------- > >>>>> 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] > >>>> > >>>> > >>>> > >>>> > >>> --------------------------------------------------------------------- > >>> 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] > >> > >> > >> > > > > --------------------------------------------------------------------- > > 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] > > --------------------------------------------------------------------- Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]