fix: shut down sauce connect properly

Closes #5
karma-runner · Jul 17, 2013 · 17f94f8 · 17f94f8
1 parent e31123f
commit 17f94f8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 8 deletions.
diff --git a/index.js b/index.js
@@ -32,10 +32,6 @@ var SauceConnect = function(emitter, logger) {
 
     alreadyRunningDefered = q.defer();
     launchSauceConnect(options, function(err, p) {
-      if (onKilled) {
-        return onKilled();
-      }
-
       alreadyRunningProces = p;
       alreadyRunningDefered.resolve();
     });
@@ -46,8 +42,7 @@ var SauceConnect = function(emitter, logger) {
   emitter.on('exit', function(done) {
     if (alreadyRunningProces) {
       log.info('Shutting down Sauce Connect');
-      onKilled = done;
-      alreadyRunningProces.close();
+      alreadyRunningProces.close(done);
     } else {
       done();
     }
@@ -103,7 +98,7 @@ var SauceLabsBrowser = function(id, args, sauceConnect, /* config.sauceLabs */ c
       'build': args.build || config.build || process.env.TRAVIS_BUILD_NUMBER ||
               process.env.BUILD_NUMBER || process.env.BUILD_TAG ||
               process.env.CIRCLE_BUILD_NUM || null
-               
+
     };
 
     url = url + '?id=' + id;

diff --git a/package.json b/package.json
@@ -20,7 +20,7 @@
   "author": "Vojta Jina <vojta.jina@gmail.com>",
   "dependencies": {
     "wd": "~0.0.32",
-    "sauce-connect-launcher": "~0.1.5",
+    "sauce-connect-launcher": "~0.1.10",
     "q": "~0.9.6"
   },
   "peerDependencies": {