[9] RFR: 8076486: javax/security/auth/Subject/doAs/NestedActions.java fails if extra VM options are given

[Weijun Wang]

Great. It's now at http://hg.openjdk.java.net/jdk9/dev/jdk/rev/fff8ab918557.

Thanks
Max

On 5/6/2015 12:01 AM, Artem Smotrakov wrote:
> Hi Max,
>
> It works fine if a sting starts with a whitespace. It looks like that
> jtreg removes whitespaces. Here is a command to run the test, and a
> piece of jtr file:
>
> ${JAVA_HOME}/bin/java -jar /home/artem/tools/jtreg/4.1/lib/jtreg.jar -a
> -v -testjdk ${JAVA_HOME} -javaoptions:"     -XX:+UseG1GC
> -XX:+CreateMinidumpOnCrash     "
> -dir:/home/artem/ws/jdk/jdk9_dev_jaas_nested_action_tests/jdk/test
> javax/security/auth/Subject/doAs/NestedActions.java
>
> ...
> /home/artem/jdk/jdk1.9.0b61/bin/java \\
>          -Dtest.vm.opts= \\
>          -Dtest.jdk=/home/artem/jdk/jdk1.9.0b61 \\
>          -Dtest.timeout.factor=1.0 \\
> -Dtest.src.path=/home/artem/ws/jdk/jdk9_dev_jaas_nested_action_tests/jdk/test/javax/security/auth/Subject/doAs:/home/artem/ws/jdk/jdk9_dev_jaas_nested_action_tests/jdk/test/lib/testlibrary
> \\
>          -Dtest.compiler.opts= \\
>          -Dcompile.jdk=/home/artem/jdk/jdk1.9.0b61 \\
> -Dtest.classes=/home/artem/results/jaas_nested_action_tests/JTwork/classes/javax/security/auth/Subject/doAs
> \\
> -Dtest.class.path=/home/artem/results/jaas_nested_action_tests/JTwork/classes/javax/security/auth/Subject/doAs:/home/artem/results/jaas_nested_action_tests/JTwork/classes/lib/testlibrary
> \\
>          -Dtest.java.opts='-XX:+UseG1GC -XX:+CreateMinidumpOnCrash' \\
> -Dtest.src=/home/artem/ws/jdk/jdk9_dev_jaas_nested_action_tests/jdk/test/javax/security/auth/Subject/doAs
> \\
>          -Dtest.tool.vm.opts= \\
>          -XX:+UseG1GC -XX:+CreateMinidumpOnCrash \\
>          com.sun.javatest.regtest.MainWrapper
> /home/artem/results/jaas_nested_action_tests/JTwork/classes/javax/security/auth/Subject/doAs/NestedActions.jta
> NestedActionsTwoPrincipals policy.two.principals
> NestedActionsTwoPrincipals.jar WriteToFileAction.jar
> ReadFromFileAction.jar ReadPropertyAction.jar
> ...
>
> But anyway, I added trim(), so now the test doesn't rely on jtreg.
> Please take a look:
>
> http://cr.openjdk.java.net/~asmotrak/8076486/webrev.02/
>
> Artem
>
> On 05/04/2015 11:49 AM, Weijun Wang wrote:
>> Artem
>>
>> If a string starts with whitespace(s), the first element of
>> split("\\s+") will be an empty string. If you can be sure JAVA_OPTS
>> does not have them, your code change is fine.
>>
>> Thanks
>> Max
>>
>> On 4/29/2015 11:44 PM, Artem Smotrakov wrote:
>>> Hi Max,
>>>
>>> These fancy things sometimes make me forget about simpler ways :-)
>>>
>>> Please take a look
>>>
>>> http://cr.openjdk.java.net/~asmotrak/8076486/webrev.01/
>>>
>>> Thanks!
>>>
>>> Artem
>>>
>>> On 04/29/2015 05:04 PM, Weijun Wang wrote:
>>>> Maybe you can call JAVA_OPTS.trim().split("\\s+") to avoid the
>>>> filtering.
>>>>
>>>> And it seems Collections.addAll() can add an array to a list.
>>>>
>>>> Of course, you are free to exercise any fancy jdk8 features. :-)
>>>>
>>>> Thanks
>>>> Max
>>>>
>>>> On 4/29/2015 3:23 PM, Artem Smotrakov wrote:
>>>>> Hello,
>>>>>
>>>>> Please review this fix for
>>>>> javax/security/auth/Subject/doAs/NestedActions.java test.
>>>>>
>>>>> The test runs java in a separate process. It fails if multiple java
>>>>> options are passed because ProcessBuilder expects each parameter to be
>>>>> put in to a separate array element, but the test didn't do that. I
>>>>> updated the test to split java option string, and put each option in a
>>>>> separate array element.
>>>>>
>>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8076486
>>>>> Webrev: http://cr.openjdk.java.net/~asmotrak/8076486/webrev.00/
>>>>>
>>>>> Artem
>>>
>