Add test for Issue saintedlama#96

test/alternative-query-field.js

@@ -5,7 +5,7 @@ var passportLocalMongoose = require('../lib/passport-local-mongoose');
 var mongotest = require('./mongotest');
 
 describe('alternative query field', function () {
-    beforeEach(mongotest.prepareDb('mongodb://localhost/passportlocalmongooseissues'));
+    beforeEach(mongotest.prepareDb('mongodb://boot2docker/passportlocalmongooseissues'));
     afterEach(mongotest.disconnect());
 
     it('should find an existing user by alternative query field', function (done) {
@@ -21,7 +21,7 @@ describe('alternative query field', function () {
         var user = new User({ username : 'hugo', email : email });
         user.save(function (err) {
             assert.ifError(err);
-            
+
             User.findByUsername(email, function (err, user) {
                 assert.ifError(err);
                 assert.ok(user);
@@ -31,7 +31,7 @@ describe('alternative query field', function () {
             });
         });
     });
-    
+
     it('should authenticate an existing user by alternative query field', function (done) {
         this.timeout(5000); // Five seconds - mongo db access needed
 
@@ -45,7 +45,7 @@ describe('alternative query field', function () {
         var user = new User({ username : 'hugo', email : email });
         User.register(user, 'password', function (err) {
             assert.ifError(err);
-            
+
             User.authenticate()('[email protected]', 'password', function(err, user, message) {
                 assert.ifError(err);
                 assert.ok(user);
@@ -54,8 +54,8 @@ describe('alternative query field', function () {
                 done();
             });
         });
-    });  
-    
+    });
+
     it('should authenticate an existing user by default username field', function (done) {
         this.timeout(5000); // Five seconds - mongo db access needed
 
@@ -69,7 +69,7 @@ describe('alternative query field', function () {
         var user = new User({ username : 'hugo', email : email });
         User.register(user, 'password', function (err) {
             assert.ifError(err);
-            
+
             User.authenticate()('hugo', 'password', function(err, user, message) {
                 assert.ifError(err);
                 assert.ok(user);
@@ -78,8 +78,8 @@ describe('alternative query field', function () {
                 done();
             });
         });
-    });    
-    
+    });
+
     it('should not authenticate an existing user by unconfigured alternative query field', function (done) {
         this.timeout(5000); // Five seconds - mongo db access needed
 
@@ -93,7 +93,7 @@ describe('alternative query field', function () {
         var user = new User({ username : 'hugo', email : email });
         User.register(user, 'password', function (err) {
             assert.ifError(err);
-            
+
             User.authenticate()('[email protected]', 'password', function(err, user, message) {
                 assert.ifError(err);
                 assert.ok(!user);
@@ -102,5 +102,5 @@ describe('alternative query field', function () {
                 done();
             });
         });
-    });  
-});
+    });
+});

test/issues.js

@@ -5,7 +5,7 @@ var passportLocalMongoose = require('../lib/passport-local-mongoose');
 var mongotest = require('./mongotest');
 
 describe('issues', function () {
-    beforeEach(mongotest.prepareDb('mongodb://localhost/passportlocalmongooseissues'));
+    beforeEach(mongotest.prepareDb('mongodb://boot2docker/passportlocalmongooseissues'));
     afterEach(mongotest.disconnect());
 
     it('should support nested fields - Issue #9', function(done) {
@@ -169,4 +169,32 @@ describe('issues', function () {
             });
         });
     });
-});
+
+    it.only('authentication should work with salt/hash field marked as select: false - Issue #96', function(done) {
+        this.timeout(5000); // Five seconds - mongo db access needed
+
+        var UserSchema = new Schema({});
+
+        UserSchema.plugin(passportLocalMongoose, { });
+        var User = mongoose.model('ShouldAuthenticateWithSaltAndHashNotExposed_Issue_96', UserSchema);
+
+        User.register({username: 'nicolascage'}, 'password', function (err, user) {
+            assert.ifError(err);
+            assert.ok(user);
+            User.findOne({username: 'nicolascage'}, function(err, user) {
+                assert.ifError(err);
+                assert.ok(user);
+                assert.equal(user.username, 'nicolascage');
+				user.authenticate('password', function(err, auth, reason) {
+					console.log(err)
+					assert.ifError(err);
+
+					console.log(reason);
+                    assert.ok(auth);
+
+					done();
+				});
+            });
+        });
+    });
+});

test/passport-local-mongoose.js

@@ -173,12 +173,15 @@ describe('passportLocalMongoose', function () {
 
             var user = new DefaultUser();
 
-            setPasswordAndAuthenticate(user, 'password', 'nopassword', function (err, result) {
-                assert.ifError(err);
-                assert.ok(result === false);
+			var testUser = DefaultUser.findById(user._id, function(err, user) {
 
-                done();
-            });
+				setPasswordAndAuthenticate(user, 'password', 'nopassword', function (err, result) {
+					assert.ifError(err);
+					assert.ok(result === false);
+
+					done();
+				});
+			});
         });
 
         it('should supply a message when authentication fails', function (done) {
@@ -196,7 +199,7 @@ describe('passportLocalMongoose', function () {
     });
 
     describe('static #authenticate()', function () {
-        beforeEach(mongotest.prepareDb('mongodb://localhost/passportlocalmongoosetests'));
+        beforeEach(mongotest.prepareDb('mongodb://boot2docker/passportlocalmongoosetests'));
         afterEach(mongotest.disconnect());
 
         it('should yield false with message option for authenticate', function (done) {
@@ -334,7 +337,7 @@ describe('passportLocalMongoose', function () {
                                   debugger;
                                     expect(err).to.not.exist;
                                     expect(result).to.be.false;
-                                    
+
                                     done();
                                 });
                             });
@@ -436,7 +439,7 @@ describe('passportLocalMongoose', function () {
     });
 
     describe('static #deserializeUser()', function () {
-        beforeEach(mongotest.prepareDb('mongodb://localhost/passportlocalmongoosetests'));
+        beforeEach(mongotest.prepareDb('mongodb://boot2docker/passportlocalmongoosetests'));
         afterEach(mongotest.disconnect());
 
         it('should define a static deserializeUser function for passport', function () {
@@ -480,7 +483,7 @@ describe('passportLocalMongoose', function () {
     });
 
     describe('static #findByUsername()', function () {
-        beforeEach(mongotest.prepareDb('mongodb://localhost/passportlocalmongoosetests'));
+        beforeEach(mongotest.prepareDb('mongodb://boot2docker/passportlocalmongoosetests'));
         afterEach(mongotest.disconnect());
 
         it('should define static findByUsername helper function', function () {
@@ -607,7 +610,7 @@ describe('passportLocalMongoose', function () {
     });
 
     describe('static #register()', function () {
-        beforeEach(mongotest.prepareDb('mongodb://localhost/passportlocalmongoosetests'));
+        beforeEach(mongotest.prepareDb('mongodb://boot2docker/passportlocalmongoosetests'));
         afterEach(mongotest.disconnect());
 
         it('should define static register helper function', function () {
@@ -673,7 +676,7 @@ describe('passportLocalMongoose', function () {
                 });
             });
         });
-        
+
         it('should not authenticate registered user with wrong password', function (done) {
             this.timeout(5000); // Five seconds - mongo db access needed
 
@@ -693,7 +696,7 @@ describe('passportLocalMongoose', function () {
                 });
             });
         });
-        
+
         it('it should add username existing user without username', function (done) {
             this.timeout(5000); // Five seconds - mongo db access needed