On Mon, 2010-01-04 at 20:17 +0000, Stephen Connolly wrote: > I see the common code being aabstract base class that extends > abstractmojo
Probably safest. Your pseudocode showing a delegated method call just had my mouth watering for a smart way to transfer all the wired properties and mojo configuration onto a separate object. > but I'll take a look while refactoring. my main concern is that the > surefire test coverage is not as good as I'd like (andi guess i'm not > the only one, given the slow release rate) > I think these two things are related. From a testing perspective it is almost impossible to properly test code with such a large number of if/then/elses combined as a fairly monolithic block. But by the looks of it, you're not going to change much at all...? I've had my eye on increasing testability of surefire for quite a few years, but I agree that's a different battle to be fought. Kristian --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
