On Fri, 3 Mar 2023 14:00:09 GMT, Kevin Rushforth <k...@openjdk.org> wrote:

>> Jose Pereda has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Add system test
>
> tests/system/src/test/resources/test/sandbox/empty.policy line 2:
> 
>> 1: // Policy file for sandbox tests
>> 2: // Use as follows: -Djava.security.policy=test.policy
> 
> Minor: should be `empty.policy`

Done

> tests/system/src/test/resources/test/sandbox/empty.policy line 4:
> 
>> 2: // Use as follows: -Djava.security.policy=test.policy
>> 3: 
>> 4: // No permissions to read resource files are granted
> 
> Minor: maybe also say that all FX permissions are granted?

Right, I've added only the required permission for the test.

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

PR: https://git.openjdk.org/jfx/pull/1042

Reply via email to