TheThing/nconf-lite
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

fix random fails on tests that use child process

Browse source 
Listen to 'close' event rather than 'exit' event which can be fired before
stdio is closed.
This commit is contained in:
Pierre Beaujeu 2015-07-09 17:33:52 +02:00
parent d335b5a0f5
commit 339e59afd5
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
test/hierarchy-test.js
View file

@ -62,7 +62,7 @@ vows.describe('nconf/hierarchy').addBatch({
data += d; data += d;
}); });
child.on('exit', function () { child.on('close', function () {
fs.readFile(configFile, 'utf8', that.callback.bind(null, null, data)); fs.readFile(configFile, 'utf8', that.callback.bind(null, null, data));
}); });
}, },
@ -90,7 +90,7 @@ vows.describe('nconf/hierarchy').addBatch({
data += d; data += d;
}); });
child.on('exit', function() { child.on('close', function() {
that.callback(null, data); that.callback(null, data);
}); });
}, },