Conversation
## 📝 Description
**What does this PR do and why is this change necessary?**
Implements:
- **GET** /entities
- **GET** /iam/role-permissions
- **GET** /iam/users/{username}/role-permissions
- **PUT** /iam/users/{username}/role-permissions
## ✔️ How to Test
**How do I run the relevant unit/integration tests?**
```bash
make TEST_ARGS='-run TestIAM' test-unit
```
```bash
make TEST_ARGS='-run TestEntities' test-unit
```
vshanthe
requested review from
Copilot,
ezilber-akamai and
zliang-akamai
and removed request for
a team
There was a problem hiding this comment.
Pull Request Overview
This PR adds integration tests for IAM role-permissions and entity management endpoints. The tests verify the functionality of retrieving account and user role permissions, updating user permissions, and listing entities.
Key changes:
- Added integration tests for IAM role permissions (get account/user permissions, update user permissions)
- Added integration test for listing IAM entities
- Added test fixtures (YAML files) containing HTTP request/response recordings for the tests
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| test/integration/iam_roles_test.go | Integration tests for getting and updating account/user role permissions |
| test/integration/iam_entities_test.go | Integration test for listing entities across the account |
| test/integration/fixtures/TestIAM_GetAccountRolePermissions.yaml | HTTP recording fixture for account role permissions test |
| test/integration/fixtures/TestIAM_GetUserRolePermissions.yaml | HTTP recording fixture for user role permissions test |
| test/integration/fixtures/TestIAM_UpdateUserRolePermissions.yaml | HTTP recording fixture for updating user permissions test |
| test/integration/fixtures/TestIAM_ListEntities.yaml | HTTP recording fixture for entities listing test |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
ezilber-akamai
left a comment
ezilber-akamai
left a comment
There was a problem hiding this comment.
Tests passed locally! CI failures should be fixed by merging main back into the project branch later down the line.
zliang-akamai
force-pushed
the
iam_tests
branch
from
10160dc to
5d98ead
Compare
|
Merging main into iam here (pending reviews): #856 |
Merge main into IAM project branch
jriddle-linode
dismissed
ezilber-akamai’s stale review
The base branch was changed.
|
since some of the function name changed closing this PR and raising new PR with new endpoints added |
📝 Description
Integrations for IAM role-permissions and Entities
✔️ How to Test
Test Against Devcloud using API TOKEN
and URL - api.devcloud.linode.com
test-int TEST_ARGS="-run TestIAM"
📷 Preview
If applicable, include a screenshot or code snippet of this change. Otherwise, please remove this section.