Skip to content

Commit cccc32b

Browse files
test: Defer assignments to improve performance slightly
1 parent ae2eb86 commit cccc32b

1 file changed

Lines changed: 6 additions & 6 deletions

File tree

tests/templates/kuttl/opa-authorization/trino_rules/trino/actual_permissions.rego

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -47,19 +47,19 @@ default match_user_group(_) := false
4747

4848
match_user_group(rule) if {
4949
user_pattern := object.get(rule, "user", ".*")
50-
group_pattern := object.get(rule, "group", ".*")
51-
5250
match_entire(user_pattern, identity.user)
51+
52+
group_pattern := object.get(rule, "group", ".*")
5353
match_any_group(group_pattern)
5454
}
5555

5656
default match_original_user_group(_) := false
5757

5858
match_original_user_group(rule) if {
5959
user_pattern := object.get(rule, "original_user", ".*")
60-
group_pattern := object.get(rule, "original_group", ".*")
61-
6260
match_entire(user_pattern, identity.user)
61+
62+
group_pattern := object.get(rule, "original_group", ".*")
6363
match_any_group(group_pattern)
6464
}
6565

@@ -394,9 +394,9 @@ schema_visibility(catalog_name, schema_name) if {
394394
match_user_group(rule)
395395

396396
catalog_pattern := object.get(rule, "catalog", ".*")
397-
schema_pattern := object.get(rule, "schema", ".*")
398-
399397
match_entire(catalog_pattern, catalog_name)
398+
399+
schema_pattern := object.get(rule, "schema", ".*")
400400
match_entire(schema_pattern, schema_name)
401401

402402
count(rule.privileges) != 0

0 commit comments

Comments
 (0)