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