ComputerSurge/shields
3
0
Fork 0
Code Pull Requests 1 Activity

Fix tests to wait for server launch test stub to finish starting

Browse Source
This commit is contained in:
Mike Taylor
2015-06-10 11:45:40 -06:00
parent 7e9eae14e4
commit dd54c25541
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
ass-stubs/server-test.js
View File

@@ -1,5 +1,5 @@
// We don't require ass because we can't test a majority of the server reliably. // We don't require ass because we can't test a majority of the server reliably.
//require('ass'); //require('ass');
require('../server.js'); require('../server.js');
console.log('done'); console.log('ready');
process.on('SIGTERM', function() { process.exit(0); }); process.on('SIGTERM', function() { process.exit(0); });

2
test.js
View File

@@ -74,7 +74,7 @@ describe('the server', function() {
server = cproc.spawn('node', ['ass-stubs/server-test.js', port]); server = cproc.spawn('node', ['ass-stubs/server-test.js', port]);
var isDone = false; var isDone = false;
server.stdout.on('data', function(data) { server.stdout.on('data', function(data) {
if (!isDone) { done(); isDone = true; } if (data.toString().indexOf('ready') >= 0 && !isDone) { done(); isDone = true; }
}); });
server.stderr.on('data', function(data) { console.log(''+data); }); server.stderr.on('data', function(data) { console.log(''+data); });
}); });