On 09/07/2018 11:38, Srinivas Dama wrote:
Hi,

Please review
webrev: http://cr.openjdk.java.net/~sdama/8206445/webrev.00/
Bug: https://bugs.openjdk.java.net/browse/JDK-8206445

testListEmptyFile looks okay but you can reduce the code if if you use Path.of("."). testListNotAnImage can use Path.of(".") too. Also it can use Files.writeString which will avoid the test trying to list the contents of the image while the file is open.

Will you make sure to test this on all platforms before pushing? These tests have been on and off the exclude list several times in the last week and it would be good to check that this one is stable before it runs again.

-Alan

Reply via email to