RFR: JDK-8295087: Manual Test to Automated Test Conversion [v5]
Bill Huang
bhuang at openjdk.org
Tue Dec 20 19:13:12 UTC 2022
On Fri, 16 Dec 2022 12:06:14 GMT, Sibabrata Sahoo <ssahoo at openjdk.org> wrote:
>> Bill Huang has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision:
>>
>> - Merge branch 'master' into JDK-8295087
>> - Added an extra line to the end of the policy file.
>> - AssertThrows an exception in InconsistentEntries test.
>> - Refactored to use testng framework for test enviroment setup.
>> - Converted security manual tests to automated tests.
>
> test/jdk/sun/security/provider/PolicyParser/ExtDirs.java line 37:
>
>> 35: * @summary standard extensions path is hard-coded in default
>> 36: * system policy file
>> 37: * @run main/othervm/policy=ExtDirs.policy/secure=default ExtDirs
>
> May be "/secure=default" is optional here.
Yes, you are right and we can remove the /secure. According to the JTReg spec.
`If the /secure option is not used then the default security manager will be installed.`
-------------
PR: https://git.openjdk.org/jdk/pull/10637
More information about the security-dev
mailing list