Skip to content
Snippets Groups Projects
Unverified Commit 28ddf3ad authored by Cam Saul's avatar Cam Saul Committed by GitHub
Browse files

Fix the flaky paging test (#28475)

parent de21b7d6
No related branches found
No related tags found
No related merge requests found
...@@ -6,6 +6,7 @@ ...@@ -6,6 +6,7 @@
[metabase.server.handler :as handler] [metabase.server.handler :as handler]
[metabase.server.middleware.offset-paging :as mw.offset-paging] [metabase.server.middleware.offset-paging :as mw.offset-paging]
[metabase.server.middleware.security :as mw.security] [metabase.server.middleware.security :as mw.security]
[metabase.test :as mt]
[ring.mock.request :as ring.mock] [ring.mock.request :as ring.mock]
[ring.util.response :as response]) [ring.util.response :as response])
(:import (:import
...@@ -47,9 +48,11 @@ ...@@ -47,9 +48,11 @@
"paged?" true "paged?" true
"params" {"whatever" "true"}}} "params" {"whatever" "true"}}}
(read-response (handler (ring.mock/request :get "/" {:offset "200", :limit "100", :whatever "true"})))))) (read-response (handler (ring.mock/request :get "/" {:offset "200", :limit "100", :whatever "true"}))))))
;; set the system clock here so this doesn't flake if we cross the second boundary between evaluating the expected
(testing "invalid params" ;; form and the actual form
(is (=? {:status 400 (mt/with-clock #t "2023-02-20T15:01:00-08:00[US/Pacific]"
:headers (mw.security/security-headers) (testing "invalid params"
:body {"message" #"Error parsing paging parameters.*"}} (is (=? {:status 400
(read-response (handler (ring.mock/request :get "/" {:offset "abc", :limit "100"}))))))) :headers (mw.security/security-headers)
:body {"message" #"Error parsing paging parameters.*"}}
(read-response (handler (ring.mock/request :get "/" {:offset "abc", :limit "100"}))))))))
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