Fix fingerprinting Postgres table with time fields [ci drivers]

src/metabase/query_processor/middleware/resolve.clj

@@ -168,8 +168,9 @@
                                                                (merge-non-nils (select-keys this [:fk-field-id :remapped-from :remapped-to :field-display-name])))]
     ;; try to resolve the Field with the ones available in field-id->field
     (cond
-      (or (isa? base-type :type/DateTime)
-          (isa? special-type :type/DateTime))
+      (and (or (isa? base-type :type/DateTime)
+               (isa? special-type :type/DateTime))
+           (not (isa? base-type :type/Time)))
       (i/map->DateTimeField {:field field
                              :unit  (or datetime-unit :day)}) ; default to `:day` if a unit wasn't specified
 

test/metabase/driver/postgres_test.clj

@@ -277,3 +277,28 @@
 (expect-with-engine :postgres
   "UTC"
   (tu/db-timezone-id))
+
+
+;; Make sure we're able to fingerprint TIME fields (#5911)
+(expect-with-engine :postgres
+  [#metabase.models.field.FieldInstance{:name "start_time", :fingerprint {:global {:distinct-count 1}}}
+   #metabase.models.field.FieldInstance{:name "end_time",   :fingerprint {:global {:distinct-count 1}}}
+   #metabase.models.field.FieldInstance{:name "reason",     :fingerprint {:global {:distinct-count 1}
+                                                                          :type   {:type/Text {:percent-json    0.0
+                                                                                               :percent-url     0.0
+                                                                                               :percent-email   0.0
+                                                                                               :average-length 12.0}}}}]
+  (do
+    (drop-if-exists-and-create-db! "time_field_test")
+    (let [details (i/database->connection-details pg-driver :db {:database-name "time_field_test"})]
+      (jdbc/execute! (sql/connection-details->spec pg-driver details)
+                     [(str "CREATE TABLE toucan_sleep_schedule ("
+                           "  start_time TIME WITHOUT TIME ZONE NOT NULL, "
+                           "  end_time TIME WITHOUT TIME ZONE NOT NULL, "
+                           "  reason VARCHAR(256) NOT NULL"
+                           ");"
+                           "INSERT INTO toucan_sleep_schedule (start_time, end_time, reason) "
+                           "  VALUES ('22:00'::time, '9:00'::time, 'Beauty Sleep');")])
+      (tt/with-temp Database [database {:engine :postgres, :details (assoc details :dbname "time_field_test")}]
+        (sync/sync-database! database)
+        (db/select [Field :name :fingerprint] :table_id (db/select-one-id Table :db_id (u/get-id database)))))))