[9] RFR: 8156054: Test Task: Develop new tests for JEP C155: Remove FilePermission Pathname Canonicalization

Artem Smotrakov artem.smotrakov at oracle.com
Tue Sep 13 17:24:09 UTC 2016


Hi Siba,

I see that the test expects only true or false. You can just pass these 
boolean values, and check() can make sure that everything returns the 
expected boolean value without building a string. That would be clearer 
to me. Not an issue, it's up to you to change it or not.

Artem


On 09/13/2016 03:14 AM, Sibabrata Sahoo wrote:
>
> Hi,
>
> Please review the patch for,
>
> JBS: https://bugs.openjdk.java.net/browse/JDK-8156054
>
> Webrev: http://cr.openjdk.java.net/~ssahoo/8156054/webrev.00/ 
> <http://cr.openjdk.java.net/%7Essahoo/8156054/webrev.00/>
>
> Description:
>
> It checks equals(), implies() and hashCode () of FilePermission when 
> “jdk.io.permissionsUseCanonicalPath” set and un-set. Along with, it 
> also verify compatibility with previous JDK version.
>
> Thanks,
>
> Siba
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/security-dev/attachments/20160913/99f159b0/attachment.htm>


More information about the security-dev mailing list