mirror of
https://github.com/apache/superset.git
synced 2024-09-19 03:59:49 -04:00
a04f1d4c87
* chore(cache): default to SimpleCache in debug mode * lint * clean up type * use util * fix integration test cache configs * remove util from cache manager * remove trailing comma * fix more tests * fix truthiness check * fix tests and improve deprecation notice * fix default cache threshold * move debug check to cache_manager * remove separate getter * update docs * remove default cache config
352 lines
11 KiB
Python
352 lines
11 KiB
Python
# Licensed to the Apache Software Foundation (ASF) under one
|
|
# or more contributor license agreements. See the NOTICE file
|
|
# distributed with this work for additional information
|
|
# regarding copyright ownership. The ASF licenses this file
|
|
# to you under the Apache License, Version 2.0 (the
|
|
# "License"); you may not use this file except in compliance
|
|
# with the License. You may obtain a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing,
|
|
# software distributed under the License is distributed on an
|
|
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
|
|
# KIND, either express or implied. See the License for the
|
|
# specific language governing permissions and limitations
|
|
# under the License.
|
|
import json
|
|
from unittest.mock import patch
|
|
|
|
import pytest
|
|
from flask_appbuilder.security.sqla.models import User
|
|
from sqlalchemy.orm import Session
|
|
|
|
from superset.connectors.sqla.models import SqlaTable
|
|
from superset.datasets.commands.exceptions import DatasetAccessDeniedError
|
|
from superset.explore.form_data.commands.state import TemporaryExploreState
|
|
from superset.extensions import cache_manager
|
|
from superset.models.slice import Slice
|
|
from tests.integration_tests.base_tests import login
|
|
from tests.integration_tests.fixtures.world_bank_dashboard import (
|
|
load_world_bank_dashboard_with_slices,
|
|
load_world_bank_data,
|
|
)
|
|
from tests.integration_tests.test_app import app
|
|
|
|
key = "test-key"
|
|
form_data = "test"
|
|
|
|
|
|
@pytest.fixture
|
|
def client():
|
|
with app.test_client() as client:
|
|
with app.app_context():
|
|
yield client
|
|
|
|
|
|
@pytest.fixture
|
|
def chart_id(load_world_bank_dashboard_with_slices) -> int:
|
|
with app.app_context() as ctx:
|
|
session: Session = ctx.app.appbuilder.get_session
|
|
chart = session.query(Slice).filter_by(slice_name="World's Population").one()
|
|
return chart.id
|
|
|
|
|
|
@pytest.fixture
|
|
def admin_id() -> int:
|
|
with app.app_context() as ctx:
|
|
session: Session = ctx.app.appbuilder.get_session
|
|
admin = session.query(User).filter_by(username="admin").one()
|
|
return admin.id
|
|
|
|
|
|
@pytest.fixture
|
|
def dataset_id() -> int:
|
|
with app.app_context() as ctx:
|
|
session: Session = ctx.app.appbuilder.get_session
|
|
dataset = (
|
|
session.query(SqlaTable)
|
|
.filter_by(table_name="wb_health_population")
|
|
.first()
|
|
)
|
|
return dataset.id
|
|
|
|
|
|
@pytest.fixture(autouse=True)
|
|
def cache(chart_id, admin_id, dataset_id):
|
|
entry: TemporaryExploreState = {
|
|
"owner": admin_id,
|
|
"dataset_id": dataset_id,
|
|
"chart_id": chart_id,
|
|
"form_data": form_data,
|
|
}
|
|
cache_manager.explore_form_data_cache.set(key, entry)
|
|
|
|
|
|
def test_post(client, chart_id: int, dataset_id: int):
|
|
login(client, "admin")
|
|
payload = {
|
|
"dataset_id": dataset_id,
|
|
"chart_id": chart_id,
|
|
"form_data": form_data,
|
|
}
|
|
resp = client.post("api/v1/explore/form_data", json=payload)
|
|
assert resp.status_code == 201
|
|
|
|
|
|
def test_post_bad_request(client, chart_id: int, dataset_id: int):
|
|
login(client, "admin")
|
|
payload = {
|
|
"dataset_id": dataset_id,
|
|
"chart_id": chart_id,
|
|
"form_data": 1234,
|
|
}
|
|
resp = client.post("api/v1/explore/form_data", json=payload)
|
|
assert resp.status_code == 400
|
|
|
|
|
|
def test_post_access_denied(client, chart_id: int, dataset_id: int):
|
|
login(client, "gamma")
|
|
payload = {
|
|
"dataset_id": dataset_id,
|
|
"chart_id": chart_id,
|
|
"form_data": form_data,
|
|
}
|
|
resp = client.post("api/v1/explore/form_data", json=payload)
|
|
assert resp.status_code == 404
|
|
|
|
|
|
def test_post_same_key_for_same_context(client, chart_id: int, dataset_id: int):
|
|
login(client, "admin")
|
|
payload = {
|
|
"dataset_id": dataset_id,
|
|
"chart_id": chart_id,
|
|
"form_data": "new form_data",
|
|
}
|
|
resp = client.post("api/v1/explore/form_data?tab_id=1", json=payload)
|
|
data = json.loads(resp.data.decode("utf-8"))
|
|
first_key = data.get("key")
|
|
resp = client.post("api/v1/explore/form_data?tab_id=1", json=payload)
|
|
data = json.loads(resp.data.decode("utf-8"))
|
|
second_key = data.get("key")
|
|
assert first_key == second_key
|
|
|
|
|
|
def test_post_different_key_for_different_context(
|
|
client, chart_id: int, dataset_id: int
|
|
):
|
|
login(client, "admin")
|
|
payload = {
|
|
"dataset_id": dataset_id,
|
|
"chart_id": chart_id,
|
|
"form_data": "new form_data",
|
|
}
|
|
resp = client.post("api/v1/explore/form_data?tab_id=1", json=payload)
|
|
data = json.loads(resp.data.decode("utf-8"))
|
|
first_key = data.get("key")
|
|
payload = {
|
|
"dataset_id": dataset_id,
|
|
"form_data": "new form_data",
|
|
}
|
|
resp = client.post("api/v1/explore/form_data?tab_id=1", json=payload)
|
|
data = json.loads(resp.data.decode("utf-8"))
|
|
second_key = data.get("key")
|
|
assert first_key != second_key
|
|
|
|
|
|
def test_post_same_key_for_same_tab_id(client, chart_id: int, dataset_id: int):
|
|
login(client, "admin")
|
|
payload = {
|
|
"dataset_id": dataset_id,
|
|
"chart_id": chart_id,
|
|
"form_data": "new form_data",
|
|
}
|
|
resp = client.post("api/v1/explore/form_data?tab_id=1", json=payload)
|
|
data = json.loads(resp.data.decode("utf-8"))
|
|
first_key = data.get("key")
|
|
resp = client.post("api/v1/explore/form_data?tab_id=1", json=payload)
|
|
data = json.loads(resp.data.decode("utf-8"))
|
|
second_key = data.get("key")
|
|
assert first_key == second_key
|
|
|
|
|
|
def test_post_different_key_for_different_tab_id(
|
|
client, chart_id: int, dataset_id: int
|
|
):
|
|
login(client, "admin")
|
|
payload = {
|
|
"dataset_id": dataset_id,
|
|
"chart_id": chart_id,
|
|
"form_data": "new form_data",
|
|
}
|
|
resp = client.post("api/v1/explore/form_data?tab_id=1", json=payload)
|
|
data = json.loads(resp.data.decode("utf-8"))
|
|
first_key = data.get("key")
|
|
resp = client.post("api/v1/explore/form_data?tab_id=2", json=payload)
|
|
data = json.loads(resp.data.decode("utf-8"))
|
|
second_key = data.get("key")
|
|
assert first_key != second_key
|
|
|
|
|
|
def test_post_different_key_for_no_tab_id(client, chart_id: int, dataset_id: int):
|
|
login(client, "admin")
|
|
payload = {
|
|
"dataset_id": dataset_id,
|
|
"chart_id": chart_id,
|
|
"form_data": "new form_data",
|
|
}
|
|
resp = client.post("api/v1/explore/form_data", json=payload)
|
|
data = json.loads(resp.data.decode("utf-8"))
|
|
first_key = data.get("key")
|
|
resp = client.post("api/v1/explore/form_data", json=payload)
|
|
data = json.loads(resp.data.decode("utf-8"))
|
|
second_key = data.get("key")
|
|
assert first_key != second_key
|
|
|
|
|
|
def test_put(client, chart_id: int, dataset_id: int):
|
|
login(client, "admin")
|
|
payload = {
|
|
"dataset_id": dataset_id,
|
|
"chart_id": chart_id,
|
|
"form_data": "new form_data",
|
|
}
|
|
resp = client.put(f"api/v1/explore/form_data/{key}", json=payload)
|
|
assert resp.status_code == 200
|
|
|
|
|
|
def test_put_same_key_for_same_tab_id(client, chart_id: int, dataset_id: int):
|
|
login(client, "admin")
|
|
payload = {
|
|
"dataset_id": dataset_id,
|
|
"chart_id": chart_id,
|
|
"form_data": "new form_data",
|
|
}
|
|
resp = client.put(f"api/v1/explore/form_data/{key}?tab_id=1", json=payload)
|
|
data = json.loads(resp.data.decode("utf-8"))
|
|
first_key = data.get("key")
|
|
resp = client.put(f"api/v1/explore/form_data/{key}?tab_id=1", json=payload)
|
|
data = json.loads(resp.data.decode("utf-8"))
|
|
second_key = data.get("key")
|
|
assert first_key == second_key
|
|
|
|
|
|
def test_put_different_key_for_different_tab_id(client, chart_id: int, dataset_id: int):
|
|
login(client, "admin")
|
|
payload = {
|
|
"dataset_id": dataset_id,
|
|
"chart_id": chart_id,
|
|
"form_data": "new form_data",
|
|
}
|
|
resp = client.put(f"api/v1/explore/form_data/{key}?tab_id=1", json=payload)
|
|
data = json.loads(resp.data.decode("utf-8"))
|
|
first_key = data.get("key")
|
|
resp = client.put(f"api/v1/explore/form_data/{key}?tab_id=2", json=payload)
|
|
data = json.loads(resp.data.decode("utf-8"))
|
|
second_key = data.get("key")
|
|
assert first_key != second_key
|
|
|
|
|
|
def test_put_different_key_for_no_tab_id(client, chart_id: int, dataset_id: int):
|
|
login(client, "admin")
|
|
payload = {
|
|
"dataset_id": dataset_id,
|
|
"chart_id": chart_id,
|
|
"form_data": "new form_data",
|
|
}
|
|
resp = client.put(f"api/v1/explore/form_data/{key}", json=payload)
|
|
data = json.loads(resp.data.decode("utf-8"))
|
|
first_key = data.get("key")
|
|
resp = client.put(f"api/v1/explore/form_data/{key}", json=payload)
|
|
data = json.loads(resp.data.decode("utf-8"))
|
|
second_key = data.get("key")
|
|
assert first_key != second_key
|
|
|
|
|
|
def test_put_bad_request(client, chart_id: int, dataset_id: int):
|
|
login(client, "admin")
|
|
payload = {
|
|
"dataset_id": dataset_id,
|
|
"chart_id": chart_id,
|
|
"form_data": 1234,
|
|
}
|
|
resp = client.put(f"api/v1/explore/form_data/{key}", json=payload)
|
|
assert resp.status_code == 400
|
|
|
|
|
|
def test_put_access_denied(client, chart_id: int, dataset_id: int):
|
|
login(client, "gamma")
|
|
payload = {
|
|
"dataset_id": dataset_id,
|
|
"chart_id": chart_id,
|
|
"form_data": "new form_data",
|
|
}
|
|
resp = client.put(f"api/v1/explore/form_data/{key}", json=payload)
|
|
assert resp.status_code == 404
|
|
|
|
|
|
def test_put_not_owner(client, chart_id: int, dataset_id: int):
|
|
login(client, "gamma")
|
|
payload = {
|
|
"dataset_id": dataset_id,
|
|
"chart_id": chart_id,
|
|
"form_data": "new form_data",
|
|
}
|
|
resp = client.put(f"api/v1/explore/form_data/{key}", json=payload)
|
|
assert resp.status_code == 404
|
|
|
|
|
|
def test_get_key_not_found(client):
|
|
login(client, "admin")
|
|
resp = client.get(f"api/v1/explore/form_data/unknown-key")
|
|
assert resp.status_code == 404
|
|
|
|
|
|
def test_get(client):
|
|
login(client, "admin")
|
|
resp = client.get(f"api/v1/explore/form_data/{key}")
|
|
assert resp.status_code == 200
|
|
data = json.loads(resp.data.decode("utf-8"))
|
|
assert form_data == data.get("form_data")
|
|
|
|
|
|
def test_get_access_denied(client):
|
|
login(client, "gamma")
|
|
resp = client.get(f"api/v1/explore/form_data/{key}")
|
|
assert resp.status_code == 404
|
|
|
|
|
|
@patch("superset.security.SupersetSecurityManager.can_access_datasource")
|
|
def test_get_dataset_access_denied(mock_can_access_datasource, client):
|
|
mock_can_access_datasource.side_effect = DatasetAccessDeniedError()
|
|
login(client, "admin")
|
|
resp = client.get(f"api/v1/explore/form_data/{key}")
|
|
assert resp.status_code == 403
|
|
|
|
|
|
def test_delete(client):
|
|
login(client, "admin")
|
|
resp = client.delete(f"api/v1/explore/form_data/{key}")
|
|
assert resp.status_code == 200
|
|
|
|
|
|
def test_delete_access_denied(client):
|
|
login(client, "gamma")
|
|
resp = client.delete(f"api/v1/explore/form_data/{key}")
|
|
assert resp.status_code == 404
|
|
|
|
|
|
def test_delete_not_owner(client, chart_id: int, dataset_id: int, admin_id: int):
|
|
another_key = "another_key"
|
|
another_owner = admin_id + 1
|
|
entry: TemporaryExploreState = {
|
|
"owner": another_owner,
|
|
"dataset_id": dataset_id,
|
|
"chart_id": chart_id,
|
|
"form_data": form_data,
|
|
}
|
|
cache_manager.explore_form_data_cache.set(another_key, entry)
|
|
login(client, "admin")
|
|
resp = client.delete(f"api/v1/explore/form_data/{another_key}")
|
|
assert resp.status_code == 403
|