diff --git a/src/metabase/query_processor/middleware/annotate.clj b/src/metabase/query_processor/middleware/annotate.clj index 42c574fb54370997ee636f5c644b235422028202..d984cf4f0118e362d4af5b56507a94a841c8567d 100644 --- a/src/metabase/query_processor/middleware/annotate.clj +++ b/src/metabase/query_processor/middleware/annotate.clj @@ -44,8 +44,7 @@ (vec (for [i (range (count columns)) :let [col (nth columns i)]] {:name (name col) - :display_name (or (humanization/name->human-readable-name (u/keyword->qualified-name col)) - (u/keyword->qualified-name col)) + :display_name (u/keyword->qualified-name col) :base_type (or (driver.common/values->base-type (for [row rows] (nth row i))) :type/*) diff --git a/test/metabase/api/table_test.clj b/test/metabase/api/table_test.clj index 4874ef146dad66871e2bed4f1952aebbe2ebbb28..3a4deccf7e123fedcbe772de9bf57f663d823517 100644 --- a/test/metabase/api/table_test.clj +++ b/test/metabase/api/table_test.clj @@ -450,7 +450,7 @@ :fields (map (comp #(merge (default-card-field-for-venues card-virtual-table-id) %) with-field-literal-id) [{:name "NAME" - :display_name "Name" + :display_name "NAME" :base_type "type/Text" :special_type "type/Name" :fingerprint (:name mutil/venue-fingerprints)} @@ -461,13 +461,13 @@ :fingerprint (:id mutil/venue-fingerprints)} (with-numeric-dimension-options {:name "PRICE" - :display_name "Price" + :display_name "PRICE" :base_type "type/Integer" :special_type nil :fingerprint (:price mutil/venue-fingerprints)}) (with-coordinate-dimension-options {:name "LATITUDE" - :display_name "Latitude" + :display_name "LATITUDE" :base_type "type/Float" :special_type "type/Latitude" :fingerprint (:latitude mutil/venue-fingerprints)})])}) @@ -496,7 +496,7 @@ :description nil :dimension_options (default-dimension-options) :fields [{:name "NAME" - :display_name "Name" + :display_name "NAME" :base_type "type/Text" :table_id card-virtual-table-id :id ["field-literal" "NAME" "type/Text"] @@ -508,7 +508,7 @@ :type {:type/Text {:percent-json 0.0, :percent-url 0.0, :percent-email 0.0, :average-length 13.27}}}} {:name "LAST_LOGIN" - :display_name "Last Login" + :display_name "LAST_LOGIN" :base_type "type/DateTime" :table_id card-virtual-table-id :id ["field-literal" "LAST_LOGIN" "type/DateTime"] diff --git a/test/metabase/driver/sql_jdbc/native_test.clj b/test/metabase/driver/sql_jdbc/native_test.clj index a359de99d8e7cc9ced70fc91c8c452038b60e487..491844d118ba912012b471257f1ccdff93bc37a0 100644 --- a/test/metabase/driver/sql_jdbc/native_test.clj +++ b/test/metabase/driver/sql_jdbc/native_test.clj @@ -33,8 +33,8 @@ [99 "Golden Road Brewing" 10]] :columns ["ID" "NAME" "CATEGORY_ID"] :cols [{:name "ID", :display_name "ID", :source :native, :base_type :type/Integer} - {:name "NAME", :display_name "Name", :source :native, :base_type :type/Text} - {:name "CATEGORY_ID", :display_name "Category ID", :source :native, :base_type :type/Integer}] + {:name "NAME", :display_name "NAME", :source :native, :base_type :type/Text} + {:name "CATEGORY_ID", :display_name "CATEGORY_ID", :source :native, :base_type :type/Integer}] :native_form {:query "SELECT ID, NAME, CATEGORY_ID FROM VENUES ORDER BY ID DESC LIMIT 2", :params []}}} (-> (qp/process-query {:native {:query "SELECT ID, NAME, CATEGORY_ID FROM VENUES ORDER BY ID DESC LIMIT 2"} :type :native diff --git a/test/metabase/query_processor/middleware/annotate_test.clj b/test/metabase/query_processor/middleware/annotate_test.clj index 2247bb96c85a21cacbbf2533913cb89e1b7c7668..86b012eef9d442678a2981ee7af1ee44423e0b8a 100644 --- a/test/metabase/query_processor/middleware/annotate_test.clj +++ b/test/metabase/query_processor/middleware/annotate_test.clj @@ -19,8 +19,8 @@ ;; make sure that `add-native-column-info` can still infer types even if the initial value(s) are `nil` (#4256) (expect - [{:name "a", :display_name "A", :base_type :type/Integer, :source :native} - {:name "b", :display_name "B", :base_type :type/Integer, :source :native}] + [{:name "a", :display_name "a", :base_type :type/Integer, :source :native} + {:name "b", :display_name "b", :base_type :type/Integer, :source :native}] (:cols (#'annotate/add-native-column-info {:columns [:a :b], :rows [[1 nil] [2 nil] [3 nil] @@ -30,7 +30,7 @@ ;; make sure that `add-native-column-info` defaults `base_type` to `type/*` if there are no non-nil ;; values when we peek. (expect - [{:name "a", :display_name "A", :base_type :type/*, :source :native}] + [{:name "a", :display_name "a", :base_type :type/*, :source :native}] (:cols (#'annotate/add-native-column-info {:columns [:a], :rows [[nil]]}))) diff --git a/test/metabase/query_processor/middleware/results_metadata_test.clj b/test/metabase/query_processor/middleware/results_metadata_test.clj index b92c0149360a140a15d2102f6004cbb574f9878a..8f8c733657dbf1b375174dd3b4023e82d32d95a5 100644 --- a/test/metabase/query_processor/middleware/results_metadata_test.clj +++ b/test/metabase/query_processor/middleware/results_metadata_test.clj @@ -48,7 +48,9 @@ :special_type "type/Longitude", :fingerprint (:longitude mutil/venue-fingerprints)}]) (def ^:private default-card-results-native - (update-in default-card-results [3 :fingerprint] assoc :type {:type/Number {:min 2.0, :max 74.0, :avg 29.98, :q1 7.0, :q3 49.0 :sd 23.06}})) + (for [column (-> default-card-results + (update-in [3 :fingerprint] assoc :type {:type/Number {:min 2.0, :max 74.0, :avg 29.98, :q1 7.0, :q3 49.0 :sd 23.06}}))] + (assoc column :display_name (:name column)))) ;; test that Card result metadata is saved after running a Card (expect