On Thu, 11 Jul 2002, Cedric Dumoulin wrote:
> Date: Thu, 11 Jul 2002 12:36:50 +0200
> From: Cedric Dumoulin <[EMAIL PROTECTED]>
> Reply-To: Struts Developers List <[EMAIL PROTECTED]>
> To: Struts Developers List <[EMAIL PROTECTED]>
> Subject: Re: Running Unit Tests Before Commits
>
> Sorry for the inconvenience.
> I install the test process and will run it before any further commit.
> Do you recommend to do testing for any tomcat version (3 and 4), or is
> the latest one enough ?
>
I do my tests on the latest released 4.0 and 4.1 versions (4.0.4 and 4.1.7
at the moment). I'm not particularly concerned with 3.2, because it is so
ancient.
Running "ant test.tomcat.all" will run the Cactus based tests against both
if you've defined "tomcat.home.40" and "tomcat.home.41" properties.
> Cedric
>
Craig
> Craig R. McClanahan wrote:
>
> >Doing the fix for 7751, I ran into a case where the undo of the "/" adding
> >change caused the unit tests to fail, because I'd added some tests for the
> >new behavior. I've commented those tests out until we figure out what to
> >do about that particular situation.
> >
> >I'd like to ask all Struts committers to follow these practices:
> >
> >* When fixing a bug, build a unit test to verify the corrected behavior
> > if this is at all reasonable to do. There are some mock objects in
> > the src/test hierarchy now, so you should be able to write JUnit tests
> > for pretty much any class other than the tag implementations.
> >
> >* Always run "ant test.junit" and "ant test.tomcat.all" before checking
> > in your changes, to ensure that we don't introduce any regressions on
> > stuff that is already tested.
> >
> >Thanks!
> >
> >Craig
> >
> >
> >
> >--
> >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]>
