Hello Daniel, On 29/08/19 2:56 PM, Daniel Fuchs wrote: > Thanks for the test Jaikiran! > > Since an IOException is expected at line 76, I believe > it would be good for the test to trow an assertion error > if that doesn't happen. > > Maybe simply insert: > throw new AssertionError("expected exception not thrown"); > between lines 75 and 76?
You are right of course, I missed that. I have now updated the test and the webrev is at the same location http://cr.openjdk.java.net/~jpai/webrev/8230310/2/webrev/ -Jaikiran