Skip to content

Commit cc1ce62

Browse files
STAC-22844: Describing the resource that you just deleted fails by definition for ResourcePermissions
1 parent 6278149 commit cc1ce62

2 files changed

Lines changed: 1 addition & 3 deletions

File tree

cmd/rbac/rbac_revoke.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ func RunRevokePermissionsCommand(args *RevokePermissionsArgs) di.CmdWithApiFn {
4949
return common.NewResponseError(revokeErr, revokeResp)
5050
}
5151

52-
description, descrResp, descrErr := describePermissions(cli, api, args.Subject, args.Permission, args.Resource).Execute()
52+
description, descrResp, descrErr := describePermissions(cli, api, args.Subject, args.Permission, "").Execute()
5353

5454
if descrErr != nil {
5555
return common.NewResponseError(descrErr, descrResp)

cmd/rbac/rbac_revoke_test.go

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,6 @@ func TestRevokePermissions(t *testing.T) {
2626
assert.Len(t, describeCalls, 1)
2727
assert.Equal(t, SomeSubject, describeCalls[0].Psubject)
2828
assert.Equal(t, SomePermission, *describeCalls[0].Ppermission)
29-
assert.Equal(t, "system", *describeCalls[0].Presource)
3029

3130
di.ExecuteCommandWithContextUnsafe(&cli.Deps, cmd, "--subject", SomeSubject, "--permission", SomePermission, "--resource", SomeResource)
3231

@@ -40,7 +39,6 @@ func TestRevokePermissions(t *testing.T) {
4039
assert.Len(t, describeCalls, 2)
4140
assert.Equal(t, SomeSubject, describeCalls[1].Psubject)
4241
assert.Equal(t, SomePermission, *describeCalls[1].Ppermission)
43-
assert.Equal(t, SomeResource, *describeCalls[1].Presource)
4442

4543
expectedResult := []printer.TableData{
4644
ExpectedTable,

0 commit comments

Comments
 (0)