Skip to content

Commit 5614dc3

Browse files
legalcodestargos
authored andcommitted
test: check error message in test-fs-make-callback
Add a RegExp to `throws` assertions. PR-URL: #10914 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Italo A. Casas <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 087516a commit 5614dc3

File tree

1 file changed

+10
-9
lines changed

1 file changed

+10
-9
lines changed

test/parallel/test-fs-make-callback.js

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
const common = require('../common');
33
const assert = require('assert');
44
const fs = require('fs');
5+
const cbTypeError = /^TypeError: "callback" argument must be a function$/;
56

67
function test(cb) {
78
return function() {
@@ -26,13 +27,13 @@ process.once('warning', common.mustCall((warning) => {
2627
assert.doesNotThrow(test());
2728

2829
function invalidArgumentsTests() {
29-
assert.throws(test(null));
30-
assert.throws(test(true));
31-
assert.throws(test(false));
32-
assert.throws(test(1));
33-
assert.throws(test(0));
34-
assert.throws(test('foo'));
35-
assert.throws(test(/foo/));
36-
assert.throws(test([]));
37-
assert.throws(test({}));
30+
assert.throws(test(null), cbTypeError);
31+
assert.throws(test(true), cbTypeError);
32+
assert.throws(test(false), cbTypeError);
33+
assert.throws(test(1), cbTypeError);
34+
assert.throws(test(0), cbTypeError);
35+
assert.throws(test('foo'), cbTypeError);
36+
assert.throws(test(/foo/), cbTypeError);
37+
assert.throws(test([]), cbTypeError);
38+
assert.throws(test({}), cbTypeError);
3839
}

0 commit comments

Comments
 (0)