From ac35f0fec81db8c5d9f71c3b430140dd55007b68 Mon Sep 17 00:00:00 2001 From: Liran Tal Date: Thu, 23 Jul 2015 14:06:08 +0300 Subject: [PATCH] adding couple more tests to confirm users model works as expected --- .../tests/server/user.server.model.tests.js | 42 ++++++++++++++++++- 1 file changed, 41 insertions(+), 1 deletion(-) diff --git a/modules/users/tests/server/user.server.model.tests.js b/modules/users/tests/server/user.server.model.tests.js index 9f0eeead3c..6a0db65257 100644 --- a/modules/users/tests/server/user.server.model.tests.js +++ b/modules/users/tests/server/user.server.model.tests.js @@ -10,7 +10,7 @@ var should = require('should'), /** * Globals */ -var user, user2; +var user, user2, user3; /** * Unit tests @@ -35,6 +35,15 @@ describe('User Model Unit Tests:', function() { password: 'password', provider: 'local' }); + user3 = new User({ + firstName: 'Different', + lastName: 'User', + displayName: 'Full Different Name', + email: 'test3@test.com', + username: 'different_username', + password: 'different_password', + provider: 'local' + }); done(); }); @@ -77,6 +86,37 @@ describe('User Model Unit Tests:', function() { done(); }); }); + + it('should be able to save 2 different users', function(done) { + user.remove(function(err) { + should.not.exist(err); + user.save(function(err) { + user3.save(function(err) { + should.not.exist(err); + user3.remove(function(err) { + should.not.exist(err); + done(); + }); + + }); + }); + }); + }); + + it('should not be able to save different user with the same email address', function(done) { + user.remove(function(err) { + should.not.exist(err); + user.save(function(err) { + user3.email = user.email; + user3.save(function(err) { + should.exist(err); + done(); + }); + }); + }); + + }); + }); after(function(done) {