Skip to content

Commit 3a9ab07

Browse files
authored
acc: always include 'inherited' field in permissions (#3736)
## Why This matches real backend behaviour. ## Tests Restores acceptance/bundle/deployment/unbind/permissions which is broken on cloud since #3730
1 parent 359cfd6 commit 3a9ab07

File tree

2 files changed

+4
-0
lines changed

2 files changed

+4
-0
lines changed

acceptance/bundle/deployment/unbind/permissions/output.txt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ Deployment complete!
1111
{
1212
"all_permissions": [
1313
{
14+
"inherited": false,
1415
"permission_level": "CAN_MANAGE"
1516
}
1617
],
@@ -31,6 +32,7 @@ Deployment complete!
3132
{
3233
"all_permissions": [
3334
{
35+
"inherited": false,
3436
"permission_level": "CAN_MANAGE"
3537
}
3638
],

libs/testserver/permissions.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,9 @@ func (s *FakeWorkspace) SetPermissions(req Request) any {
141141
// Convert PermissionLevel to Permission
142142
if acl.PermissionLevel != "" {
143143
response.AllPermissions = append(response.AllPermissions, iam.Permission{
144+
Inherited: false,
144145
PermissionLevel: acl.PermissionLevel,
146+
ForceSendFields: []string{"Inherited"},
145147
})
146148
}
147149

0 commit comments

Comments
 (0)