arid_cache 1.4.2 → 1.4.3

data/Gemfile.rails2.lock

@@ -1,7 +1,7 @@
 PATH
   remote: ./
   specs:
-    arid_cache (1.4.0)
+    arid_cache (1.4.3)
       will_paginate
 
 GEM

data/Gemfile.rails3.lock

@@ -1,7 +1,7 @@
 PATH
   remote: ./
   specs:
-    arid_cache (1.3.6)
+    arid_cache (1.4.3)
       will_paginate
 
 GEM

data/VERSION

@@ -1 +1 @@
-1.4.2
+1.4.3

data/arid_cache.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = %q{arid_cache}
-  s.version = "1.4.2"
+  s.version = "1.4.3"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Karl Varga"]

data/lib/arid_cache/cache_proxy/utilities.rb

@@ -39,7 +39,9 @@ module AridCache
       # Find and return records of the given +klass+ which have id in +ids+.
       # +find_opts+ is a hash of options which are passed to find.
       # If no order option is given, the ordering of the ids is preserved.
+      # No query is performed if +ids+ is empty.
       def find_all_by_id(klass, ids, find_opts={})
+        return ids if ids.empty?
         find_opts = Options.new(find_opts.merge(:result_klass => klass)).opts_for_find(ids)
         if AridCache.framework.active_record?(3)
           option_map = {

data/spec/arid_cache/cache_proxy/result_processor_spec.rb

@@ -7,8 +7,6 @@ describe AridCache::CacheProxy::ResultProcessor do
   end
 
   before :each do
-    AridCache.store.delete! # so no options get stored and interfere with other tests
-    Rails.cache.clear
     AridCache.raw_with_options = true
   end
 

data/spec/arid_cache/cache_proxy/utilities_spec.rb

@@ -34,5 +34,11 @@ describe AridCache::CacheProxy::Utilities do
       @result.size.should == 1
       @result.first.should == @user.companies.reverse[1]
     end
+    
+    it "should not fail when ids is empty" do
+      lambda {
+        AridCache::CacheProxy::Utilities.find_all_by_id(Company, []).inspect  
+      }.should query(0)
+    end
   end
 end

data/spec/spec_helper.rb

@@ -32,6 +32,8 @@ RSpec.configure do |config|
   end
 
   config.before(:each) do
+    AridCache.store.delete! # so no options get stored and interfere with other tests
+    Rails.cache.respond_to?(:clear) ? Rails.cache.clear : Rails.cache.delete_matched(/.*/)
     Sham.reset(:before_each)
     full_example_description = "#{self.class.description} #{@method_name}"
     RAILS_DEFAULT_LOGGER.info("\n\n#{full_example_description}\n#{'-' * (full_example_description.length)}")

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: arid_cache
 version: !ruby/object:Gem::Version 
-  hash: 3
+  hash: 1
   prerelease: 
   segments: 
   - 1
   - 4
-  - 2
-  version: 1.4.2
+  - 3
+  version: 1.4.3
 platform: ruby
 authors: 
 - Karl Varga