[CLJ-1313] Correct a few unit tests Created: 23/Dec/13 Updated: 12/Sep/14
|Affects Version/s:||Release 1.5, Release 1.6|
|Patch:||Code and Test|
Several unit tests do not test what they appear to have been intended to test, because of missing is statements around (= expr1 expr2) expressions, or because of use of (is (thrown? ...)) instead of (is (thrown-with-msg? ...))
Found with pre-release version of Eastwood Clojure lint tool.
|Comment by Andy Fingerhut [ 23/Dec/13 3:11 AM ]|
Patch clj-1313-v1.diff wraps (is ...) around several = expressions in unit tests that appeared to have been missing them, and changes several thrown? to thrown-with-msg? when there were regexes that were unused.
|Comment by Stuart Halloway [ 31/Jan/14 12:36 PM ]|
please update to apply cleanly on master
|Comment by Andy Fingerhut [ 31/Jan/14 3:29 PM ]|
clj-1313-v2.diff is identical to clj-1313-v1.diff except that it removes the portion that conflicts with the latest Clojure master. That portion needs updating for a different reason anyway (ticket