diff --git a/packages/core/src/queries/organizations.ts b/packages/core/src/queries/organizations.ts index 81272d36b..41e2922f1 100644 --- a/packages/core/src/queries/organizations.ts +++ b/packages/core/src/queries/organizations.ts @@ -68,7 +68,7 @@ class UserRelationQueries extends TwoRelationsQueries(sql` select ${users.table}.*, - ${this.#aggregateRoles('organization_roles')} + ${this.#aggregateRoles()} from ${this.table} left join ${users.table} on ${fields.userId} = ${users.fields.id} @@ -87,9 +87,9 @@ class UserRelationQueries extends TwoRelationsQueries { constructor() { super('organizations'); @@ -26,8 +20,10 @@ class OrganizationApi extends ApiFactory { - await authedAdminApi.post(`${this.path}/${id}/users/${userId}/roles`, { json: { roleIds } }); + async addUserRoles(id: string, userId: string, organizationRoleIds: string[]): Promise { + await authedAdminApi.post(`${this.path}/${id}/users/${userId}/roles`, { + json: { organizationRoleIds }, + }); } async getUserRoles(id: string, userId: string): Promise { diff --git a/packages/integration-tests/src/tests/api/organization.test.ts b/packages/integration-tests/src/tests/api/organization.test.ts index adcfedd1e..47927fad6 100644 --- a/packages/integration-tests/src/tests/api/organization.test.ts +++ b/packages/integration-tests/src/tests/api/organization.test.ts @@ -205,10 +205,10 @@ describe('organization APIs', () => { const organization1WithRoles = organizations.find((org) => org.id === organization1.id); assert(organization1WithRoles); expect(organization1WithRoles.id).toBe(organization1.id); - expect(organization1WithRoles.roles).toContainEqual( + expect(organization1WithRoles.organizationRoles).toContainEqual( expect.objectContaining({ id: role1.id }) ); - expect(organization1WithRoles.roles).not.toContainEqual( + expect(organization1WithRoles.organizationRoles).not.toContainEqual( expect.objectContaining({ id: role2.id }) ); @@ -216,10 +216,10 @@ describe('organization APIs', () => { const organization2WithRoles = organizations.find((org) => org.id === organization2.id); assert(organization2WithRoles); expect(organization2WithRoles.id).toBe(organization2.id); - expect(organization2WithRoles.roles).toContainEqual( + expect(organization2WithRoles.organizationRoles).toContainEqual( expect.objectContaining({ id: role1.id }) ); - expect(organization2WithRoles.roles).toContainEqual( + expect(organization2WithRoles.organizationRoles).toContainEqual( expect.objectContaining({ id: role2.id }) );