Skip to content
Snippets Groups Projects
Unverified Commit 8eb45851 authored by lbrdnk's avatar lbrdnk Committed by GitHub
Browse files

Unskip one and skip another relative datetime test (#36580)

* Revert "Temporarily skip server-side-relative-datetime-honeysql-form (#36567)"

This reverts commit 769b5eb4.
This was not flaking.

* Skip test that flakes
parent a7bb5969
No related branches found
No related tags found
No related merge requests found
......@@ -4,7 +4,7 @@
[clojure.string :as str]
[clojure.test :refer :all]
[honey.sql :as sql]
#_[java-time.api :as t]
[java-time.api :as t]
[metabase.driver :as driver]
[metabase.driver.redshift :as redshift]
[metabase.driver.sql-jdbc.connection :as sql-jdbc.conn]
......@@ -578,7 +578,8 @@
;; => 108
)
(deftest server-side-relative-datetime-various-units-test
;;;; Flakes in CI around 0400 UTC. Investigating.
#_(deftest server-side-relative-datetime-various-units-test
(mt/test-driver
:redshift
(mt/with-metadata-provider (mt/id)
......@@ -588,8 +589,7 @@
:let [test-thunk (getdate-vs-ss-ts-test-thunk-generator unit value)]]
(test-thunk))))))
;; Dec 8th 2023, Temporarily skip this to unblock master, QP should fix this soon
#_(deftest server-side-relative-datetime-truncation-test
(deftest server-side-relative-datetime-truncation-test
(mt/test-driver
:redshift
(testing "Datetime _truncation_ works correctly over different timezones"
......
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