Skip to content
Snippets Groups Projects
Commit 3b17415d authored by Cam Saül's avatar Cam Saül
Browse files

Use new driver/format-custom-field-name method in tests [ci drivers]

parent b3f4d08f
No related branches found
No related tags found
No related merge requests found
(ns metabase.query-processor-test.expression-aggregations-test
"Tests for expression aggregations and for named aggregations."
(:require [expectations :refer :all]
[metabase.driver :as driver]
[metabase.models.metric :refer [Metric]]
[metabase.query-processor :as qp]
[metabase.query-processor.expand :as ql]
[metabase.query-processor-test :refer :all]
[metabase.test.data :as data]
[metabase.test.data.datasets :as datasets, :refer [*engine*]]
[metabase.test.data.datasets :as datasets, :refer [*engine* *driver*]]
[metabase.test.util :as tu]
[metabase.util :as u]))
......@@ -167,7 +168,7 @@
[3 52]
[4 30]]
:columns [(data/format-name "price")
(if (= *engine* :redshift) "new price" "New Price")]} ; Redshift annoyingly always lowercases column aliases
(driver/format-custom-field-name *driver* "New Price")]} ; Redshift annoyingly always lowercases column aliases
(format-rows-by [int int]
(rows+column-names (data/run-query venues
(ql/aggregation (ql/named (ql/sum (ql/+ $price 1)) "New Price"))
......@@ -180,7 +181,7 @@
[3 -2]
[4 -17]]
:columns [(data/format-name "price")
(if (= *engine* :redshift) "sum-41" "Sum-41")]}
(driver/format-custom-field-name *driver* "Sum-41")]}
(format-rows-by [int int]
(rows+column-names (data/run-query venues
(ql/aggregation (ql/named (ql/- (ql/sum $price) 41) "Sum-41"))
......@@ -208,7 +209,7 @@
[3 39]
[4 24]]
:columns [(data/format-name "price")
(if (= *engine* :redshift) "my cool metric" "My Cool Metric")]}
(driver/format-custom-field-name *driver* "My Cool Metric")]}
(tu/with-temp Metric [metric {:table_id (data/id :venues)
:definition {:aggregation [:sum [:field-id (data/id :venues :price)]]
:filter [:> [:field-id (data/id :venues :price)] 1]}}]
......@@ -226,7 +227,7 @@
[3 39]
[4 24]]
:columns [(data/format-name "price")
(if (= *engine* :redshift) "my cool metric" "My Cool Metric")]}
(driver/format-custom-field-name *driver* "My Cool Metric")]}
(tu/with-temp Metric [metric {:table_id (data/id :venues)
:definition {:aggregation [[:sum [:field-id (data/id :venues :price)]]]
:filter [:> [:field-id (data/id :venues :price)] 1]}}]
......@@ -240,7 +241,7 @@
;; check that named aggregations come back with the correct column metadata (#4002)
(datasets/expect-with-engines (engines-that-support :expression-aggregations)
(let [col-name (if (= *engine* :redshift) "count of things" "Count of Things")]
(let [col-name (driver/format-custom-field-name *driver* "Count of Things")]
(assoc (aggregate-col :count)
:name col-name
:display_name col-name))
......
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