harmony-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Richard Liang" <richard.lian...@gmail.com>
Subject Re: [test] Jetty integration progress ? (was Re: [classlib] jetty based tests)
Date Wed, 20 Sep 2006 09:35:32 GMT
On 9/19/06, Stepan Mishura <stepan.mishura@gmail.com> wrote:
> On 9/19/06, Richard Liang wrote:
> >
> > On 9/18/06, Geir Magnusson Jr. wrote:
> > >
> > >
> > > Andrew Zhang wrote:
> > > > On 9/18/06, Richard Liang <richard.liangyx@gmail.com> wrote:
> > > >>
> > > >> On 9/18/06, Andrew Zhang <zhanghuangzhu@gmail.com> wrote:
> > > >> > Hi,
> > > >> >
> > > >> > It's me again. Seems no big progress on jetty. I'd like to take
the
> > job
> > > >> if
> > > >> > no one objects. Here are my suggestions:
> > > >>
> > > >> Great :-)
> > > >>
> > > >> >
> > > >> > 1. jetty version:  I suggest that Harmony adopt jetty 6. Because
> > many
> > > >> > 5.xAPIs are deprecated in jetty 6, we'd better follow latest
jetty
> > > >> > version.
> > > >>
> > > >> +1
> > > >>
> > > >> >
> > > >> > 2. location to put jetty jars: support module.
> > > >> >
> > > >>
> > > >> Do you mean we will check the jetty jars into Harmony svn?
> > > >
> > > >
> > > > Yes. Is it OK? Or put the jar in depends folder?
> > >
> > > Just make it a depends.  We should avoid checking in jars.
> > >
> >
> > Yes. It's good make jetty as a depends, and we could add jetty.jar
> > into their build scripts if some modules (e.g., luni) require jetty.
> > Just thinking about another question, how shall we handle the
> > .classpath of luni in Eclipse? Use external jar"?
>
>
>
> I'd use it as 'support.jar' - so it is downloaded to 'depends', copied by
> the build to 'deploy/build/test' and added as external jar in Eclipse.

Yes. It's reasonable. Let's have a try ;-) Andrew?

>
> Thanks,
> Stepan.
>
> > >
> > > >> 3. how to write jetty test? I suggest that we could start jetty in
> > any
> > > >> test
> > > >> > if necessary. If we found there are heavy code duplicates, we
could
> > > >> extract
> > > >> > them as utility methods in support module. So far, I'd like to
> > write
> > > >> jetty
> > > >> > test directly in each module, because the code is rather simple,
> > only a
> > > >> few
> > > >> > lines.[1] It's also easy to write user-customized handler for
> > > >> > negative tests. Let's make it work, and then make it better.
:-)
> > > >> >
> > > >> > Any suggestions/comments/objections?  I volunteer to upload patches
> > > >> when
> > > >> we
> > > >> > reach an agreement.
> > > >> >
> > > >> > Best regards,
> > > >> > Andrew
> > > >> >
> > > >> > [1]
> > > >> > jetty-based test example:
> > > >> > setUp code:
> > > >> >             port = Support_PortManager.getNextPort();
> > > >> >             Server server = new Server(port);
> > > >> >             ResourceHandler resource_handler=new ResourceHandler();
> > > >> >             resource_handler.setResourceBase("somewhere");
> > > >> >             server.setHandler(resource_handler);
> > > >> >             server.start();
> > > >> > tearDown code:
> > > >> >             server.stop();
> > > >> >
> > > >> >
> > > >>
> > > >>
> > > >> --
> > > >> Richard Liang
> > > >> China Development Lab, IBM
> >
> >
> ------------------------------------------------------
> 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
>
>


-- 
Richard Liang
China Development Lab, IBM

---------------------------------------------------------------------
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


Mime
View raw message