Code Review 6935199: java/net regression tests failing with Assertions (original) (raw)

Michael McMahon Michael.McMahon at Sun.COM
Tue Mar 16 06:54:06 PDT 2010

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.

More information about the net-dev mailing list