QT/superset2
1
0
Fork 0
mirror of https://github.com/apache/superset.git synced 2026-04-19 16:14:52 +00:00
Code Issues Packages Projects Releases Wiki Activity

feat(dashboards): security permissions simplification (#12012)

Browse Source 
* feat(dashboards): security perm simplification

* fix tests

* fix tests

* remove unnecessary mapping

* fix JS tests

* fix cypress test and explore endpoint

* update alembic down revision

* Merge branch 'master' into feat/security-converge-dashboards

# Conflicts:
#	superset/constants.py
#	tests/security_tests.py
This commit is contained in:
Daniel Vaz Gaspar
2020-12-17 09:53:13 +00:00
committed by GitHub
parent 700dee6db6
commit dd5cdb1b5f
11 changed files with 146 additions and 27 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
tests/dashboards/api_tests.py
View File

@@ -198,6 +198,20 @@ class TestDashboardApi(SupersetTestCase, ApiOwnersTestCaseMixin):
rv = self.get_assert_metric(uri, "info")
self.assertEqual(rv.status_code, 200)
def test_info_security_database(self):
"""
Dashboard API: Test info security
"""
self.login(username="admin")
params = {"keys": ["permissions"]}
uri = f"api/v1/dashboard/_info?q={prison.dumps(params)}"
rv = self.get_assert_metric(uri, "info")
data = json.loads(rv.data.decode("utf-8"))
assert rv.status_code == 200
assert "can_read" in data["permissions"]
assert "can_write" in data["permissions"]
assert len(data["permissions"]) == 2
def test_get_dashboard_not_found(self):
"""
Dashboard API: Test get dashboard not found