Hi John,
Sorry for the delay in reviewing this and thanks for the patience.
The changes look fine in general, however I feel that it may be
worthwhile to put the Solaris version+distro checking/detection code
into the general test base class, i.e. UcryptoTest, as a utility method.
This way, other tests can also leverage the code to skip certain cases
if needed.
Thanks,
Valerie
On 6/21/2017 7:40 PM, [email protected] wrote:
Hi,
According to JDK-8173708, the cases on CFB128 in test
com/oracle/security/ucrypto/TestAES.java should be skipped on Solaris
11.2 and previous versions due to a Solaris bug.
Please review the patch at:
http://cr.openjdk.java.net/~jjiang/8177017/webrev.00/
Best regards,
John Jiang
