COMPASS-147: SSH Tunnel Refactoring (#563)

* Prevent multiple connections in same window

* Disconnect data-service on successful connection test

Now that we’re in 1 window, this would create orphaned connections

* COMPASS-147: Update mongodb-connection-model with ssh tunnel rewrite

Author: imlucas

package.json

@@ -124,7 +124,7 @@
     "moment": "^2.10.6",
     "mongodb": "^2.2.8",
     "mongodb-collection-model": "^0.3.0",
-    "mongodb-connection-model": "^6.2.0",
+    "mongodb-connection-model": "^6.3.0",
     "mongodb-data-service": "^2.0.0",
     "mongodb-database-model": "^0.1.2",
     "mongodb-explain-plan-model": "^0.2.1",

src/app/index.js

@@ -335,6 +335,10 @@ app.extend({
     return this.preferences.isFeatureEnabled(feature);
   },
   setConnectionId: function(connectionId, done) {
+    if (state.connection && state.connection.getId() === connectionId) {
+      debug('Already connected to connectionId', connectionId);
+      return done();
+    }
     var StatusAction = app.appRegistry.getAction('Status.Actions');
     StatusAction.configure({
       visible: true,

src/app/models/connection.js

@@ -72,16 +72,18 @@ module.exports = Connection.extend(storageMixin, {
     }
   },
   test: function(done) {
+    var dataService = new DataService(this);
     var onTested = function(err) {
       if (err) {
         metrics.error(err);
         return done(err);
       }
 
       debug('test worked!');
+      dataService.disconnect();
       done(null, this);
     }.bind(this);
-    var dataService = new DataService(this);
+
     debug('Testing connection to `%j`...', this.serialize());
     dataService.connect(onTested);
     return this;