>> Let’s try again, this time there are tests.  Please review the following 
>> webrev that adds support for multi-release jars as specified in JEP-238.
> Good to have tests this time :-)
> 
> I see several JAR files in the webrev, shouldn't the tests create these so 
> that we don't have to check in binary files? I realize the signed case might 
> be a bit of work but there may be some security tests that could provide 
> examples.

The current test directory contains binary jar files. In fact in all the test 
directories, there are 52 binary .jar files. I added three more.  I thought 
about generating the jar files but the problem I run into is that there are two 
test classes that use the same files and creating the files twice seemed to be 
a bit wasteful.  I couldn’t figure out a reliable way to have one test class 
create the files and the other one just use the files.

Reply via email to