Skip to content

Conversation

@danielkee
Copy link
Contributor

Resolves ticket: CISC-3191

Summary of changes:

  • Added support for the PE RBAC API endpoint GET /role/:id
  • Added an accompanying unit test and integration test

Integration test output:

% go run cmd/test/main.go epicyclical-pro.delivery.puppetlabs.net AL40f61YgFIhtIT_n1XBeWTbclTilG2r8NTSFX7ndHxP
Connecting to: epicyclical-pro.delivery.puppetlabs.net

* Testing RBAC API client function CreateRole...

Create role "Testing 1680519311359000000" was successful, location: /rbac-api/v1/roles/12
Create role "Testing 1680519311359000000" failed as expected because role already exists

* Testing RBAC API client function GetRole...

Get role was successful, as the role "Testing 1680519311359000000" was returned in response

* Testing RBAC API client function GetRoles...

Get roles was successful, as role "Testing 1680519311359000000" was present in response

@danielkee danielkee marked this pull request as ready for review April 3, 2023 11:08
@danielkee danielkee requested a review from a team as a code owner April 3, 2023 11:08
@danielkee danielkee requested review from cliveweir and eoinmcq April 3, 2023 11:08
@danielkee danielkee merged commit 07d5308 into master Apr 3, 2023
@danielkee danielkee deleted the cisc-3191 branch April 3, 2023 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants