netshade-cache-money 0.2.5.3 → 0.2.5.4

data/lib/cash/accessor.rb

@@ -1,3 +1,5 @@
+require 'digest/md5'
+
 module Cash
   module Accessor
     def self.included(a_module)
@@ -8,6 +10,9 @@ module Cash
     end
 
     module ClassMethods
+
+      MAX_KEY_SIZE = 250
+      
       def fetch(keys, options = {}, &block)
         case keys
         when Array
@@ -66,7 +71,11 @@ module Cash
       end
 
       def cache_key(key)
-        "#{name}:#{cache_config.version}/#{key.to_s.gsub(' ', '+')}"
+        s = "#{name}:#{cache_config.version}/#{key.to_s.gsub(' ', '+')}"
+        if s.size >= MAX_KEY_SIZE
+          s = Digest::MD5.hexdigest(s)
+        end
+        s
       end
     end
 

data/spec/cash/accessor_spec.rb

@@ -16,6 +16,14 @@ module Cash
             Story.fetch("yabba").should == "dabba"
           end
         end
+
+        describe "when the key size is very large" do
+          it "should hash the key to fit" do
+            k = "a" * 1024
+            Story.set(k, "foo")
+            Story.fetch(k).should == "foo"
+          end
+        end
       end
 
       describe '#fetch([...])', :shared => true do

data/spec/cash/active_record_spec.rb

@@ -34,6 +34,17 @@ module Cash
                 Story.find(story.id, nil).should == story
               end
             end
+
+            describe "#find(id,id,id..)" do
+              it "accepts a larger than normal set of ids" do
+                st = []
+                1.upto(250) do |i|
+                  st << Story.create!
+                end
+                ids = st.collect(&:id)
+                Story.find(ids).should == st
+              end
+            end
           end
 
           describe 'when given nonexistent ids' do
@@ -73,6 +84,17 @@ module Cash
             end
           end
 
+          describe "when given a large array of ids" do
+            it "#finds all objects with those ids" do
+              st = []
+              1.upto(250) do |i|
+                st << Story.create!
+              end
+              ids = st.collect(&:id)
+              Story.find(:all, :conditions => { :id => ids }).should == st              
+            end
+          end
+
           describe '#find([])' do
             it 'returns the empty array' do
               Story.find([]).should == []

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: netshade-cache-money
 version: !ruby/object:Gem::Version 
-  version: 0.2.5.3
+  version: 0.2.5.4
 platform: ruby
 authors: 
 - Nick Kallen