corrected users test cases and added new test cases for do-register #178

Merged
merged 1 commit into from Nov 21, 2011

4 participants

@ckirkendall

I was able to use midje function mocking to test most base cases

@amalloy

Why are we reloading here? I'm having trouble imagining a scenario in which this is necessary.

@amalloy
4clojure member

This is pretty awesome, thanks Creighton. I'm a little sad that we're mocking out specific error messages for flash-error - can we use anything or something like that here? OTOH, the fact that we want those messages in more than one place indicates that maybe they should be pulled out into some sort of global constant.

@amalloy amalloy merged commit 896f83c into 4clojure:develop Nov 21, 2011
@gigasquid
4clojure member

Nicely done. We should also think about putting some extra documentation in the README about running lein test to run the test cases. That way - as we build up our tests, new contributors know to run them to make sure that nothing breaks.

@dbyrne
4clojure member

@ckirkendall Very cool. Keep up the good work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment