getUser renamed to fetchUser

Author: xbill82

src/security/kuzzleSecurity.js

@@ -423,23 +423,23 @@ Security.prototype.profile = function(id, content) {
  * @param {object|responseCallback} [options] - (optional) arguments
  * @param {responseCallback} cb - returns Kuzzle's response
  */
-Security.prototype.getUser = function (id, options, cb) {
+Security.prototype.fetchUser = function (id, options, cb) {
   var
     data = {_id: id},
     self = this;
 
   if (!id || typeof id !== 'string') {
-    throw new Error('Id parameter is mandatory for getUser function');
+    throw new Error('Id parameter is mandatory for fetchUser function');
   }
 
   if (!cb && typeof options === 'function') {
     cb = options;
     options = null;
   }
 
-  self.kuzzle.callbackRequired('Security.getUser', cb);
+  self.kuzzle.callbackRequired('Security.fetchUser', cb);
 
-  self.kuzzle.query(this.buildQueryArgs('getUser'), data, options, function (err, response) {
+  self.kuzzle.query(this.buildQueryArgs('fetchUser'), data, options, function (err, response) {
     cb(err, err ? undefined : new User(self, response.result._id, response.result._source));
   });
 };

test/security/kuzzleSecurity/constructor.test.js

@@ -32,7 +32,7 @@ describe('kuzzleSecurity constructor', function () {
     should.exist(kuzzle.security.searchProfilesPromise);
     should.exist(kuzzle.security.deleteProfilePromise);
     should.exist(kuzzle.security.createUserPromise);
-    should.exist(kuzzle.security.getUserPromise);
+    should.exist(kuzzle.security.fetchUserPromise);
     should.exist(kuzzle.security.searchUsersPromise);
     should.exist(kuzzle.security.deleteUserPromise);
 

test/security/kuzzleSecurity/userMethods.test.js

@@ -40,21 +40,21 @@ describe('Security user methods', function () {
       }
     };
 
-  describe('#getUser', function () {
+  describe('#fetchUser', function () {
     beforeEach(function () {
       kuzzle = new Kuzzle('foo', {defaultIndex: 'bar'});
       kuzzle.query = queryStub;
       error = null;
       result = { result: {_id: 'foobar', _source: {profileIds: ['profile']}}};
       expectedQuery = {
-        action: 'getUser',
+        action: 'fetchUser',
         controller: 'security',
         _id: 'foobar'
       };
     });
 
     it('should send the right query to Kuzzle', function (done) {
-      should(kuzzle.security.getUser(result.result._id, function (err, res) {
+      should(kuzzle.security.fetchUser(result.result._id, function (err, res) {
         should(err).be.null();
         should(res).be.instanceof(User);
 
@@ -66,7 +66,7 @@ describe('Security user methods', function () {
     });
 
     it('should send the right query to Kuzzle with id as profile', function (done) {
-      should(kuzzle.security.getUser(result.result._id, function (err, res) {
+      should(kuzzle.security.fetchUser(result.result._id, function (err, res) {
         should(err).be.null();
         should(res).be.instanceof(User);
 
@@ -78,18 +78,18 @@ describe('Security user methods', function () {
     });
 
     it('should raise an error if no callback is provided', function () {
-      should(function () { kuzzle.security.getUser('test'); }).throw(Error);
+      should(function () { kuzzle.security.fetchUser('test'); }).throw(Error);
     });
 
     it('should throw an error when no id is provided', function () {
-      should(function () { kuzzle.security.getUser(null, function () {}); }).throw(Error);
+      should(function () { kuzzle.security.fetchUser(null, function () {}); }).throw(Error);
     });
 
     it('should call the callback with an error if one occurs', function (done) {
       error = 'error';
       this.timeout(50);
 
-      kuzzle.security.getUser('foobar', function (err, res) {
+      kuzzle.security.fetchUser('foobar', function (err, res) {
         should(err).be.exactly('error');
         should(res).be.undefined();
         done();