_评论由:grzm_
对于 `
:failure
` 的 `
false
` 返回值确实非常令人困惑。正如 [~djebbz] 指出的,`
stest/abbrev-result
` 在 `
{{{:failure false}}}
` 中非常让人费解,因为它不提供通常失败时所做的额外信息。
{code:title=
https://github.com/clojure/spec.alpha/blob/2824ad49df8deadcb4b75acdf624e732a85b4ac7/src/main/clojure/clojure/spec/test/alpha.clj#L438-L446}
(defn abbrev-result
"给定一个检查结果,返回一个适合总结使用的简略版本。"
[x]
(if (:failure x)
(-> (dissoc x ::stc/ret)
(update :spec s/describe)
(update :failure unwrap-failure))
(dissoc x :spec ::stc/ret)))
以下是一个示例,说明它可能会多么具有误导性
(require '[clojure.spec.alpha :as s]
'[clojure.spec.test.alpha :as stest])
(alias 'stc 'clojure.spec.test.check)
(defn adder [a b]
(+ a b))
(s/fdef adder
:args (s/cat :a int? :b int?)
:ret string?)
(-> (stest/check `adder) first stest/abbrev-result)
;; => {:sym ex.check-test/adder, :failure false}
;; 写一个 `abbrev-result` 的替代版本,它检查为 `true`
(defn- failure-type [x] (::s/failure (ex-data x)))
(defn- unwrap-failure [x] (if (failure-type x) (ex-data x) x))
(defn- abbrev-result [x]
(let [failure (:failure x)]
(if-not (or (true? failure)
(nil? failure)))
(-> (dissoc x ::stc/ret)
(update :spec s/describe)
(update :failure unwrap-failure))
(dissoc x :spec ::stc/ret))))
(-> (stest/check `adder) first abbrev-result)
;; => {:spec (fspec :args (cat :a int? :b int?) :ret string? :fn nil),
;; :sym ex.check-test/adder,
;; :failure false}
再次注意,任何有意义的 {{:failure}} 值都将提供附加详情,而不做声明的 {{:failure}} 值则不会。
我理解了不更改 {{:failure}} 键值的动机。如果该值要保留,我认为 {{stest/abbrev-result}} 应当相应更新,以便显式地测试 {{nil}} 和 {{true}},而不是测试有意义的值以保持与 {{stest/check}} 的结果一致性。