Michael McMahon wrote:
Christopher Hegarty -Sun Microsystems Ireland wrote:
Michael McMahon wrote:
Christopher Hegarty -Sun Microsystems Ireland wrote:
Hi Michael,
The following tests have incorrect assertion failures if run with
assertions enabled:
test/java/net/CookieHandler/TestHttpCookie.java
Would it be better to just remove the assertion in the first one as
well?
I don't think it really matters. The assertion just verifies that the
that the construction is not called with null, which seems reasonable.
It is just this one specific test that doesn't conform since it
verifies that NPE is actually thrown. If you feel strongly I can
remove it.
Personally, I think it would be better. It doesn't seem right to have
code that works
around an assertion.
OK, I'll just remove it.
-Chris.
- Michael
