RFR: 8256202: Some tweaks for jarsigner tests PosixPermissionsTest and SymLinkTest
Matthias Baesken
mbaesken at openjdk.java.net
Thu Nov 12 06:46:56 UTC 2020
On Thu, 12 Nov 2020 06:44:10 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:
>> Working on 11u backports of JDK-8218021 and JDK-8250968, I found some minor points for improvement in tests
>> test/jdk/sun/security/tools/jarsigner/PosixPermissionsTest.java and
>> test/jdk/sun/security/tools/jarsigner/SymLinkTest.java
>>
>> The details
>>
>> PosixPermissionsTest:
>> - it can run on any system, no matter if the default filesystem supports Posix or not since Posix support is only required for the zipfs which is always true.
>> - improve some comments for the test flow
>>
>> SymLinkTest:
>> - make output of createByteArray prettier
>> - improve inline comments to ease understanding
>> - add an option to main for generating ZIPBYTES
>> - use "Files.write(Path.of(ZIPFILENAME), ZIPBYTES)" for creating the test zipfile
>
> Marked as reviewed by mbaesken (Reviewer).
Looks good to me !
-------------
PR: https://git.openjdk.java.net/jdk/pull/1166
More information about the security-dev
mailing list