> -----Original Message----- > From: Christopher Lenz [mailto:[EMAIL PROTECTED] > Sent: 24 February 2003 12:02 > To: Cactus Developers List > Subject: Re: [Proposal] Package Refactoring Proposal > > Vincent, > > I think it'd be interesting to get a JDepend report into our framework > build process. That would help us identify problems with the current > package structure as well as with any structure we'll be moving to. > > It's pretty easy to do (even without using Maven ;-) ), and I'd give it > a shot when a find some spare hours. What do you think?
Hum... One of the reason I wanted a cleaner package structure is that I needed to perform some refactoring to decouple classes. For example, I would like to be able to use the FormAuthentication class both from a ServletTestCase and from a StaticTestCase (a Static test case simply calls a URL and has no testXXX() methods - only begin and end ones). However, FormAuthentication requires a WebRequest and does not currently work with a BaseWebRequest. Thus, we should not rely too much on the current dependencies as they may not be correct evyerwhere. That said, I have never been able to get anything useful from jdepend (not that it isn't nice, just that I always don't know what to do after reading the report). If you could demonstrate it is useful for us, sure please go ahead! :-) -Vincent > > Vincent Massol wrote: > [snip lots] > > Indeed, there are several options: > > > > 1/ o.a.c.client.servlet and o.a.c.server.servlet > > 2/ o.a.c.servlet.client and o.a.c.servlet.server > > 3/ o.a.c.servlet > > > > but then there are lots of other support classes like: > > AbstractClientTestCase, AbstractWebClientTestCase, > > AbstractWebServerTestCase, etc. > > > > where do we put these? I wanted to put them in o.a.c.testcase: > > > > o.a.c.testcase > > o.a.c.testcase.servlet > > o.a.c.testcase.jsp > > [...] > > > > As you mention, there are 2 functional aspects: Servlet/JSP/etc vs > > TestCase/Redirector/Connectors/etc. ATM, I was more following the later > > route (i.e. functional WRT what the class does) rather than the former > > (i.e. functional WRT a test domain). > [snip lots more] > > -- > Christopher Lenz > /=/ cmlenz at gmx.de > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
