21 Added a new replaceUser method to Security

Author: samniisan

src/security/Security.js

@@ -518,6 +518,33 @@ Security.prototype.createUser = function (id, content, options, cb) {
   });
 };
 
+/**
+ * Replace an user in Kuzzle.
+ *
+ * @param {string} id - user identifier
+ * @param {object} content - attribute `profileIds` in `content` must only contain an array of profile ids
+ * @param {object|responseCallback} [options] - (optional) arguments
+ * @param {responseCallback} [cb] - (optional) Handles the query response
+ */
+Security.prototype.replaceUser = function (id, content, options, cb) {
+  var
+    self = this,
+    data = {_id: id, body: content};
+
+  if (!id || typeof id !== 'string') {
+    throw new Error('Security.replaceUser: cannot replace a user without a user ID');
+  }
+
+  if (!cb && typeof options === 'function') {
+    cb = options;
+    options = null;
+  }
+
+  self.kuzzle.query(this.buildQueryArgs('replaceUser'), data, options, cb && function (err, res) {
+    cb(err, err ? undefined : new User(self, res.result._id, res.result._source));
+  });
+};
+
 /**
  * Create a new restricted user in Kuzzle.
  *

test/security/kuzzleSecurity/userMethods.test.js

@@ -316,6 +316,43 @@ describe('Security user methods', function () {
     });
   });
 
+  describe('#replaceUser', function () {
+    beforeEach(function () {
+      kuzzle = new Kuzzle('foo', {defaultIndex: 'bar'});
+      kuzzle.query = queryStub;
+      error = null;
+      result = { result: {_id: 'foobar', _index: '%kuzzle', _type: 'users', _source: {profileIds: ['foobar']} } };
+      expectedQuery = {
+        action: 'replaceUser',
+        controller: 'security'
+      };
+    });
+
+    it('should send the right query to Kuzzle', function(done) {
+      expectedQuery._id = 'foobar';
+      expectedQuery.body = {'profileIds': ['foobar']};
+
+      should(kuzzle.security.replaceUser(expectedQuery._id, expectedQuery.body, function (err, res) {
+        should(err).be.null();
+        should(res).be.instanceOf(User);
+        should(res).containDeep(new User(kuzzle.security, result.result._id, result.result._source));
+        done();
+      }));
+    });
+
+    it('should send the right query to Kuzzle even without callback', function (done) {
+      expectedQuery.body = {'foo': 'bar'};
+      expectedQuery._id = result.result._id;
+
+      kuzzle.security.replaceUser(result.result._id, {'foo': 'bar'});
+      done();
+    });
+
+    it('should throw an error if no id is provided', function () {
+      should(function () { kuzzle.security.replaceUser(null, {'foo': 'bar'}); }).throw(Error);
+    });
+  });
+
   describe('#updateUser', function () {
     beforeEach(function () {
       kuzzle = new Kuzzle('foo', {defaultIndex: 'bar'});