On Mon, 20 Sep 2021 17:05:33 GMT, Severin Gehwolf <sgehw...@openjdk.org> wrote:

>> Hi all,
>> 
>> Please review this patch which addresses the issue where Zip FS will throw a 
>> UOE instead of returning null  when  Files.getFileAttributeView() is invoked 
>> and  the view not supported.
>> 
>> Mach5 tiers1 - tier3 are clean.
>> 
>> Best
>> Lance
>
> test/jdk/jdk/nio/zipfs/testng/test/PosixAttributeViewTest.java line 100:
> 
>> 98:             PosixFileAttributeView view = 
>> Files.getFileAttributeView(entry,
>> 99:                     PosixFileAttributeView.class);
>> 100:             System.out.printf("View returned: %s%n", view);
> 
> It might be useful to also print whether the environment is empty:
> Suggestion: `System.out.printf("View returned: %s, (empty_env=%s)%n", view, 
> env.isEmpty());`

Well, you can see that whether the Map is/is not empty in the jtr file for the 
test run (based on the DataProvider Properties for a given run).

However, per your suggestion, I tweaked the test to dump the Map values.

-------------

PR: https://git.openjdk.java.net/jdk/pull/5579

Reply via email to