Skip to content

Commit 6274eb7

Browse files
authored
test: use assert.match for non-literal regexp tests
PR-URL: #60879 Refs: #60832 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
1 parent 6a1a3ba commit 6274eb7

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

test/parallel/test-max-old-space-size-percentage.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ invalidPercentages.forEach((input) => {
4141
`--max-old-space-size-percentage=${input[0]}`,
4242
], { stdio: ['pipe', 'pipe', 'pipe'] });
4343
assert.notStrictEqual(result.status, 0, `Expected non-zero exit for invalid input ${input[0]}`);
44-
assert(input[1].test(result.stderr.toString()), `Unexpected error message for invalid input ${input[0]}`);
44+
assert.match(result.stderr.toString(), input[1]);
4545
});
4646

4747
// Test NODE_OPTIONS with valid percentages
@@ -61,7 +61,7 @@ invalidPercentages.forEach((input) => {
6161
env: { ...process.env, NODE_OPTIONS: `--max-old-space-size-percentage=${input[0]}` }
6262
});
6363
assert.notStrictEqual(result.status, 0, `NODE_OPTIONS: Expected non-zero exit for invalid input ${input[0]}`);
64-
assert(input[1].test(result.stderr.toString()), `NODE_OPTIONS: Unexpected error message for invalid input ${input[0]}`);
64+
assert.match(result.stderr.toString(), input[1]);
6565
});
6666

6767
// Test percentage calculation validation

test/parallel/test-tls-junk-server.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ server.listen(0, common.mustCall(function() {
2828
expectedErrorMessage = new RegExp('packet length too long');
2929
};
3030
req.once('error', common.mustCall(function(err) {
31-
assert(expectedErrorMessage.test(err.message));
31+
assert.match(err.message, expectedErrorMessage);
3232
server.close();
3333
}));
3434
}));

0 commit comments

Comments
 (0)