[DEVOPS-4655] feat(users): add user and user group listing - #45
Conversation
|
🤖 You did not bump the version, a release won't be created |
There was a problem hiding this comment.
🟢 Approval recommended
The implementation and tests are coherent, with only a non-blocking documentation capitalization nit remaining.
Pull request overview
Adds user, application-account, and user-group discovery APIs for permission and role assignment workflows.
Changes:
- Adds listing and name-based lookup services.
- Introduces authentication and group type enums.
- Adds unit and integration coverage.
File summaries
| File | Description |
|---|---|
utils.go |
Adds shared list-fetching and matching helpers. |
users.go |
Implements user and application APIs. |
users_unit_test.go |
Tests user APIs. |
users_test.go |
Adds integration coverage. |
user_groups.go |
Implements user-group APIs. |
user_groups_unit_test.go |
Tests user-group APIs. |
dvlstypes.go |
Adds enum values and group types. |
userauthenticationtype_string.go |
Updates generated authentication strings. |
usergrouptype_string.go |
Adds generated group-type strings. |
authentication.go |
Registers the new services. |
mock_test.go |
Adds shared test identifiers. |
README.md |
Documents principal discovery. |
Review details
Files not reviewed (2)
- userauthenticationtype_string.go: Generated file
- usergrouptype_string.go: Generated file
- Files reviewed: 10/12 changed files
- Comments generated: 1
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
|
🤖 You did not bump the version, a release won't be created |
|
Copilot resolve the merge conflicts in this pull request |
…users # Conflicts: # mock_test.go # utils.go Co-authored-by: dion-gionet <29441318+dion-gionet@users.noreply.github.com>
|
🤖 You did not bump the version, a release won't be created |
Resolved the merge conflicts with |
No description provided.