Skip to content

Commit

Permalink
Fix wrong test after fixing audit log permission
Browse files Browse the repository at this point in the history
The test for security/permissions had not been run after modifying
default permissions in apache#37501 (to be investigated why). This PR
makes main green again.
  • Loading branch information
potiuk committed Feb 19, 2024
1 parent 1851a71 commit f4b9cc7
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions tests/www/test_security.py
Expand Up @@ -418,7 +418,6 @@ def test_get_user_roles(app_builder, security_manager):

def test_get_user_roles_for_anonymous_user(app, security_manager):
viewer_role_perms = {
(permissions.ACTION_CAN_READ, permissions.RESOURCE_AUDIT_LOG),
(permissions.ACTION_CAN_READ, permissions.RESOURCE_DAG),
(permissions.ACTION_CAN_READ, permissions.RESOURCE_DAG_DEPENDENCIES),
(permissions.ACTION_CAN_READ, permissions.RESOURCE_DAG_CODE),
Expand Down Expand Up @@ -446,7 +445,6 @@ def test_get_user_roles_for_anonymous_user(app, security_manager):
(permissions.ACTION_CAN_ACCESS_MENU, permissions.RESOURCE_DATASET),
(permissions.ACTION_CAN_ACCESS_MENU, permissions.RESOURCE_CLUSTER_ACTIVITY),
(permissions.ACTION_CAN_ACCESS_MENU, permissions.RESOURCE_JOB),
(permissions.ACTION_CAN_ACCESS_MENU, permissions.RESOURCE_AUDIT_LOG),
(permissions.ACTION_CAN_ACCESS_MENU, permissions.RESOURCE_PLUGIN),
(permissions.ACTION_CAN_ACCESS_MENU, permissions.RESOURCE_SLA_MISS),
(permissions.ACTION_CAN_ACCESS_MENU, permissions.RESOURCE_TASK_INSTANCE),
Expand Down

0 comments on commit f4b9cc7

Please sign in to comment.