I think the AllTest makes sense only when some one want to run test
suites partly. I had never used it but I'm not sure if there is
somebody who perfer that way.

Well, seems AllTest is a history now. It doesn't make use of exclude
lists and etc. Moreover, some modules does not support it - i.e. does
not have AllTest.. IIRC we have agreed that storing grouping
information in java source is (generally) not a very good idea.

Thanks,

2007/2/28, Tony Wu <[EMAIL PROTECTED]>:
On 2/28/07, Mikhail Loenko <[EMAIL PROTECTED]> wrote:
> Thanks, Stepan
>
> Seems like the key is "cleaned my working copy" here. Because I saw no
> problems when tested my commit.
>
yes, there might be a TestBasicControl.class in classpath if did not clean.

> BTW, do these "AllTest" make sense? Does anybody use them?
> We do'nt have all the tests included into "AllTest", so it might make
> sense to get rid of them
>

I think the AllTest makes sense only when some one want to run test
suites partly. I had never used it but I'm not sure if there is
somebody who perfer that way.

> Thanks,
> Mikhail
>
> 2007/2/28, Stepan Mishura <[EMAIL PROTECTED]>:
> > On 2/28/07, Tony Wu <[EMAIL PROTECTED]> wrote:
> > >
> > > I cleaned my working copy and  ran "ant -Dbuild.module=jndi test",
> > > then I got following compile error.
> >
> >
> > Fixed at r512588.
> >
> > -Stepan.
> >
> > [javac] 1. ERROR in
> > >
> > > 
/home/clear/trunk/modules/jndi/src/test/java/org/apache/harmony/jndi/tests/javax/naming/ldap/AllTestLdap.java
> > >    [javac]  (at line 49)
> > >    [javac]     suite.addTest(new TestSuite(TestBasicControl.class));
> > >    [javac]                                 ^^^^^^^^^^^^^^^^
> > >    [javac] TestBasicControl cannot be resolved to a type
> > >    [javac] ----------
> > >    [javac] 1 problem (1 error)
> > >
> > > I did not find the TestBasicControl.java in jndi src.
> > >

--
Alexei Zakharov,
Intel ESSD

Reply via email to