stubbing methods on nil is bad; change mock => double to fix deprecation warnings

Author: cbeer

spec/controllers/catalog_controller_spec.rb

@@ -277,6 +277,7 @@ def assigns_response
 
     describe "@document" do
       before do
+        @mock_response = double()
         @mock_response.stub(:docs => [{ :id => 'my_fake_doc' }])
         @mock_document = double()
         controller.stub(:find => @mock_response, 
@@ -303,6 +304,7 @@ def export_as_mock
         end
       end
       before do
+        @mock_response = double()
         @mock_response.stub(:docs => [{ :id => 'my_fake_doc' }])
         @mock_document = double()
         controller.stub(:find => @mock_response, 

spec/views/catalog/_thumbnail_default.erb_spec.rb

@@ -17,7 +17,7 @@
   end
 
   before do
-    assign :response, mock(:params => {})
+    assign :response, double(:params => {})
     view.stub(:render_grouped_response?).and_return false
     view.stub(:blacklight_config).and_return(blacklight_config)
   end