Phil Hagelberg <p...@hagelb.org> writes:

> Would it be a good idea to change deftest so that it accepts an optional
> metadata map just like defn? I can implement it if it's wanted.

Here's the ticket and patch:

https://www.assembla.com/spaces/clojure/tickets/201-Make-deftest-keep-var-metadata

I ran into a very confusing problem while I was testing my change. The
custom-report function defined at the bottom of the test makes it so
that any tests that are defined that do not have "Should pass" as their
message will fail. I flailed around for quite some time, starting to
wonder if I was losing it when my simple (is true) tests did not pass.

I understand the need to be able to make sure that certain tests cause
errors or failures, but perhaps making the msg argument required only in
those cases (rather than for every single test) would cause less
confusion.

thanks,
Phil

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google
Groups "Clojure" group.
To post to this group, send email to clojure@googlegroups.com
Note that posts from new members are moderated - please be patient with your 
first post.
To unsubscribe from this group, send email to
clojure+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/clojure?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to