mirror of
https://github.com/apache/superset.git
synced 2026-04-19 08:04:53 +00:00
fix(dashboard): Return columns and verbose_map for groupby values of Pivot Table v2 [ID-7] (#17287)
* fix(dashboard): Return columns and verbose_map for groupby values of Pivot Table v2 * Refactor * Fix test and lint * Fix test * Refactor * Fix lint
This commit is contained in:
committed by
GitHub
parent
ab1fcf3068
commit
fa51b3234e
@@ -518,7 +518,7 @@ class TestSqlaTableModel(SupersetTestCase):
|
||||
self.assertTrue("Metric 'invalid' does not exist", context.exception)
|
||||
|
||||
@pytest.mark.usefixtures("load_birth_names_dashboard_with_slices")
|
||||
def test_data_for_slices(self):
|
||||
def test_data_for_slices_with_no_query_context(self):
|
||||
tbl = self.get_table(name="birth_names")
|
||||
slc = (
|
||||
metadata_db.session.query(Slice)
|
||||
@@ -532,9 +532,35 @@ class TestSqlaTableModel(SupersetTestCase):
|
||||
assert len(data_for_slices["columns"]) == 1
|
||||
assert data_for_slices["metrics"][0]["metric_name"] == "sum__num"
|
||||
assert data_for_slices["columns"][0]["column_name"] == "gender"
|
||||
assert set(data_for_slices["verbose_map"].keys()) == set(
|
||||
["__timestamp", "sum__num", "gender",]
|
||||
assert set(data_for_slices["verbose_map"].keys()) == {
|
||||
"__timestamp",
|
||||
"sum__num",
|
||||
"gender",
|
||||
}
|
||||
|
||||
@pytest.mark.usefixtures("load_birth_names_dashboard_with_slices")
|
||||
def test_data_for_slices_with_query_context(self):
|
||||
tbl = self.get_table(name="birth_names")
|
||||
slc = (
|
||||
metadata_db.session.query(Slice)
|
||||
.filter_by(
|
||||
datasource_id=tbl.id,
|
||||
datasource_type=tbl.type,
|
||||
slice_name="Pivot Table v2",
|
||||
)
|
||||
.first()
|
||||
)
|
||||
data_for_slices = tbl.data_for_slices([slc])
|
||||
assert len(data_for_slices["metrics"]) == 1
|
||||
assert len(data_for_slices["columns"]) == 2
|
||||
assert data_for_slices["metrics"][0]["metric_name"] == "sum__num"
|
||||
assert data_for_slices["columns"][0]["column_name"] == "name"
|
||||
assert set(data_for_slices["verbose_map"].keys()) == {
|
||||
"__timestamp",
|
||||
"sum__num",
|
||||
"name",
|
||||
"state",
|
||||
}
|
||||
|
||||
|
||||
def test_literal_dttm_type_factory():
|
||||
|
||||
Reference in New Issue
Block a user