test: added test cases to improve coverage report

Author: harshitha-cstk

lib/organization/teams/index.js

@@ -87,7 +87,7 @@ export function Teams (http, data) {
      * import * as contentstack from '@contentstack/management'
      * const client = contentstack.client()
      * client.organization('organizationUid').teams('teamUid').users().fetchAll()
-     * .then((response) => console.log(response))
+     *  .then((response) => console.log(response))
      *
      */
     this.users = (userId = null) => {
@@ -99,6 +99,18 @@ export function Teams (http, data) {
       return new TeamUsers(http, data)
     }
 
+    /**
+     * @description The stackRoleMappings call on team will get the stack role Mapping.
+     * @memberof Teams
+     * @func users
+     * @returns {Promise<Teams.Teams>} Response Object.
+     * @example
+     * import * as contentstack from '@contentstack/management'
+     * const client = contentstack.client()
+     * client.organization('organizationUid').teams('teamUid').stackRoleMappings().fetchAll()
+     *  .then((response) => console.log(response))
+     *
+     */
     this.stackRoleMappings = (stackApiKey = null) => {
       data.organizationUid = this.organizationUid
       data.teamUid = this.uid

lib/organization/teams/stackRoleMappings/index.js

@@ -109,10 +109,3 @@ export function StackRoleMappings (http, data) {
     }
   }
 }
-export function stackRoleMappingsCollection (http, data) {
-  const obj = cloneDeep(data.stackRoleMappings) || []
-  const stackRoleMappingCollection = obj.map((stackRoleMappings) => {
-    return stackRoleMappings(http, { stackRoleMappings: stackRoleMappings })
-  })
-  return new StackRoleMappings(http, stackRoleMappingCollection)
-}

test/unit/organization-test.js

@@ -263,6 +263,25 @@ describe('Organization Test', () => {
       })
       .catch(done)
   })
+  it('should get teams', done => {
+    const mock = new MockAdapter(Axios)
+    mock.onGet(`/organizations/UID`).reply(200, {
+      organization: {
+        ...orgMock
+      }
+    })
+    makeOrganization({ organization: {
+      ...systemUidMock
+    } })
+      .fetch()
+      .then((response) => {
+        const teams = response.teams()
+        expect(teams).to.not.equal(undefined)
+        expect(teams.organizationUid).to.be.equal('UID')
+        done()
+      })
+      .catch(done)
+  })
 })
 
 function makeOrganization (params = {}) {

test/unit/team-test.js

@@ -3,7 +3,7 @@ import { expect } from 'chai'
 import { describe, it } from 'mocha'
 import MockAdapter from 'axios-mock-adapter'
 import { Teams } from '../../lib/organization/teams'
-import { systemUidMock, teamsMock, noticeMock } from './mock/objects'
+import { systemUidMock, teamsMock, noticeMock, teamUsersMock, stackRoleMappingMock } from './mock/objects'
 
 describe('Contentstack Team test', () => {
   it('should get all the teams when correct organization uid is passed', done => {
@@ -70,6 +70,28 @@ describe('Contentstack Team test', () => {
       })
       .catch(done)
   })
+  it('should fetch all users', done => {
+    var mock = new MockAdapter(Axios)
+    mock.onGet(`/organizations/organization_uid/teams/UID/users`).reply(200, teamUsersMock)
+    makeTeams({ ...systemUidMock }).users().fetchAll()
+      .then((users) => {
+        users.items.forEach((user) => {
+          expect(user.uidId).to.be.not.equal(null)
+        })
+        done()
+      })
+      .catch(done)
+  })
+  it('should fetch all the roles', done => {
+    var mock = new MockAdapter(Axios)
+    mock.onGet(`/organizations/organization_uid/teams/UID/stack_role_mappings`).reply(200, stackRoleMappingMock)
+    makeTeams({ ...systemUidMock }).stackRoleMappings().fetchAll()
+      .then((response) => {
+        expect(response.stackRoleMappings).to.be.not.equal(undefined)
+        done()
+      })
+      .catch(done)
+  })
 })
 
 function makeTeams (data = {}) {

test/unit/team-users-test.js

@@ -6,7 +6,7 @@ import { TeamUsers } from '../../lib/organization/teams/teamUsers'
 import { teamUsersMock, noticeMock } from './mock/objects'
 
 describe('Contentstack Team Users test', () => {
-  it('should query and find all users', done => {
+  it('should fetch all users', done => {
     var mock = new MockAdapter(Axios)
     mock.onGet(`/organizations/organization_uid/teams/team_uid/users`).reply(200, teamUsersMock)
     makeTeamUsers().fetchAll()