Skip to content

Commit b5d0a03

Browse files
TrottFishrock123
authored andcommitted
test: fix error handling test-http-full-response
The way it is currently written, test-http-full-response will fail if there is a problem with spawning that doesn't include `ab` or `api` in `stderr`, but it will fail with a misleading mismatched-calls `common.mustCall()` error. Alter the error handling so that it rethrows the actual error, providing better information. PR-URL: #14252 Reviewed-By: Yuta Hiroto <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 32ba8ae commit b5d0a03

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

test/parallel/test-http-full-response.js

+1-2
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,7 @@ function runAb(opts, callback) {
4646
common.printSkipMessage(`problem spawning \`ab\`.\n${stderr}`);
4747
process.reallyExit(0);
4848
}
49-
process.exit();
50-
return;
49+
throw err;
5150
}
5251

5352
let m = /Document Length:\s*(\d+) bytes/i.exec(stdout);

0 commit comments

Comments
 (0)