From 75ad329b4fbf3273ffbe04c37ceced12e1906390 Mon Sep 17 00:00:00 2001 From: Andrey Sidorov Date: Sat, 8 Mar 2014 18:04:28 +1100 Subject: [PATCH] fix tests --- test/connect.js | 7 +++++++ test/errors.js | 3 +-- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/test/connect.js b/test/connect.js index 4ce9c33..e6df0e4 100644 --- a/test/connect.js +++ b/test/connect.js @@ -52,9 +52,16 @@ describe('Client', function() { }); it('returns error when connecting to non existent display', function(done) { + var errorCbCalled = false; var client = x11.createClient({ display : ':44' }, function(err, display) { assert(util.isError(err)); + errorCbCalled = true; done(); }); + // TODO: stop writing to socket after first error + client.on('error', function() { + if (!errorCbCalled) + done('should not reach here before first done()'); + }); }); }); diff --git a/test/errors.js b/test/errors.js index 3ac867a..2ff8332 100644 --- a/test/errors.js +++ b/test/errors.js @@ -6,7 +6,7 @@ describe('Client', function() { var display; beforeEach(function(done) { - var client = x11.createClient(function(err, dpy) { + var client = x11.createClient({ debug: true }, function(err, dpy) { console.log(err) display = dpy; done(err); @@ -14,7 +14,6 @@ describe('Client', function() { }); it('should emit error which is instance of Error with sequence number corresponding to source request', function(done) { - display.client.options.debug = true; display.client.CreateWindow(); // should emit error var seq = display.client.seq_num; display.client.once('error', function(err) {