make sure to populate cache on finds by non-primary-key attributes

lib/cash/query/abstract.rb

@@ -184,7 +184,7 @@ def deserialize_objects(objects)
 
       def find_from_keys(*missing_keys)
         missing_ids = Array(missing_keys).flatten.collect { |key| key.split('/')[2].to_i }
-        find_from_ids_without_cache(missing_ids, @options1)
+        find_from_ids_without_cache(missing_ids, {})
       end
     end
   end

spec/cash/finders_spec.rb

@@ -339,10 +339,20 @@ module Cash
         end
 
         describe '#find_by_attr' do
+          before(:each) do
+            Story.find_by_title(@story.title)  # populates cache for title with [@story.id]
+          end
+
           it 'populates the cache' do
-            Story.find_by_title(@story.title)
             Story.fetch("title/#{@story.title}").should == [@story.id]
           end
+
+          it 'populates the cache when finding by non-primary-key attribute' do
+            Story.find_by_title(@story.title)  # populates cache for id with record
+
+            mock(Story.connection).execute.never
+            Story.find_by_title(@story.title).id.should == @story.id  # should hit cache only
+          end
         end
 
         describe '#find(:all, :conditions => ...)' do