jaceklaskowski / librarian-clojure

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

Fixed routes for CRUD. Added tests for compojure routes #4

Closed konrad-garus closed 12 years ago

konrad-garus commented 12 years ago

Added unit tests around compojure routes.

Apparently with Compojure /books/delete/15 is preferred over /books/delete?id=15. It also makes forms and tests easier, so I decided to go for this URL path scheme.

Generally, actions like add vs. update are now easily distinguished on URL mapping level and don't need conditions in our functions in books.clj.