From e3d9d25083fe022015d7ee6b07fe3212bb3dde2b Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Mon, 12 Oct 2015 20:48:41 -0700 Subject: [PATCH] test: add test-child-process-emfile fail message When the test fails (as it does frequently on FreeBSD unfortunately) provide a non-cryptic error message that also provides a line number. PR-URL: https://github.com/nodejs/node/pull/3335 Reviewed-By: Ben Noordhuis --- test/sequential/test-child-process-emfile.js | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/test/sequential/test-child-process-emfile.js b/test/sequential/test-child-process-emfile.js index c4c467c3392482..ae7964d9e7a7d6 100644 --- a/test/sequential/test-child-process-emfile.js +++ b/test/sequential/test-child-process-emfile.js @@ -27,8 +27,10 @@ proc.on('error', common.mustCall(function(err) { assert(err.code === 'EMFILE' || err.code === 'ENFILE'); })); -// 'exit' should not be emitted, the process was never spawned. -proc.on('exit', assert.fail); +proc.on('exit', function() { + const msg = '"exit" should not be emitted (the process never spawned!)'; + assert.fail(null, null, msg); +}); // close one fd for LSan if (openFds.length >= 1) {