RFR 8075618: Create tests to check jarsigner work with multi-version jar

[Amanda Jiang]

Hi Paul, Artem and Max,

Thanks for your comments.  Please check the new webrev at: 
http://cr.openjdk.java.net/~amjiang/8075618/webrev.03/  (It looks like 
"cr.openjdk.java.net" is down, I also attach the webrev with this email).

In the new webrev, I fixed some syntax issue mentioned by Paul and also 
simplied the test codes with Artem and Max's suggestions.  Unfortunately 
I cannot fully apply some functions from 
http://hg.openjdk.java.net/jdk9/dev/jdk/file/d4d7f1f0d688/test/lib/security/SecurityTools.java 
. Those functions returns deprecated "OutputAnalyzer " class in 
http://hg.openjdk.java.net/jdk9/dev/jdk/file/ab164f8b8569/test/lib/testlibrary/jdk/testlibrary/OutputAnalyzer.java 
,  which conflicts with the "OubputAnalyzer" class I imported for my 
test ( 
http://hg.openjdk.java.net/jdk9/dev/file/5e79c9bac1b5/test/lib/jdk/test/lib/process/OutputAnalyzer.java)

Max,

I also add one test case for permission granted to signed multi-release 
jar files, other functional tests are covered by 
http://hg.openjdk.java.net/jdk9/dev/jdk/file/ab164f8b8569/test/java/util/jar/JarFile/mrjar/MultiReleaseJarSecurity.java

Thanks
Amanda
On 12/12/16 6:31 PM, Wang Weijun wrote:
> Hi Amanda
>
> Can you also test the new JarSigner API?
>
>     http://hg.openjdk.java.net/jdk9/dev/jdk/rev/ce33c780cfbd
>
> line 2.72 has an example on it.
>
>> On Dec 13, 2016, at 9:22 AM, Artem Smotrakov <artem.smotrakov at oracle.com> wrote:
>>
>> You can use http://hg.openjdk.java.net/jdk9/dev/jdk/file/d4d7f1f0d688/test/lib/security/SecurityTools.java which would simplify the code. This lib was added to be used in such tests.
> Correct. I also wonder if there are existing methods on javac compilation.
>
> Do we have existing tests on checking if a signed multi-version jar works as expected? Say, permission granted, getCertificates() returning non-null, etc?
>
> Thanks
> Max
>

-------------- next part --------------
A non-text attachment was scrubbed...
Name: webrev.zip
Type: application/zip
Size: 60311 bytes
Desc: not available
URL: <https://mail.openjdk.org/pipermail/security-dev/attachments/20161218/ebb217fc/webrev.zip>