test: make multi-byte randomFill tests flaky-safe

Signed-off-by: kyungrae <kyungrae2002@gmail.com>

Author: kyungrae2002

test/parallel/test-crypto-random.js

@@ -219,47 +219,54 @@ common.expectWarning('DeprecationWarning',
 }
 
 {
-  const buf = new Uint16Array(4);
+  const buf = new Uint16Array(10);
   const before = Buffer.from(buf.buffer).toString('hex');
-  crypto.randomFillSync(buf, 1, 1);
+  crypto.randomFillSync(buf, 1, 8);
   const after = Buffer.from(buf.buffer).toString('hex');
   assert.notStrictEqual(before, after);
   assert.deepStrictEqual(before.slice(0, 4), after.slice(0, 4));
-  assert.deepStrictEqual(before.slice(8), after.slice(8));
+  assert.deepStrictEqual(before.slice(-4), after.slice(-4));
 }
 
 {
-  const buf = new Uint32Array(4);
+  const buf = new Uint32Array(10);
   const before = Buffer.from(buf.buffer).toString('hex');
-  crypto.randomFillSync(buf, 1, 1);
+  crypto.randomFillSync(buf, 1, 8);
   const after = Buffer.from(buf.buffer).toString('hex');
   assert.notStrictEqual(before, after);
   assert.deepStrictEqual(before.slice(0, 8), after.slice(0, 8));
-  assert.deepStrictEqual(before.slice(16), after.slice(16));
+  assert.deepStrictEqual(before.slice(-8), after.slice(-8));
 }
 
 {
-  const buf = new Uint16Array(4);
+  const buf = new Uint16Array(10);
   const before = Buffer.from(buf.buffer).toString('hex');
-  crypto.randomFill(buf, 1, 1, common.mustSucceed((buf) => {
+  crypto.randomFill(buf, 1, 8, common.mustSucceed((buf) => {
     const after = Buffer.from(buf.buffer).toString('hex');
     assert.notStrictEqual(before, after);
     assert.deepStrictEqual(before.slice(0, 4), after.slice(0, 4));
-    assert.deepStrictEqual(before.slice(8), after.slice(8));
+    assert.deepStrictEqual(before.slice(-4), after.slice(-4));
   }));
 }
 
 {
-  const buf = new Uint32Array(4);
+  const buf = new Uint32Array(10);
   const before = Buffer.from(buf.buffer).toString('hex');
-  crypto.randomFill(buf, 1, 1, common.mustSucceed((buf) => {
+  crypto.randomFill(buf, 1, 8, common.mustSucceed((buf) => {
     const after = Buffer.from(buf.buffer).toString('hex');
     assert.notStrictEqual(before, after);
     assert.deepStrictEqual(before.slice(0, 8), after.slice(0, 8));
-    assert.deepStrictEqual(before.slice(16), after.slice(16));
+    assert.deepStrictEqual(before.slice(-8), after.slice(-8));
   }));
 }
 
+{
+  // randomFill() with an offset and no size must not throw for types
+  // without a .length property, matching randomFillSync().
+  crypto.randomFill(new ArrayBuffer(10), 2, common.mustSucceed());
+  crypto.randomFill(new DataView(new ArrayBuffer(10)), 2, common.mustSucceed());
+}
+
 {
   [
     Buffer.alloc(10),