https://issues.apache.org/bugzilla/show_bug.cgi?id=47124





--- Comment #2 from Marc Guillemot <mguille...@yahoo.fr>  2009-05-04 05:07:33 
PST ---
I'm glad to see that you've committed some changes, nevertheless for me it
doesn't really change the situation that unit tests are not considered
seriously in Tomcat project.

Tests *should* be integral part of the build. It's normal that a build takes
(far) longer than 5 minutes if it runs tests (nevertheless these tests should
have some sense to be worth this time, what is perhaps not yet the case).

I'm sorry if I didn't followed the convention for external libs. I thought that
it was the case. What about using Ivy rather than this self made "strategy"?

This is is far less important that running the tests as part of the build, but
I think that the tests should be renamed for consistency and as it just allows
to use name patterns to select the tests (currently there is no convention
here). In the current state of the test suite, I would prefer to select the
tests by name pattern rather than having a TestAll as it doesn't have any
advantage and this file needs to be modified each time a test is added (with
the risk to have tests that aren't run).

I don't understand either why you don't use Ant's junit task.

At the end I have the feeling that this issue has been closed as WONTFIX rather
than as FIXED :-(

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to