On Thu, 11 Jul 2002, Cedric Dumoulin wrote:
> Date: Thu, 11 Jul 2002 17:13:00 +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
>
> Craig R. McClanahan wrote:
>
> I have try to run "ant test.junit" and "ant test.tomcat.all" on the
> nightly build, and they report some errors. As I have just installed
> cactus, I don't really know if it is my configuration or the nightly build !
> So, does someone have run successfully the test suites on the nightly
> build ?
>
Both tests run successfully against my current CVS repository (which
should match the 20020711 nightly build). What errors are you getting?
Setting up Cactus was pretty painful. I found it easiest to download the
"jakarta-cactus-ant-1.5-20020421.zip" file and grab the JARs out of there,
to put into my $ANT_HOME/lib directory.
> Cedric
>
Craig
> >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]>
