Use klass.method_defined? instead of .methods.include? because it works the same way in Ruby 1.8 & 1.9+ (plus is faster).

Author: sarahmei

lib/rspec/rails/view_spec_methods.rb

@@ -9,8 +9,7 @@ module ViewSpecMethods
       # When class is `::ActionView::TestCase::TestController`, these methods
       # are exposed in view specs on the `controller` object.
       def add_to(klass)
-        return if klass.instance_methods.include?(:extra_params) && klass.instance_methods.include?(:extra_params=)
-        return if klass.instance_methods.include?("extra_params") && klass.instance_methods.include?("extra_params=")
+        return if klass.method_defined?(:extra_params) && klass.method_defined?(:extra_params=)
 
         klass.module_exec do
           # Set any extra parameters that rendering a URL for this view
@@ -46,10 +45,9 @@ def extra_params
 
       # Removes methods `extra_params=` and `extra_params` from the indicated class.
       def remove_from(klass)
-        methods = klass.instance_methods
         klass.module_exec do
-          undef extra_params= if methods.include?(:extra_params=) || methods.include?("extra_params=")
-          undef extra_params  if methods.include?(:extra_params)  || methods.include?("extra_params")
+          undef extra_params= if klass.method_defined?(:extra_params=)
+          undef extra_params  if klass.method_defined?(:extra_params)
         end
       end
     end