diff --git a/superset/security/manager.py b/superset/security/manager.py index ea739376eef..8490c049e3b 100644 --- a/superset/security/manager.py +++ b/superset/security/manager.py @@ -358,7 +358,8 @@ class SupersetSecurityManager( # pylint: disable=too-many-public-methods ("can_export_csv", "Query"), ("can_get_results", "SQLLab"), ("can_execute_sql_query", "SQLLab"), - ("can_estimate_query_cost", "SQL Lab"), + ("can_estimate_query_cost", "SQLLab"), + ("can_format_sql", "SQLLab"), ("can_export_csv", "SQLLab"), ("can_read", "SQLLab"), ("can_sqllab_history", "Superset"), diff --git a/tests/integration_tests/security_tests.py b/tests/integration_tests/security_tests.py index 7aba9ffd341..96b2cb3e201 100644 --- a/tests/integration_tests/security_tests.py +++ b/tests/integration_tests/security_tests.py @@ -1474,9 +1474,11 @@ class TestRolePermission(SupersetTestCase): ("can_csv", "Superset"), ("can_delete_query", "TabStateView"), ("can_delete", "TabStateView"), + ("can_estimate_query_cost", "SQLLab"), ("can_execute_sql_query", "SQLLab"), ("can_export", "SavedQuery"), ("can_export_csv", "SQLLab"), + ("can_format_sql", "SQLLab"), ("can_get", "TabStateView"), ("can_get_results", "SQLLab"), ("can_migrate_query", "TabStateView"),