Merge pull request #9 from TinderBox/memory-store-get

memory_store: avoid nil array access by checking key exists. Returns nil if no key is found.

lib/api_cache/memory_store.rb

@@ -13,7 +13,7 @@ def set(key, value)
     end
 
     def get(key)
-      data = @cache[key][1]
+      data = exists?(key) ? @cache[key][1] : nil
       APICache.logger.debug("cache: #{data.nil? ? "miss" : "hit"} (#{key})")
       data
     end

spec/memory_store_spec.rb

@@ -0,0 +1,18 @@
+require 'spec_helper'
+
+describe APICache::MemoryStore do
+  before :each do
+    @store = APICache::MemoryStore.new
+  end
+
+  it "should set" do
+    @store.exists?('foo').should be_false
+    @store.set('foo', 'bar')
+    @store.exists?('foo').should be_true
+	@store.get('foo').should == 'bar'
+  end
+
+  it "should return nil if not found" do
+    @store.get('nothing').should be_nil
+  end
+end