Please let me know if I can help. I want to get more tests in. I started
looking at the build.xml but I would prefer to remain away if you already
looking at it Danny.
Started Javadoc on all test methods. I don't think javadocs can be a a
justification to remove files, esp. not files for testing, but I am willing
to go the extra mile.
Javadocs are for end users and tests are not.
So I am doing things like this:
+ /** login to pop3 server */
public void login() throws Exception {
client.login(username, password);
}
Hopefully it will make things easier to accept.
Harmeet
----- Original Message -----
From: "Noel J. Bergman" <[EMAIL PROTECTED]>
To: "James Developers List" <[EMAIL PROTECTED]>
Sent: Sunday, October 13, 2002 10:53 AM
Subject: RE: cvs commit: jakarta-james/src/java/org/apache/james/testing
EndToEnd.java SMTPDeliveryWorker.java
> Danny,
>
> And I thank you for doing so! I'm prepared, personally, to go find the
> missing packages so that I can use the code. :-) If you're willing to
move
> them, do the build.xml changes, and put the necessary libraries in ...
> where? tools/? ... that's great! :-)
>
> I think that the examples.IOUtils import should be removed, yes?
>
> --- Noel
>
> -----Original Message-----
> From: Danny Angus [mailto:[EMAIL PROTECTED]]
> Sent: Sunday, October 13, 2002 13:25
> To: James Developers List
> Subject: RE: cvs commit: jakarta-james/src/java/org/apache/james/testing
> EndToEnd.java SMTPDeliveryWorker.java
>
>
> Actually I commited this test *only* because
> a) Noel expressed an interest
> and
> b) I'm prepared to move the tests and alter build.xml to build them, run
> them, and leave them out of the distribution
>
> d.
>
> > -----Original Message-----
> > From: Noel J. Bergman [mailto:[EMAIL PROTECTED]]
> > Sent: 13 October 2002 18:13
> > To: James Developers List
> > Subject: RE: cvs commit: jakarta-james/src/java/org/apache/james/testing
> > EndToEnd.java SMTPDeliveryWorker.java
> >
> >
> > > Hey Danny, you haven't got javadocs on your methods.
> > > Peter are you going to remove this class too ?
> >
> > Now, now ... let's not get personal.
> >
> > More of an issue is that it references things we don't have, but
> > that can be cleaned up.
> >
> > --- Noel
> >
> >
> > --
> > To unsubscribe, e-mail:
> > <mailto:[EMAIL PROTECTED]>
> > For additional commands, e-mail:
> > <mailto:[EMAIL PROTECTED]>
> >
>
>
> --
> To unsubscribe, e-mail:
<mailto:[EMAIL PROTECTED]>
> For additional commands, e-mail:
<mailto:[EMAIL PROTECTED]>
>
>
>
> --
> To unsubscribe, e-mail:
<mailto:[EMAIL PROTECTED]>
> For additional commands, e-mail:
<mailto:[EMAIL PROTECTED]>
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
