Skip to content
Snippets Groups Projects
Unverified Commit 1852be04 authored by Cam Saul's avatar Cam Saul Committed by GitHub
Browse files

Test for #11210 (#35520)

parent 866c34e8
No related branches found
No related tags found
No related merge requests found
......@@ -29,6 +29,7 @@
[:field {:binning {:strategy :num-bins :num-bins 4}} 1]
Pass `nil` `binning` to remove any binning."
{:style/indent [:form]}
[x binning :- [:maybe [:or ::lib.schema.binning/binning ::lib.schema.binning/binning-option]]]
(with-binning-method x (if (contains? binning :mbql)
(:mbql binning)
......
......@@ -253,3 +253,49 @@
[:field {} (meta/id :people :longitude)]
50
55]]}]}})))))
(deftest ^:parallel zoom-in-on-join-test
(testing "#11210"
(let [query (-> (lib/query meta/metadata-provider (meta/table-metadata :orders))
(lib/join (meta/table-metadata :people))
(lib/aggregate (lib/count))
(lib/breakout (lib/with-binning (meta/field-metadata :people :latitude)
{:strategy :bin-width, :bin-width 10}))
(lib/breakout (lib/with-binning (meta/field-metadata :people :longitude)
{:strategy :bin-width, :bin-width 10})))]
(doseq [column ["LATITUDE" "LONGITUDE"]]
(lib.drill-thru.tu/test-drill-application
{:click-type :cell
:query-type :aggregated
:custom-query query
:custom-row {"count" 100
"LATITUDE" 20
"LONGITUDE" 50}
:column-name column
:drill-type :drill-thru/zoom-in.geographic
:expected {:type :drill-thru/zoom-in.geographic
:subtype :drill-thru.zoom-in.geographic/binned-lat-lon->binned-lat-lon
:latitude {:column {:name "LATITUDE"}
:bin-width 1.0
:min 20
:max 30}
:longitude {:column {:name "LONGITUDE"}
:bin-width 1.0
:min 50
:max 60}}
:expected-query {:stages [{:source-table (meta/id :orders)
:aggregation [[:count {}]]
:breakout [[:field
{:binning {:strategy :bin-width, :bin-width 1.0}}
(meta/id :people :latitude)]
[:field
{:binning {:strategy :bin-width, :bin-width 1.0}}
(meta/id :people :longitude)]]
:filters [[:between {}
[:field {} (meta/id :people :latitude)]
20
30]
[:between {}
[:field {} (meta/id :people :longitude)]
50
60]]}]}})))))
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment