use customer query param instead of domain to fetch all users (#2594)

* use customer query param instead of domain to fetch all users

* members list doesn't take customer param

* Fix query param in tests

---------

Co-authored-by: Aswin V <vaswin91@gmail.com>
This commit is contained in:
Deepak Prabhakara 2024-04-17 12:06:42 +01:00 committed by GitHub
parent 3b024693b7
commit 1d6a7e8e31
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 4 additions and 6 deletions

View File

@ -63,7 +63,7 @@ export class GoogleProvider implements IDirectoryProvider {
async getUsers(directory: Directory, options: PaginationParams | null) {
const query = {
maxResults: 200,
domain: directory.google_domain,
customer: 'my_customer',
};
if (options?.pageToken) {
@ -106,7 +106,7 @@ export class GoogleProvider implements IDirectoryProvider {
const query = {
maxResults: 200,
domain: directory.google_domain,
customer: 'my_customer',
};
if (options?.pageToken) {
@ -146,7 +146,6 @@ export class GoogleProvider implements IDirectoryProvider {
const query = {
maxResults: 200,
groupKey: group.id,
domain: directory.google_domain,
};
let nextPageToken: string | undefined | null = null;

View File

@ -122,7 +122,7 @@ const mockUsersAPI = async (users: any[]) => {
for (let i = 0; i < users.length / pageSize; i++) {
const query: any = {
maxResults: 200,
domain: 'boxyhq.com',
customer: 'my_customer',
};
if (i !== 0) {
query.pageToken = `${i}`;
@ -141,7 +141,7 @@ const mockGroupsAPI = (groups: any[]) => {
.get('/admin/directory/v1/groups')
.query({
maxResults: 200,
domain: 'boxyhq.com',
customer: 'my_customer',
})
// Gets invoked 2 times - 1 for syncGroups, 2nd time inside syncGroupMembers
.times(2)
@ -154,7 +154,6 @@ const mockGroupMembersAPI = (groupKey: string, members: any[]) => {
.get(`/admin/directory/v1/groups/${groupKey}/members`)
.query({
maxResults: 200,
domain: 'boxyhq.com',
})
.reply(200, { members });
};