Code Review 6935199: java/net regression tests failing with Assertions

[Christopher Hegarty -Sun Microsystems Ireland]

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