Skip to content

Commit

Permalink
fix: gamma sql_json test (#23135)
Browse files Browse the repository at this point in the history
  • Loading branch information
dpgaspar committed Feb 21, 2023
1 parent d4362a3 commit 8f8fba2
Show file tree
Hide file tree
Showing 2 changed files with 56 additions and 1 deletion.
49 changes: 49 additions & 0 deletions tests/integration_tests/fixtures/users.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,49 @@
# 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 pytest
from flask_appbuilder.security.sqla.models import Role, User

from superset import db, security_manager
from tests.integration_tests.test_app import app


@pytest.fixture()
def create_gamma_sqllab_no_data():

with app.app_context():
gamma_role = db.session.query(Role).filter(Role.name == "Gamma").one_or_none()
sqllab_role = (
db.session.query(Role).filter(Role.name == "sql_lab").one_or_none()
)

security_manager.add_user(
"gamma_sqllab_no_data",
"gamma_sqllab_no_data",
"gamma_sqllab_no_data",
"gamma_sqllab_no_data@apache.org",
[gamma_role, sqllab_role],
password="general",
)

yield
user = (
db.session.query(User)
.filter(User.username == "gamma_sqllab_no_data")
.one_or_none()
)
db.session.delete(user)
db.session.commit()
8 changes: 7 additions & 1 deletion tests/integration_tests/sqllab_tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@
load_birth_names_dashboard_with_slices,
load_birth_names_data,
)
from tests.integration_tests.fixtures.users import create_gamma_sqllab_no_data

QUERY_1 = "SELECT * FROM birth_names LIMIT 1"
QUERY_2 = "SELECT * FROM NO_TABLE"
Expand Down Expand Up @@ -752,20 +753,25 @@ def test_sql_json_parameter_authorized(self):
)
assert data["status"] == "success"

@pytest.mark.usefixtures("create_gamma_sqllab_no_data")
@pytest.mark.usefixtures("load_birth_names_dashboard_with_slices")
@mock.patch.dict(
"superset.extensions.feature_flag_manager._feature_flags",
{"ENABLE_TEMPLATE_PROCESSING": True},
clear=True,
)
def test_sql_json_parameter_forbidden(self):
self.login("gamma")
self.login("gamma_sqllab_no_data")

data = self.run_sql(
"SELECT name FROM {{ table }} LIMIT 10",
"4",
template_params=json.dumps({"table": "birth_names"}),
)
assert data["errors"][0]["message"] == (
"The database referenced in this query was not found."
" Please contact an administrator for further assistance or try again."
)
assert data["errors"][0]["error_type"] == "GENERIC_BACKEND_ERROR"

@mock.patch("superset.sql_lab.get_query")
Expand Down

0 comments on commit 8f8fba2

Please sign in to comment.