Add support for query params for getRoles method (#530)

* feat(roles): add tests

feat(roles): adjust tests

* feat(roles): add query options for getRoles

Author: ruderngespra

lib/create-space-api.ts

@@ -995,9 +995,10 @@ export default function createSpaceApi({
      * .catch(console.error)
      * ```
      */
-    getRoles() {
+    getRoles(query: QueryOptions = {}) {
+      normalizeSelect(query)
       return http
-        .get('roles')
+        .get('roles', createRequestConfig({ query }))
         .then((response) => wrapRoleCollection(http, response.data), errorHandler)
     },
 

test/integration/role-integration.js

@@ -20,13 +20,38 @@ const roleDefinition = {
 
 export default function roleTests(t, space) {
   t.test('Gets roles', (t) => {
-    t.plan(2)
+    t.plan(3)
     return space.getRoles().then((response) => {
       t.ok(response.sys, 'sys')
       t.ok(response.items, 'fields')
+      t.equal(response.items.length, 4)
     })
   })
 
+  t.test('Gets roles with a limit parameter', (t) => {
+    t.plan(2)
+    return space
+      .getRoles({
+        limit: 2,
+      })
+      .then((response) => {
+        t.ok(response.items, 'items')
+        t.equal(response.items.length, 2)
+      })
+  })
+
+  t.test('Gets roles with skip parameter', (t) => {
+    t.plan(2)
+    return space
+      .getRoles({
+        skip: 2,
+      })
+      .then((response) => {
+        t.ok(response.items, 'items')
+        t.equal(response.skip, 2)
+      })
+  })
+
   t.test('Create role with id', (t) => {
     const id = generateRandomId('role')
     return space.createRoleWithId(id, roleDefinition).then((role) => {