verdnatura-mirror
/
loopback
mirror of https://github.com/strongloop/loopback.git
0
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #2440 from strongloop/issue414_token2 

Fix verificationToken bug
This commit is contained in:
Loay 2016-06-17 11:12:16 -04:00 committed by GitHub
parent 03e85b838d ec51e833b6
commit 45e523d66c
2 changed files with 16 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
common/models/user.js
View File

@ -485,7 +485,7 @@ module.exports = function(User) {
fn(err); fn(err);
} else { } else {
if (user && user.verificationToken === token) { if (user && user.verificationToken === token) {
user.verificationToken = undefined; user.verificationToken = null;
user.emailVerified = true; user.emailVerified = true;
user.save(function(err) { user.save(function(err) {
if (err) { if (err) {

15
test/user.test.js
View File

@ -1418,6 +1418,21 @@ describe('User', function() {
}, done); }, done);
}); });
it('sets verificationToken to null after confirmation', function(done) {
testConfirm(function(result, done) {
User.confirm(result.uid, result.token, false, function(err) {
if (err) return done(err);
// Verify by loading user data stored in the datasource
User.findById(result.uid, function(err, user) {
if (err) return done(err);
expect(user).to.have.property('verificationToken', null);
done();
});
});
}, done);
});
it('Should report 302 when redirect url is set', function(done) { it('Should report 302 when redirect url is set', function(done) {
testConfirm(function(result, done) { testConfirm(function(result, done) {
request(app) request(app)