cached_counts 0.2.0 → 0.2.1

data/lib/cached_counts/cache.rb

@@ -5,14 +5,7 @@ module CachedCounts
     end
 
     def count(*args)
-      if all_keys.include?(current_key)
-        Rails.cache.fetch(current_key)
-      else
-        @scope.count_without_caching(*args).tap do |count|
-          Rails.cache.write(current_key, count)
-          Rails.cache.write(list_key, all_keys + [current_key])
-        end
-      end
+      cached_count || uncached_count(*args)
     end
 
     # Clear out any count caches which have SQL that includes the scopes table
@@ -26,6 +19,17 @@ module CachedCounts
 
     private
 
+    def cached_count
+      Rails.cache.fetch(current_key)
+    end
+
+    def uncached_count(*args)
+      @scope.count_without_caching(*args).tap do |count|
+        Rails.cache.write(current_key, count)
+        Rails.cache.write(list_key, all_keys + [current_key])
+      end
+    end
+
     def all_keys
       Rails.cache.fetch(list_key) || []
     end

data/lib/cached_counts/version.rb

@@ -1,3 +1,3 @@
 module CachedCounts
-  VERSION = '0.2.0'
+  VERSION = '0.2.1'
 end

data/spec/cached_counts/cache_spec.rb

@@ -20,6 +20,18 @@ describe CachedCounts::Cache do
       end
     end
 
+    context 'when the cached key is blank' do
+      before do
+        scope.stub(:count_without_caching => 5)
+        cache.count
+        Rails.cache.write(cache.send(:current_key), nil)
+      end
+
+      it 'returns the real count' do
+        cache.count.should == 5
+      end
+    end
+
     context 'when there is already a cached count' do
       before do
         scope.should_receive(:count_without_caching).once.and_return(8)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cached_counts
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
   prerelease: 
 platform: ruby
 authors: