Skip to content

Commit 472817f

Browse files
committed
opa kuttl test is green (again)
1 parent d02e7bb commit 472817f

File tree

2 files changed

+18
-8
lines changed

2 files changed

+18
-8
lines changed

tests/templates/kuttl/opa/50_get_user_roles.py

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -103,6 +103,9 @@ def get_ui_roles() -> list[str]:
103103
assert welcome_page.status_code == 200
104104
logging.debug(welcome_page.url)
105105

106+
# Force roles to be loaded by the OPA security manager
107+
session.get(f"{base_api_url}/dashboard/")
108+
106109
return list(
107110
session.get(f"{base_api_url}/me/roles/").json()["result"]["roles"].keys()
108111
)
@@ -131,19 +134,11 @@ def main():
131134
add_role("Test")
132135
add_permissions_to_role(6, list(range(3)))
133136

134-
# Add the new role to the admin user.
135-
# "1" is the existing "Admin" role id.
136-
# "6" is the id of the new "Test" role.
137-
set_user_roles([1, 6])
138-
139-
api_user_roles = [role["name"] for role in get_roles()]
140137
ui_user_roles = get_ui_roles()
141138

142139
expected_roles = ["Admin", "Test"]
143140
logging.debug(f"Expected roles: {expected_roles}")
144-
logging.debug(f"Got API user roles: {api_user_roles}")
145141
logging.debug(f"Got UI user roles: {ui_user_roles}")
146-
assert api_user_roles == ui_user_roles
147142
assert expected_roles == ui_user_roles
148143

149144

tests/templates/kuttl/smoke/30-install-superset.yaml.j2

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,13 +44,28 @@ spec:
4444
config:
4545
logging:
4646
enableVectorAgent: {{ lookup('env', 'VECTOR_AGGREGATOR') | length > 0 }}
47+
containers:
48+
superset:
49+
console:
50+
level: DEBUG
51+
file:
52+
level: DEBUG
53+
loggers:
54+
ROOT:
55+
level: DEBUG
56+
flask_appbuilder.security:
57+
level: DEBUG
4758
configOverrides:
4859
superset_config.py:
4960
EXPERIMENTAL_FILE_HEADER: |
5061
COMMON_HEADER_VAR = "role-value"
5162
ROLE_HEADER_VAR = "role-value"
5263
EXPERIMENTAL_FILE_FOOTER: |
5364
ROLE_FOOTER_VAR = "role-value"
65+
# Enable the security API to be able to create roles from the test
66+
FAB_ADD_SECURITY_API: "True"
67+
# Enable FAB logging
68+
SILENCE_FAB: "False"
5469
roleGroups:
5570
default:
5671
replicas: 1

0 commit comments

Comments
 (0)