Merge pull request #229 from kuzzleio/create-user-w-null-id

Allow creating a user without specifying the id

Author: stafyniaksacha

src/security/Security.js

@@ -562,10 +562,6 @@ Security.prototype.createUser = function (id, content, options, cb) {
     self = this,
     data = {_id: id, body: content};
 
-  if (!id || typeof id !== 'string') {
-    throw new Error('Security.createUser: cannot create a user without a user ID');
-  }
-
   if (!cb && typeof options === 'function') {
     cb = options;
     options = null;
@@ -619,10 +615,6 @@ Security.prototype.createRestrictedUser = function (id, content, options, cb) {
     self = this,
     data = {_id: id, body: content};
 
-  if (!id || typeof id !== 'string') {
-    throw new Error('Security.createRestrictedUser: cannot create a user without a user ID');
-  }
-
   if (content.profileIds) {
     throw new Error('Security.createRestrictedUser: cannot provide profileIds');
   }

test/security/kuzzleSecurity/userMethods.test.js

@@ -160,11 +160,6 @@ describe('Security user methods', function () {
       should(kuzzle.query).calledWith(expectedQuery, {_id: 'foobar', body: content}, null, undefined);
     });
 
-    it('should throw an error if no id provided', function () {
-      should(function () { kuzzle.security.createUser(null); }).throw(Error);
-      should(kuzzle.query).not.be.called();
-    });
-
     it('should call the callback with an error if one occurs', function (done) {
       this.timeout(50);
 
@@ -209,11 +204,6 @@ describe('Security user methods', function () {
       should(kuzzle.query).calledWith(expectedQuery, {_id: 'foobar', body: content}, null, undefined);
     });
 
-    it('should throw an error if no id provided', function () {
-      should(function () { kuzzle.security.createRestrictedUser(null); }).throw(Error);
-      should(kuzzle.query).not.be.called();
-    });
-
     it('should throw an error if profileIds is provided', function () {
       should(function () { kuzzle.security.createRestrictedUser('foobar', {profileIds: ['someProfile']}); }).throw(Error);
       should(kuzzle.query).not.be.called();