Skip to content

Commit 1544d8f

Browse files
toboidjasnell
authored andcommitted
test: improve test-buffer-includes.js
* verify error message * use arrow funcs PR-URL: #11203 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Yuta Hiroto <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent f65aa08 commit 1544d8f

File tree

1 file changed

+9
-7
lines changed

1 file changed

+9
-7
lines changed

test/parallel/test-buffer-includes.js

+9-7
Original file line numberDiff line numberDiff line change
@@ -275,15 +275,17 @@ for (let lengthIndex = 0; lengthIndex < lengths.length; lengthIndex++) {
275275
}
276276
}
277277

278-
assert.throws(function() {
279-
b.includes(function() { });
280-
});
281-
assert.throws(function() {
278+
const expectedError =
279+
/^TypeError: "val" argument must be string, number, Buffer or Uint8Array$/;
280+
assert.throws(() => {
281+
b.includes(() => {});
282+
}, expectedError);
283+
assert.throws(() => {
282284
b.includes({});
283-
});
284-
assert.throws(function() {
285+
}, expectedError);
286+
assert.throws(() => {
285287
b.includes([]);
286-
});
288+
}, expectedError);
287289

288290
// test truncation of Number arguments to uint8
289291
{

0 commit comments

Comments
 (0)