pt
/
superset
mirror of https://github.com/apache/superset.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Add all database access permission to 'Alpha' role (#7271) 

* Add all database access permission to 'Alpha' role

* Grant all database access to Alpha
This commit is contained in:
Maxime Beauchemin 2019-04-15 09:22:54 -07:00 committed by GitHub
parent b1ebfbc577
commit 9dfa80d5c8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
superset/security.py
View File

@ -68,7 +68,6 @@ class SupersetSecurityManager(SecurityManager):
}
ADMIN_ONLY_PERMISSIONS = {
'all_database_access',
'can_sql_json', # TODO: move can_sql_json to sql_lab role
'can_override_role_permissions',
'can_sync_druid_source',
@ -84,6 +83,7 @@ class SupersetSecurityManager(SecurityManager):
ALPHA_ONLY_PERMISSIONS = set([
'muldelete',
'all_database_access',
'all_datasource_access',
])

6
tests/security_tests.py
View File

@ -129,9 +129,6 @@ class RolePermissionTests(SupersetTestCase):
self.assertTrue(security_manager.is_admin_only(
security_manager.find_permission_view_menu(
'can_approve', 'Superset')))
self.assertTrue(security_manager.is_admin_only(
security_manager.find_permission_view_menu(
'all_database_access', 'all_database_access')))
def test_is_alpha_only(self):
self.assertFalse(security_manager.is_alpha_only(
@ -148,6 +145,9 @@ class RolePermissionTests(SupersetTestCase):
self.assertTrue(security_manager.is_alpha_only(
security_manager.find_permission_view_menu(
'can_delete', 'DruidMetricInlineView')))
self.assertTrue(security_manager.is_alpha_only(
security_manager.find_permission_view_menu(
'all_database_access', 'all_database_access')))
def test_is_gamma_pvm(self):
self.assertTrue(security_manager.is_gamma_pvm(