Very cool. Thanks Kazuhito. What would be excellent is if you could also provide some test cases proving that it works. You can either reuse an existing test case class if it makes sense or create a new one. That will enrich our test suite and not lower our test coverage! :-)
I've had a brief look at the patch. It looks good. Maybe, as a second step, it would be good to factorize both your check and the ones that already exist into a single place. ATM, there are lots of checks in the ClientTestCaseCaller.callGeneric*Method(...) It would be excellent if all the checks could be gathered in one place. <future idea> I've always had the idea of creating AspectJ aspects that will verify that of the environment in which the Cactus framework executes in order to verify everything is setup correctly. That could include verifying test cases are written correctly too. The reason for an aspect is that these checks have to happen in lots of different locations but they are implementing the same concern: - they are not code that deliver a business result - they are checking environment health - they know how to deal with errors Thus I think it makes an excellent candidate for some aspects. </future idea> Thanks! -Vincent > -----Original Message----- > From: Kazuhito SUGURI [mailto:[EMAIL PROTECTED] > Sent: 12 March 2004 04:46 > To: [EMAIL PROTECTED]; [EMAIL PROTECTED] > Subject: Re: Cactus/test name is null > > Hi Vincent, > > In article <[EMAIL PROTECTED]>, > Sat, 6 Mar 2004 08:40:25 +0100, > "Vincent Massol" <[EMAIL PROTECTED]> wrote: > vmassol> Yep, I had the same thought. Also cactus could check for the > static > vmassol> keyword in testXXX. Just need to find some time. If anyone wants > to > vmassol> submit a patch... > > I had some time yesterday. > > Appending is a patch against AbstractCactusTestCase.java, v 1.7. > I'm not sure if its feature is what you are looking for. > > Also, I'm not sure why the static keyword check is needed. > Would you mind explaining a little further, please? > # better to change cactus-dev ML? > > Regards, > ---- > Kazuhito SUGURI > mailto:[EMAIL PROTECTED] --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
