Skip to content

Commit a66e258

Browse files
committed
add test client for bulk update
1 parent 34c7a88 commit a66e258

5 files changed

Lines changed: 123 additions & 8 deletions

File tree

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
3ff3a71c8e5ef1fe262a88e5326f6a8b
1+
984d43394f2921d8aaa46689cc0057f7

internal/graphapi/orgmembers_test.go

Lines changed: 16 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -542,10 +542,11 @@ func TestMutationBulkUpdateOrgMemberRole(t *testing.T) {
542542
Role: &adminRole,
543543
}
544544

545-
for _, id := range ids {
546-
_, err = suite.client.api.UpdateUserRoleInOrg(org.admin.UserCtx, id, input)
547-
assert.NilError(t, err)
548-
}
545+
resp, err := suite.client.api.UpdateBulkOrgMemberRoles(org.admin.UserCtx, ids, input)
546+
assert.NilError(t, err)
547+
assert.Assert(t, resp != nil)
548+
assert.Check(t, is.Len(resp.UpdateBulkOrgMembership.UpdatedIDs, len(ids)))
549+
assert.Check(t, is.Len(resp.UpdateBulkOrgMembership.OrgMemberships, len(ids)))
549550

550551
updatedMembers, err := suite.client.db.OrgMembership.Query().
551552
Where(orgmembership.IDIn(ids...)).
@@ -567,8 +568,17 @@ func TestMutationBulkUpdateOrgMemberRole(t *testing.T) {
567568
memberRole := enums.RoleMember
568569
input.Role = &memberRole
569570

570-
_, err = suite.client.api.UpdateUserRoleInOrg(org.admin.UserCtx, ownerMember.ID, input)
571-
assert.ErrorContains(t, err, hooks.ErrOrgOwnerCannotBeUpdated.Error())
571+
ownerResp, err := suite.client.api.UpdateBulkOrgMemberRoles(org.admin.UserCtx, []string{ownerMember.ID}, input)
572+
assert.NilError(t, err)
573+
assert.Assert(t, ownerResp != nil)
574+
assert.Check(t, is.Len(ownerResp.UpdateBulkOrgMembership.UpdatedIDs, 0))
575+
assert.Check(t, is.Len(ownerResp.UpdateBulkOrgMembership.OrgMemberships, 0))
576+
577+
ownerMember, err = suite.client.db.OrgMembership.Query().
578+
Where(orgmembership.ID(ownerMember.ID)).
579+
Only(allowCtx)
580+
assert.NilError(t, err)
581+
assert.Check(t, is.Equal(enums.RoleOwner, ownerMember.Role))
572582

573583
cleanupOrganizationDataWithContext(org.owner.UserCtx, t)
574584
}

internal/graphapi/query/orgmembership.graphql

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,17 @@ mutation RemoveUserFromOrg ($deleteOrgMembershipId: ID!) {
5252
deletedID
5353
}
5454
}
55+
mutation UpdateBulkOrgMemberRoles ($ids: [ID!]!, $input: UpdateOrgMembershipInput!) {
56+
updateBulkOrgMembership(ids: $ids, input: $input) {
57+
orgMemberships {
58+
id
59+
role
60+
userID
61+
organizationID
62+
}
63+
updatedIDs
64+
}
65+
}
5566
mutation UpdateUserRoleInOrg ($updateOrgMemberId: ID!, $input: UpdateOrgMembershipInput!) {
5667
updateOrgMembership(id: $updateOrgMemberId, input: $input) {
5768
orgMembership {
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
e34d40977a3c1ec7e29a8820d26f4538b864deb80c80194f4ce9cc169ec4bd83
1+
b2b027b3e95778f44a5132f0fa1bf9ff12cefb44857b96354ce97c20313bd358

internal/graphapi/testclient/graphclient.go

Lines changed: 94 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)