Merge pull request #92 from kuzzleio/fix-91-kuzzle-profile-format

[release-1.9.1] Fix #91: format correctly roles within profile definition

Author: ballinette

CHANGELOG.md

@@ -1,3 +1,9 @@
+*__note:__ the # at the end of lines are the pull request numbers on GitHub*
+
+# 1.9.1
+
+* Fix issue #91
+
 # 1.9.0
 
 ## Features

package.json

@@ -1,6 +1,6 @@
 {
   "name": "kuzzle-sdk",
-  "version": "1.9.0",
+  "version": "1.9.1",
   "description": "Official Javascript SDK for Kuzzle",
   "author": "The Kuzzle Team <[email protected]>",
   "repository": {

src/security/kuzzleSecurity.js

@@ -289,7 +289,15 @@ KuzzleSecurity.prototype.getProfile = function (id, options, cb) {
 
     if (!hydrate) {
       response.result._source.roles = response.result._source.roles.map(function (role) {
-        return role._id;
+        var formattedRole = {_id: role._id};
+        if (role._source.restrictedTo !== undefined) {
+          formattedRole.restrictedTo = role._source.restrictedTo;
+        }
+        if (role._source.allowInternalIndex !== undefined) {
+          formattedRole.allowInternalIndex = role._source.allowInternalIndex;
+        }
+
+        return formattedRole;
       });
     }
 

test/security/kuzzleSecurity/profilesMethods.test.js

@@ -78,7 +78,7 @@ describe('KuzzleSecurity profiles methods', function () {
         should(res.content.roles).not.be.empty();
 
         res.content.roles.forEach(function (role) {
-          should(role).be.a.String();
+          should(role._id).not.be.empty().and.be.a.String();
         });
         done();
       }));