From 071ecb0dd2bb4665b9d9a09ca3f08d6d870d935b Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Sat, 24 Jun 2017 20:11:43 -0700 Subject: [PATCH] test: refactor test-child-process-send-type-error * Add exit listener to child process to check return code. Previously, child process faiilure would not cause the test to fail. * Use common.mustNotCall() to guarantee callback is not invoked. * Insert blank line per test writing guide. PR-URL: https://github.com/nodejs/node/pull/13904 Reviewed-By: Colin Ihrig Reviewed-By: James M Snell Reviewed-By: Refael Ackermann --- test/parallel/test-child-process-send-type-error.js | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/test/parallel/test-child-process-send-type-error.js b/test/parallel/test-child-process-send-type-error.js index 40c415666f20cc..00263a5d21550f 100644 --- a/test/parallel/test-child-process-send-type-error.js +++ b/test/parallel/test-child-process-send-type-error.js @@ -1,5 +1,6 @@ 'use strict'; const common = require('../common'); + const assert = require('assert'); const cp = require('child_process'); @@ -11,8 +12,13 @@ function fail(proc, args) { let target = process; -if (process.argv[2] !== 'child') +if (process.argv[2] !== 'child') { target = cp.fork(__filename, ['child']); + target.on('exit', common.mustCall((code, signal) => { + assert.strictEqual(code, 0); + assert.strictEqual(signal, null); + })); +} fail(target, ['msg', null, null]); fail(target, ['msg', null, '']); @@ -20,4 +26,4 @@ fail(target, ['msg', null, 'foo']); fail(target, ['msg', null, 0]); fail(target, ['msg', null, NaN]); fail(target, ['msg', null, 1]); -fail(target, ['msg', null, null, common.noop]); +fail(target, ['msg', null, null, common.mustNotCall()]);