Skip to content

Conversation

@lahirumaramba
Copy link
Member

@lahirumaramba lahirumaramba commented Nov 21, 2025

This change implements the accounts:query functionality, providing a new QueryUsers method that allows searching for users with filters and sorting options.

RELEASE_NOTE: Added QueryUsers() API to support querying user accounts with filters, sorting, and pagination.

google-labs-jules bot and others added 5 commits November 21, 2025 20:21
Adds the `QueryUsers` function to the Auth client, allowing users to query for user accounts with various filters.

- Defines `QueryUsersRequest` and `QueryUsersResponse` structs.
- Implements the `QueryUsers` function to call the `accounts:query` endpoint.
- Includes support for tenant-specific queries.
- Adds comprehensive unit tests for the new functionality.
- Updates the `TestQueryUsersWithTenant` to create a tenant-specific client and verify the request URL.
- Corrects the `QueryUsersRequest` struct to use `string` for `Limit` and `Offset` fields, and adds the `TenantID` field.
- Updates the test cases to reflect the struct changes.
Moves the tenant-specific test for `QueryUsers` to `tenant_mgt_test.go` to follow the existing test structure.

- Renames `TestQueryUsersWithTenant` to `TestTenantQueryUsers`.
- Moves the test function to `auth/tenant_mgt_test.go`.
Renames the `SqlExpression` struct to `SQLExpression` to conform to Go's linting standards for acronyms.

- Updates the struct definition in `auth/user_mgt.go`.
- Updates all usages of the struct in `auth/user_mgt.go` and `auth/user_mgt_test.go`.
@lahirumaramba lahirumaramba added release-note release:stage Stage a release candidate labels Nov 21, 2025
Copy link
Contributor

@jonathanedey jonathanedey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:stage Stage a release candidate release-note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants