artsy/garner

BindingIndex and SafeCacheKey key strategies both incorrectly return non-nil cache keys for Model.identify(nil)

fancyremarker opened this issue · 0 comments

fancyremarker commented 
Garner::Strategies::Binding::Key::BindingIndex.apply(Model.identify(nil))
# => "2986056a81ed9ec670d172f8"
Garner::Strategies::Binding::Key::SafeCacheKey.apply(Model.identify(nil))
# => "models/516738f913d2c34741000002-20130423234925.0000000000"

Thanks @joeyAghion for catching this.