make the cache test more @tenderlove(ing) and not depend on internal …

…implentation
veader · Oct 24, 2012 · 96f5cea · 96f5cea
1 parent 81b7112
commit 96f5cea
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/activesupport/test/caching_test.rb b/activesupport/test/caching_test.rb
@@ -198,8 +198,13 @@ def test_fetch_with_cache_miss
   end
 
   def test_fetch_with_cache_miss_passes_key_to_block
-    @cache.expects(:write).with('foo', 3, @cache.options)
-    assert_equal 3, @cache.fetch('foo') { |key| key.length }
+    cache_miss = false
+    assert_equal 3, @cache.fetch('foo') { |key| cache_miss = true; key.length }
+    assert cache_miss
+
+    cache_miss = false
+    assert_equal 3, @cache.fetch('foo') { |key| cache_miss = true; key.length }
+    assert !cache_miss
   end
 
   def test_fetch_with_forced_cache_miss