- add an ability to retrieve roles for a specific user by its id

Author: Valodya

backendless.d.ts

@@ -297,7 +297,7 @@ declare module Backendless {
 
         function findByRole<T = Backendless.User>(roleName: string, loadRoles?: boolean, query?: Backendless.DataQueryBuilder | DataQueryI): Promise<T[]>;
 
-        function getUserRoles(): Promise<string[]>;
+        function getUserRoles(userId?: string): Promise<string[]>;
 
         function describeUserClass(): Promise<Object[]>;
 

src/urls.js

@@ -290,8 +290,8 @@ export default class Urls {
     return `${this.users()}/logout`
   }
 
-  userRoles() {
-    return `${this.users()}/userroles`
+  userRoles(userId) {
+    return `${this.users()}/userroles${userId ? `/${userId}` : ''}`
   }
 
   usersRole(roleName) {

src/users/index.js

@@ -266,8 +266,8 @@ export default class Users {
     return this.roles.findByRole(roleName, loadRoles, query)
   }
 
-  async getUserRoles() {
-    return this.roles.getUserRoles()
+  async getUserRoles(userId) {
+    return this.roles.getUserRoles(userId)
   }
 
   async assignRole(identity, rolename) {

src/users/roles.js

@@ -37,9 +37,9 @@ export default class UsersRoles {
       .then(result => this.users.dataStore.parseResponse(result))
   }
 
-  async getUserRoles() {
+  async getUserRoles(userId) {
     return this.app.request.get({
-      url: this.app.urls.userRoles(),
+      url: this.app.urls.userRoles(userId),
     })
   }
 

test/tsd.ts

@@ -808,6 +808,7 @@ function testUserService() {
     promiseObject = Backendless.UserService.register(newUser);
 
     promiseListOfString = Backendless.UserService.getUserRoles();
+    promiseListOfString = Backendless.UserService.getUserRoles('test-user-id');
 
     promiseVoid = Backendless.UserService.assignRole(identity, roleName);
 

test/unit/helpers/sandbox.js

@@ -113,11 +113,11 @@ function wrapRequest(R) {
   }
 }
 
-export function prepareMockRequest(responseFn) {
+export function prepareMockRequest(response) {
   const mockRequest = {
-    responseFn: typeof responseFn === 'function'
-      ? responseFn
-      : () => ({ body: responseFn })
+    responseFn: typeof response === 'function'
+      ? response
+      : () => ({ body: response })
   }
 
   mockRequests.push(mockRequest)

test/unit/specs/users/roles.js

@@ -261,17 +261,34 @@ describe('<Users> Roles', function() {
     })
   })
 
-  it('gets user roles', async () => {
-    const req1 = prepareMockRequest()
+  it('gets current user roles', async () => {
+    const req1 = prepareMockRequest(['role-1','role-2'])
 
-    await Backendless.UserService.getUserRoles()
+    const result = await Backendless.UserService.getUserRoles()
 
     expect(req1).to.deep.include({
       method : 'GET',
       path   : `${APP_PATH}/users/userroles`,
       headers: {},
       body   : undefined
     })
+
+    expect(result).to.be.eql(['role-1','role-2'])
+  })
+
+  it('gets a specific user roles', async () => {
+    const req1 = prepareMockRequest(['role-1','role-2'])
+
+    const result = await Backendless.UserService.getUserRoles('user-id')
+
+    expect(req1).to.deep.include({
+      method : 'GET',
+      path   : `${APP_PATH}/users/userroles/user-id`,
+      headers: {},
+      body   : undefined
+    })
+
+    expect(result).to.be.eql(['role-1','role-2'])
   })
 
   it('adds user roles', async () => {