Fix dalli_store#delete error handling, #196

petergoldstein · Apr 10, 2012 · 020c2e5 · 020c2e5
1 parent 5f2fcad
commit 020c2e5
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 1 deletion.
diff --git a/History.md b/History.md
@@ -5,6 +5,7 @@ HEAD
 =======
 
 - Allow non-ascii and whitespace keys, only the text protocol has those restrictions [#145]
+- Fix DalliStore#delete error-handling [#196]
 
 2.0.2
 =======

diff --git a/lib/active_support/cache/dalli_store.rb b/lib/active_support/cache/dalli_store.rb
@@ -72,7 +72,7 @@ def exist?(name, options=nil)
       end
 
       def delete(name, options=nil)
-        @data.delete(name)
+        delete_entry(name, options)
       end
 
       # Reads multiple keys from the cache using a single call to the

diff --git a/test/test_active_support.rb b/test/test_active_support.rb
@@ -13,6 +13,7 @@
       assert_equal 1, @dalli.increment('lkjsa', 1, nil)
       assert_equal 2, @dalli.increment('lkjsa', 1, nil)
       assert_equal 1, @dalli.decrement('lkjsa', 1, nil)
+      assert_equal true, @dalli.delete('lkjsa')
     end
 
     should 'support fetch' do