noir-clojure / lib-noir

A set of libraries for ring apps, including stateful sessions.
Eclipse Public License 1.0
483 stars 47 forks source link

Move doc strings to correct place in defn, and partially corrected a tes... #92

Closed jafingerhut closed 10 years ago

jafingerhut commented 10 years ago

...t

The test still fails, but it is because the implementation of cache! allows the cache to go 1 over the max-size if it currently has max-size elements, and then an item is added that is not already in the cache. I will let the lib-noir developers decide whether they want to change the test to match the behavior, or change cache!

Issues found using a pre-release version of the Eastwood Clojure lint tool.

yogthos commented 10 years ago

looks good to me