Hi Kevin I tried to read the new test. TestMDChange and TestMDShare are OK, but I am confused on what testDigestOnOff is doing. Can you add some comments there?
Thanks Max On Aug 18, 2014, at 14:35, zaiyao liu <zaiyao....@oracle.com> wrote: > Hi Brad, > > Can you help to review this new test for DigestIOStream? > > Thanks > > Kevin > δΊ 2014/8/1 11:10, zaiyao liu ει: >> May I request you to review 1 new test to be added for messageDigest with >> DigestIOStream. New test are added to address following: >> - Test DigestInputStream and DigestOutputStream digest function when digest >> set on and off >> - Test DigestInputStream and DigestOutputStream digest function when Swap >> the message digest engines between DigestIn/OutputStream >> - Test DigestInputStream and DigestOutputStream digest function when use >> same message digest object. >> >> >> JDK Issue: https://bugs.openjdk.java.net/browse/JDK-8050370 >> Webrev: http://cr.openjdk.java.net/~rhalade/8050370/webrev.00/ >> >> Thanks, >> >> Kevin >