wip

Snowflake-Labs · Mar 27, 2024 · 268b066 · 268b066
1 parent 632d3db
commit 268b066
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/pkg/resources/grant_ownership_acceptance_test.go b/pkg/resources/grant_ownership_acceptance_test.go
@@ -721,7 +721,7 @@ func TestAcc_GrantOwnership_OnPipe(t *testing.T) {
 								Name:       sdk.NewSchemaObjectIdentifierFromFullyQualifiedName(pipeFullyQualifiedName),
 							},
 						},
-						// TODO: Fix this identifier
+						// TODO(): Fix this identifier
 					}, sdk.ObjectTypePipe, accountRoleName, fmt.Sprintf("%s\".\"%s\".%s", acc.TestDatabaseName, acc.TestSchemaName, pipeName)),
 				),
 			},

diff --git a/pkg/sdk/grants_impl.go b/pkg/sdk/grants_impl.go
@@ -292,7 +292,7 @@ func (v *grants) grantOwnershipOnPipe(ctx context.Context, pipeId SchemaObjectId
 	}
 
 	canOperateOnPipe := slices.ContainsFunc(currentGrants, func(grant Grant) bool {
-		return grant.Privilege == "OWNERSHIP" || grant.Privilege == SchemaObjectPrivilegeOperate.String() && grant.GranteeName == currentRoleName
+		return grant.GranteeName == currentRoleName && (grant.Privilege == "OWNERSHIP" || grant.Privilege == SchemaObjectPrivilegeOperate.String())
 	})
 
 	var revokeOperate func() error