_评论人: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}
;; 写一个检查`true`的`abbrev-result`的替代版本
(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 null),
;; :sym ex.check-test/adder,
;; :failure false}
再次注意,任何*真正的*{{:failure}}值都将会提供附加的详细信息,而假的{{:failure}}值则不会。
我理解了为什么不想改变 {{:failure}} 键的值。如果这个值要保留,我认为应该将 {{stest/abbrev-result}} 更新为类似地显式测试 {{nil}} 和 {{true}},而不是测试 truthy 以保持与 {{stest/check}} 的结果一致性。