Skip to content

Commit 23c58f0

Browse files
authored
test: replace deprecated regex test assertions in http trailers test
PR-URL: #60831 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Chemi Atlow <[email protected]> Reviewed-By: LiviaMedeiros <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
1 parent 7955ef1 commit 23c58f0

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

test/parallel/test-http-set-trailers.js

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -43,8 +43,9 @@ function testHttp10(port, callback) {
4343

4444
c.on('end', common.mustCall(() => {
4545
c.end();
46-
assert.ok(
47-
!/x-foo/.test(res_buffer),
46+
assert.doesNotMatch(
47+
res_buffer,
48+
/x-foo/,
4849
`No trailer in HTTP/1.0 response. Response buffer: ${res_buffer}`
4950
);
5051
callback();
@@ -68,8 +69,9 @@ function testHttp11(port, callback) {
6869
res_buffer += chunk;
6970
if (/0\r\n/.test(res_buffer)) { // got the end.
7071
clearTimeout(tid);
71-
assert.ok(
72-
/0\r\nx-foo: bar\r\n\r\n$/.test(res_buffer),
72+
assert.match(
73+
res_buffer,
74+
/0\r\nx-foo: bar\r\n\r\n$/,
7375
`No trailer in HTTP/1.1 response. Response buffer: ${res_buffer}`
7476
);
7577
callback();

0 commit comments

Comments
 (0)