1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: add property for RangeError in test-buffer-copy

Browse Source 
PR-URL: https://github.com/nodejs/node/pull/23968
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
This commit is contained in:
mritunjaygoutam12 2018-10-30 12:59:02 +05:30 committed by Rich Trott
parent 0d9d32ad15
commit a6e836a657
1 changed files with 13 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
test/parallel/test-buffer-copy.js
View File

@ -1,10 +1,17 @@
'use strict';
require('../common');
const common = require('../common');
const assert = require('assert');
const b = Buffer.allocUnsafe(1024);
const c = Buffer.allocUnsafe(512);
const errorProperty = {
code: 'ERR_OUT_OF_RANGE',
type: RangeError,
message: 'Index out of range'
};
let cntr = 0;
{
@ -96,9 +103,9 @@ bb.fill('hello crazy world');
b.copy(c, 0, 100, 10);
// copy throws at negative sourceStart
assert.throws(function() {
Buffer.allocUnsafe(5).copy(Buffer.allocUnsafe(5), 0, -1);
}, RangeError);
common.expectsError(
() => Buffer.allocUnsafe(5).copy(Buffer.allocUnsafe(5), 0, -1),
errorProperty);
{
// check sourceEnd resets to targetEnd if former is greater than the latter
@ -111,7 +118,8 @@ assert.throws(function() {
}
// throw with negative sourceEnd
assert.throws(() => b.copy(c, 0, 0, -1), RangeError);
common.expectsError(
() => b.copy(c, 0, -1), errorProperty);
// when sourceStart is greater than sourceEnd, zero copied
assert.strictEqual(b.copy(c, 0, 100, 10), 0);