luciano/reitit
1
0
Fork 0
mirror of https://github.com/metosin/reitit.git synced 2025-12-18 00:41:12 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Add test to validate middleware cannot be list

Browse source 
When meta-merges list with a vector, the resulting order is reversed. To avoid
this for the middleware chain, middleware cannot be lists.
This commit is contained in:
Daw-Ran Liou 2019-10-09 07:55:59 -07:00 committed by Tommi Reiman
parent eeeba281ac
commit 4a199b918a
1 changed files with 9 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
test/cljc/reitit/ring_spec_test.cljc
View file

@ -77,7 +77,15 @@
:data {:middleware [{:spec (s/keys :opt-un [::roles]) :data {:middleware [{:spec (s/keys :opt-un [::roles])
:wrap (fn [handler] :wrap (fn [handler]
(fn [request] (fn [request]
(handler request)))}]}}))))) (handler request)))}]}}))))
(testing "middleware cannot be a list"
(is (thrown-with-msg?
ExceptionInfo
#":reitit.ring.spec/invalid-specs"
(ring/router
["/api" {:handler identity
:middleware '()}]
{:validate rrs/validate})))))
(deftest coercion-spec-test (deftest coercion-spec-test
(is (r/router? (is (r/router?