_评论人: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}},而不是仅测试 truthy 以保持与 {{stest/check}} 结果的一致性。