test: wrap functions containing assertions inside a common.mustCall

sagirk · Nov 21, 2018 · 91fa0e1 · 91fa0e1
1 parent 0767eba
commit 91fa0e1
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/test/parallel/test-child-process-env.js b/test/parallel/test-child-process-env.js
@@ -53,10 +53,10 @@ child.stdout.on('data', (chunk) => {
   response += chunk;
 });
 
-process.on('exit', () => {
+process.on('exit', common.mustCall(() => {
   assert.ok(response.includes('HELLO=WORLD'));
   assert.ok(response.includes('FOO=BAR'));
   assert.ok(!response.includes('UNDEFINED=undefined'));
   assert.ok(response.includes('NULL=null'));
   assert.ok(response.includes(`EMPTY=${os.EOL}`));
-});
+}));