🐛 fixing serialization

Author: imlucas

src/connect/auth-fields.js

@@ -78,6 +78,7 @@ var kerberos_password = new InputView({
  * Define fields for each auth method here
  */
 module.exports = {
+  NONE: [],
   MONGODB: [
     mongodb_username,
     mongodb_password,
@@ -89,5 +90,5 @@ module.exports = {
     kerberos_password
   ],
   X509: [],
-  PLAIN: []
+  LDAP: []
 };

src/connect/index.js

@@ -31,7 +31,14 @@ var ConnectView = View.extend({
     form: 'object',
     authMethod: {
       type: 'string',
-      default: null
+      default: 'NONE',
+      values: [
+        'NONE',
+        'MONGODB',
+        'KERBEROS',
+        'X509',
+        'LDAP'
+      ]
     },
     previousAuthMethod: {
       type: 'string',
@@ -156,7 +163,7 @@ var ConnectView = View.extend({
     if (this.authOpen) {
       this.authMethod = this.previousAuthMethod || 'MONGODB';
     } else {
-      this.authMethod = null;
+      this.authMethod = 'NONE';
     }
   },
   /**
@@ -198,14 +205,14 @@ var ConnectView = View.extend({
     this.reset();
     this.form.connection_id = '';
     this.form.reset();
-    this.authMethod = null;
+    this.authMethod = 'NONE';
     this.authOpen = false;
   },
   onConnectionDestroyed: function() {
     this.reset();
     this.form.connection_id = '';
     this.form.reset();
-    this.authMethod = null;
+    this.authMethod = 'NONE';
     this.authOpen = false;
   },
 
@@ -362,6 +369,7 @@ var ConnectView = View.extend({
   onConnectionSelected: function(model) {
     // If the new model has auth, expand the auth settings container
     // and select the correct tab.
+    model.authentication = model.authentication || 'NONE';
     this.authMethod = model.authentication;
 
     if (model.authentication !== 'NONE') {

src/models/connection.js

@@ -26,15 +26,15 @@ module.exports = Connection.extend({
   test: function(done) {
     var model = this;
     debug('Testing connection to `%j`...', this);
-    client.test(app.endpoint, this, function(err) {
+    client.test(app.endpoint, model.serialize({all: true}), function(err) {
       if (err) {
         bugsnag.notify(err, 'connection test failed');
         return done(err);
       }
 
       debug('test worked!');
       debug('making sure we can get collection list...');
-      client(app.endpoint, this).instance(function(err, res) {
+      client(app.endpoint, model.serialize({all: true})).instance(function(err, res) {
         if (!err) {
           debug('woot.  all gravy!  able to see %s collections', res.collections.length);
           done(null, model);