[9] RFR: 8048596: Tests for AEAD ciphers
Artem Smotrakov
artem.smotrakov at oracle.com
Fri Jul 24 10:15:07 UTC 2015
Hi Valerie,
I thought about random data again. I agree that it doesn't make much
difference if random data is used instead of static one. And as a
result, in this case 'randomness' key may be confusing (someone may want
to filter out tests which use random data for some reason).
I updated the tests not to use random data:
- removed dependency on RandomFactory
- removed 'randomness' tag
Please take a look:
http://cr.openjdk.java.net/~asmotrak/8048596/webrev.03/
Artem
On 07/22/2015 11:36 PM, Valerie Peng wrote:
>
> Yeah, many ways to generate bytes. It's generally ok to just re-use
> the same input for various test scenarios in the same test class.
> As far as I am concerned, the benefit of get data in one line doesn't
> quite justify the extra dependency.
>
> The updated webrev looks fine.
> Valerie
>
> On 7/20/2015 11:33 PM, Artem Smotrakov wrote:
>> Hi Valerie,
>>
>> The tests can easily get data in one line with RandomFacroty. But
>> they can use static data that may be created with something like the
>> following:
>>
>> public class Helper {
>>
>> public static byte[] generateBytes(int length) {
>> byte[] bytes = new byte[length];
>> for (int i=0; i<length; i++) {
>> bytes[i] = (byte) (i % 256);
>> }
>> return bytes;
>> }
>> }
>>
>> Please take a look an updated webrev:
>>
>> http://cr.openjdk.java.net/~asmotrak/8048596/webrev.02/
>>
>> Artem
>>
>> On 07/21/2015 12:19 AM, Valerie Peng wrote:
>>> Hi Artem,
>>>
>>> Just some nit (see below). In general, I find the tests don't need
>>> to use so many random bytes. If we don't need RandomFactory, then no
>>> dependence on jdk.testlibrary. Make things easier to execute the
>>> test on its own if necessary. Just something to keep in mind for
>>> future test development.
>>>
>>> <Encrypt.java>
>>> - line 186: typo in "intiate". I think you mean initiate the cipher
>>> without parameters? I don't see how the parameters are saved here.
>>> - line 190: I think either "generated" or "specified" is better than
>>> "saved".
>>>
>>> <GCMParameterSpecTest.java>
>>> - line 117: template not reporting key length?
>>> - line 219: getInstance with "SunJCE"?
>>>
>>> <SameBuffer.java>
>>> - line 110: this check can be done earlier, e.g. on line 108.
>>>
>>> Thanks,
>>> Valerie
>>>
>>> On 7/10/2015 1:02 PM, Artem Smotrakov wrote:
>>>> Hello,
>>>>
>>>> Please review a couple of new tests for AEAD ciphers.
>>>>
>>>> Webrev: http://cr.openjdk.java.net/~asmotrak/8048596/webrev.01/
>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8048596
>>>>
>>>> Artem
>>
More information about the security-dev
mailing list