Minor fixes for reflection and a long line in the mysql tests

src/metabase/driver/generic_sql/query_processor.clj

@@ -471,11 +471,11 @@
 (defn- run-query-without-timezone [driver settings connection query]
   (do-in-transaction connection (partial run-query query nil)))
 
-(defn- run-query-with-timezone [driver settings connection query]
+(defn- run-query-with-timezone [driver {:keys [^String report-timezone] :as settings} connection query]
   (try
     (do-in-transaction connection (fn [transaction-connection]
                                     (set-timezone! driver settings transaction-connection)
-                                    (run-query query (some-> settings :report-timezone TimeZone/getTimeZone) transaction-connection)))
+                                    (run-query query (some-> report-timezone TimeZone/getTimeZone) transaction-connection)))
     (catch SQLException e
       (log/error "Failed to set timezone:\n" (with-out-str (jdbc/print-sql-exception-chain e)))
       (run-query-without-timezone driver settings connection query))

test/metabase/driver/mysql_test.clj

@@ -31,7 +31,9 @@
 
 ;; make sure connection details w/ extra params work as expected
 (expect
-  "//localhost:3306/cool?zeroDateTimeBehavior=convertToNull&useUnicode=true&characterEncoding=UTF8&characterSetResults=UTF8&useLegacyDatetimeCode=true&useJDBCCompliantTimezoneShift=true&useSSL=false&tinyInt1isBit=false"
+  (str "//localhost:3306/cool?zeroDateTimeBehavior=convertToNull&useUnicode=true&characterEncoding=UTF8"
+       "&characterSetResults=UTF8&useLegacyDatetimeCode=true&useJDBCCompliantTimezoneShift=true"
+       "&useSSL=false&tinyInt1isBit=false")
   (:subname (sql/connection-details->spec (MySQLDriver.) {:host               "localhost"
                                                           :port               "3306"
                                                           :dbname             "cool"