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

fix: openapi malli parameter descriptions

Browse source 
... should come from the parameter type, not from the parent :map
This commit is contained in:
Joel Kaasinen 2023-05-16 18:03:40 +03:00
parent 2e8e9265d9
commit b56c15b64c
2 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
modules/reitit-malli/src/reitit/coercion/malli.cljc
View file

@ -145,14 +145,14 @@
(when (seq parameters)
{:parameters
(->> (for [[in schema] parameters
:let [{:keys [properties required] :as root} (->schema-object schema {:in in :type :parameter})
:let [{:keys [properties required]} (->schema-object schema {:in in :type :parameter})
required? (partial contains? (set required))]
[k schema] properties]
(merge {:in (name in)
:name k
:required (required? k)
:schema schema}
(select-keys root [:description])))
(select-keys schema [:description])))
(into []))})
(when body
;; body uses a single schema to describe every :requestBody

2
test/cljc/reitit/openapi_test.clj
View file

@ -365,7 +365,7 @@
(deftest all-parameter-types-test
(doseq [[coercion ->schema]
[[#'malli/coercion (fn [nom] [:map {:description (str "description " nom)} [nom :string]])]
[[#'malli/coercion (fn [nom] [:map [nom [:string {:description (str "description " nom)}]]])]
[#'schema/coercion (fn [nom] {nom s/Str})]
[#'spec/coercion (fn [nom] {nom string?})]]]
(testing coercion