Fork me on GitHub
#test-check
<
2018-04-17
>
mathpunk03:04:10

I was trying to deal with this bug (https://github.com/clojure-emacs/cider/issues/1841), and in investigating it I discovered that test.check refers to a namespace that it doesn't have in the definition of test-context-stacktrace: https://github.com/clojure/test.check/blob/master/src/main/clojure/clojure/test/check/clojure_test/assertions.cljc#L12

mathpunk03:04:07

I commented that line out, and my problem went away. It's not clear to me how to contribute or report that so, I'm mentioning it here

mathpunk03:04:57

As I look at it more closely, I doubt it's that exact spot that helped; but, there are definitely several references to a clojure.test.check.clojure-test namespace. I see only a clojure_test directory, but no namespace

Alex Miller (Clojure team)04:04:53

that’s due to someone monkey-patching clojure.test

mathpunk04:04:03

While I did see this, I don't quiiiiite know what it means, nor what it implies about my project being a boot project

mathpunk04:04:24

I've heard the term but only the context of 1) Ruby objects and 2) disapproval. Sometimes at once~

Alex Miller (Clojure team)04:04:15

well this may be something similar but different, not sure. Leiningen reaches in and replaces the implementation of part of clojure.test, which breaks test.check’s expectation.

Alex Miller (Clojure team)04:04:31

that’s a pretty dirty trick imo :)

ghadi14:04:32

Nice @nwjsmith. That will be very useful, also good learning material.

nwjsmith14:04:40

I'll try to spruce it up a bit this week. I think there's a ton of performance improvements available too, so I'd like to get some benchmarks around it.