redis-cache 1.0.0.rc1 → 1.0.0.rc2

data/History.txt

@@ -1,3 +1,7 @@
+=== 1.0.0.rc2 2011-07-07
+
+  * Fixed bug: #read_multi with non-existing keys raises error.
+
 === 1.0.0.rc1 2011-06-08
 
   * Initial release.

data/lib/active_support/cache/redis_store.rb

@@ -51,14 +51,14 @@ module ActiveSupport
         values = {}
         keys_to_names.keys.zip(raw_values).each do |key, value|
           entry = deserialize_entry(value)
-          values[keys_to_names[key]] = entry.value unless entry.expired?
+          values[keys_to_names[key]] = entry.value unless entry.nil? || entry.expired?
         end
         values
       end
 
       protected
         def read_entry(key, options)
-          deserialize_entry @redis.get(key)
+          deserialize_entry(@redis.get(key))
         end
 
         def write_entry(key, entry, options)

data/spec/redis_store_spec.rb

@@ -52,6 +52,15 @@ describe "ActiveSupport::Cache::RedisStore" do
         store.read("abc").should == "test"
       end
     end
+
+    context "without any keys, #read_multi" do
+      before(:each) { store.clear }
+      subject { lambda { store.read_multi("a1", "a2", "a3") } }
+
+      it "should not raise any errors" do
+        should_not raise_error
+      end
+    end
     
     context "with keys 'a1', 'a2', 'a3', #read_multi('a1', 'a2', 'a3')" do
       before(:each) do

metadata

@@ -2,7 +2,7 @@
 name: redis-cache
 version: !ruby/object:Gem::Version 
   prerelease: 6
-  version: 1.0.0.rc1
+  version: 1.0.0.rc2
 platform: ruby
 authors: 
 - Anton Ageev
@@ -10,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-06-08 00:00:00 Z
+date: 2011-07-07 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: activesupport
@@ -71,8 +71,8 @@ files:
 - Rakefile
 - lib/redis-cache.rb
 - lib/active_support/cache/redis_store.rb
-- spec/spec_helper.rb
 - spec/redis_store_spec.rb
+- spec/spec_helper.rb
 homepage: https://github.com/antage/redis-cache
 licenses: []
 
@@ -101,5 +101,5 @@ signing_key:
 specification_version: 3
 summary: ActiveSupport cache adapter for Redis
 test_files: 
-- spec/spec_helper.rb
 - spec/redis_store_spec.rb
+- spec/spec_helper.rb