jaceklaskowski / librarian-clojure

Book manager in Clojure
http://librarian-clojure.herokuapp.com/
22 stars 4 forks source link

Core future facts #32

Closed zoldar closed 11 years ago

zoldar commented 11 years ago

I've fixed the facts in core and added a simple (and actually the only relevant in my opinion) test for function retrieving user details for use by cemerick/friend. I've tried adding some tests for signup-handler-workflow but came to conclusion that it's not worth the sweat (just one if branch actually) - while trying to accomplish it (and also on occasion of fixing earlier facts) I've tripped up on nasty "provided" behavior which - when put in the wrong context - caused a complete repl lockup. When such malformed fact was run from emacs buffer, I've often ended up forcefully killing all repl related clojure processes. Anyway that's all for tonight on my side - not much actually, but I've wasted considerable amount of time trying to sort out that "provided" issue...