Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ultra assert-expr changes :actual value #105

Open
Deraen opened this issue Mar 13, 2019 · 0 comments
Open

Ultra assert-expr changes :actual value #105

Deraen opened this issue Mar 13, 2019 · 0 comments

Comments

@Deraen
Copy link

Deraen commented Mar 13, 2019

Clojure.test built-in assert-expr will set the :actual value as list of the predicate (=) and the values: https://github.com/clojure/clojure/blob/master/src/clj/clojure/test.clj#L450

Ultra just sets the value as list of the values: https://github.com/venantius/ultra/blob/master/src/ultra/test.clj#L22 without the predicate.

This breaks third party reporters like Eftest: weavejester/eftest#67

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant