From b49fd65cf7dd5b8a85527efcfe3ee6ac44e8a543 Mon Sep 17 00:00:00 2001 From: Kasia Kucharczyk <2536609+kkucharc@users.noreply.github.com> Date: Tue, 27 Oct 2020 14:28:52 +0100 Subject: [PATCH] fix: added meaningful exception for loading births error (#11407) * Added exception for loading data without previous loaded test users. * Changed general exception to NoDataException * Changed import to absolute path --- superset/examples/birth_names.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/superset/examples/birth_names.py b/superset/examples/birth_names.py index 3f5f3beb06..d7e6940c62 100644 --- a/superset/examples/birth_names.py +++ b/superset/examples/birth_names.py @@ -25,6 +25,7 @@ from sqlalchemy.sql import column from superset import db, security_manager from superset.connectors.base.models import BaseDatasource from superset.connectors.sqla.models import SqlMetric, TableColumn +from superset.exceptions import NoDataException from superset.models.core import Database from superset.models.dashboard import Dashboard from superset.models.slice import Slice @@ -41,6 +42,12 @@ from .helpers import ( ) admin = security_manager.find_user("admin") +if admin is None: + raise NoDataException( + "Admin user does not exist. " + "Please, check if test users are properly loaded " + "(`superset load_test_users`)." + ) def gen_filter(