Merge pull request #9406 from rails/3-2-stable-ruby-2

Rails 3.2.x is now compatible with Ruby 2.0.0

.travis.yml

@@ -13,9 +13,6 @@ env:
   - "GEM=ar:mysql2"
   - "GEM=ar:sqlite3"
   - "GEM=ar:postgresql"
-matrix:
-  allow_failures:
-    - rvm: 2.0.0
 notifications:
   email: false
   irc:

actionpack/lib/action_controller/metal/compatibility.rb

@@ -58,7 +58,8 @@ def _handle_method_missing
     end
 
     def method_for_action(action_name)
-      super || (respond_to?(:method_missing) && "_handle_method_missing")
+      super || ((self.class.public_method_defined?(:method_missing) ||
+        self.class.protected_method_defined?(:method_missing)) && "_handle_method_missing")
     end
   end
 end

actionpack/lib/action_controller/metal/hide_actions.rb

@@ -27,20 +27,14 @@ def hide_action(*args)
         self.hidden_actions = hidden_actions.dup.merge(args.map(&:to_s)).freeze
       end
 
-      def inherited(klass)
-        klass.class_eval { @visible_actions = {} }
-        super
-      end
-
       def visible_action?(action_name)
-        return @visible_actions[action_name] if @visible_actions.key?(action_name)
-        @visible_actions[action_name] = !hidden_actions.include?(action_name)
+        action_methods.include?(action_name)
       end
 
       # Overrides AbstractController::Base#action_methods to remove any methods
       # that are listed as hidden methods.
       def action_methods
-        @action_methods ||= Set.new(super.reject { |name| hidden_actions.include?(name) })
+        @action_methods ||= Set.new(super.reject { |name| hidden_actions.include?(name) }).freeze
       end
     end
   end

actionpack/test/template/html-scanner/sanitizer_test.rb

@@ -210,7 +210,7 @@ def test_should_not_fall_for_ridiculous_hack
 
   # TODO: Clean up
   def test_should_sanitize_attributes
-    assert_sanitized %(<SPAN title="'><script>alert()</script>">blah</SPAN>), %(<span title="'&gt;&lt;script&gt;alert()&lt;/script&gt;">blah</span>)
+    assert_sanitized %(<SPAN title="'><script>alert()</script>">blah</SPAN>), %(<span title="#{CGI.escapeHTML "'><script>alert()</script>"}">blah</span>)
   end
 
   def test_should_sanitize_illegal_style_properties

actionpack/test/template/template_test.rb

@@ -1,3 +1,4 @@
+# encoding: US-ASCII
 require "abstract_unit"
 require "logger"
 

activesupport/lib/active_support/core_ext/hash/indifferent_access.rb

@@ -1,7 +1,6 @@
 require 'active_support/hash_with_indifferent_access'
 
 class Hash
-
   # Returns an <tt>ActiveSupport::HashWithIndifferentAccess</tt> out of its receiver:
   #
   #   {:a => 1}.with_indifferent_access["a"] # => 1

activesupport/lib/active_support/core_ext/hash/slice.rb

@@ -13,7 +13,7 @@ class Hash
   #   valid_keys = [:mass, :velocity, :time]
   #   search(options.slice(*valid_keys))
   def slice(*keys)
-    keys = keys.map! { |key| convert_key(key) } if respond_to?(:convert_key)
+    keys = keys.map! { |key| convert_key(key) } if respond_to?(:convert_key, true)
     hash = self.class.new
     keys.each { |k| hash[k] = self[k] if has_key?(k) }
     hash
@@ -23,7 +23,7 @@ def slice(*keys)
   # Returns a hash contained the removed key/value pairs
   #   {:a => 1, :b => 2, :c => 3, :d => 4}.slice!(:a, :b) # => {:c => 3, :d => 4}
   def slice!(*keys)
-    keys = keys.map! { |key| convert_key(key) } if respond_to?(:convert_key)
+    keys = keys.map! { |key| convert_key(key) } if respond_to?(:convert_key, true)
     omit = slice(*self.keys - keys)
     hash = slice(*keys)
     replace(hash)

activesupport/lib/active_support/core_ext/kernel/reporting.rb

@@ -1,4 +1,6 @@
 require 'rbconfig'
+require 'stringio'
+
 module Kernel
   # Sets $VERBOSE to nil for the duration of the block and back to its original value afterwards.
   #

activesupport/lib/active_support/hash_with_indifferent_access.rb

@@ -6,7 +6,7 @@
 
 module ActiveSupport
   class HashWithIndifferentAccess < Hash
-    
+
     # Always returns true, so that <tt>Array#extract_options!</tt> finds members of this class.
     def extractable_options?
       true

activesupport/lib/active_support/testing/isolation.rb

@@ -12,8 +12,8 @@ def initialize(exception)
     end
 
     class ProxyTestResult
-      def initialize
-        @calls = []
+      def initialize(calls = [])
+        @calls = calls
       end
 
       def add_error(e)
@@ -27,6 +27,14 @@ def __replay__(result)
         end
       end
 
+      def marshal_dump
+        @calls
+      end
+
+      def marshal_load(calls)
+        initialize(calls)
+      end
+
       def method_missing(name, *args)
         @calls << [name, args]
       end

activesupport/test/caching_test.rb

@@ -690,7 +690,6 @@ def setup
       @data = @cache.instance_variable_get(:@data)
       @cache.clear
       @cache.silence!
-      @cache.logger = Logger.new("/dev/null")
     end
 
     include CacheStoreBehavior

activesupport/test/ordered_hash_test.rb

@@ -221,7 +221,6 @@ def test_alternate_initialization_with_array
     alternate = ActiveSupport::OrderedHash[ [
       [1, 2],
       [3, 4],
-      "bad key value pair",
       [ 'missing value' ]
     ]]
 

activesupport/test/test_case_test.rb

@@ -16,6 +16,9 @@ def puke(klass, name, e)
       def options
         nil
       end
+
+      def record(*args)
+      end
     end
 
     if defined?(MiniTest::Assertions) && TestCase < MiniTest::Assertions

railties/lib/rails/generators/rails/plugin_new/plugin_new_generator.rb

@@ -214,6 +214,18 @@ def finish_template
 
       public_task :apply_rails_template, :run_bundle
 
+      def name
+        @name ||= begin
+          # same as ActiveSupport::Inflector#underscore except not replacing '-'
+          underscored = original_name.dup
+          underscored.gsub!(/([A-Z]+)([A-Z][a-z])/,'\1_\2')
+          underscored.gsub!(/([a-z\d])([A-Z])/,'\1_\2')
+          underscored.downcase!
+
+          underscored
+        end
+      end
+
     protected
 
       def app_templates_dir
@@ -254,18 +266,6 @@ def original_name
         @original_name ||= File.basename(destination_root)
       end
 
-      def name
-        @name ||= begin
-          # same as ActiveSupport::Inflector#underscore except not replacing '-'
-          underscored = original_name.dup
-          underscored.gsub!(/([A-Z]+)([A-Z][a-z])/,'\1_\2')
-          underscored.gsub!(/([a-z\d])([A-Z])/,'\1_\2')
-          underscored.downcase!
-
-          underscored
-        end
-      end
-
       def camelized
         @camelized ||= name.gsub(/\W/, '_').squeeze('_').camelize
       end

railties/test/application/assets_test.rb

@@ -2,6 +2,7 @@
 require 'isolation/abstract_unit'
 require 'active_support/core_ext/kernel/reporting'
 require 'rack/test'
+require 'yaml'
 
 module ApplicationTests
   class AssetsTest < Test::Unit::TestCase

railties/test/application/initializers/active_record_test.rb

@@ -23,10 +23,17 @@ def teardown
       boot_rails
       simple_controller
 
-      get '/foo'
-      assert last_response.body.include?("We're sorry, but something went wrong (500)")
+      # ActiveSupport::LogSubscriber.flush_all! in lib/rails/rack/logger.rb blew up in Ruby 2.0
+      # because it tries to open the database. This behavior doesn't happen in Ruby 1.9.3.
+      # However, regardless, the server blew up.
+      if RUBY_VERSION >= '2.0.0'
+        assert_raises (Errno::ENOENT) { get '/foo' }
+      else
+        get '/foo'
+        assert last_response.body.include?("We're sorry, but something went wrong (500)")
+      end
     end
-    
+
     test "uses DATABASE_URL env var when config/database.yml doesn't exist" do
       database_path = "/db/foo.sqlite3"
       FileUtils.rm_rf("#{app_path}/config/database.yml")
@@ -35,7 +42,7 @@ def teardown
 
       get '/foo'
       assert_equal 'foo', last_response.body
-      
+
       # clean up
       FileUtils.rm("#{app_path}/#{database_path}")
     end

railties/test/application/route_inspect_test.rb

@@ -18,7 +18,7 @@ def setup
 
     def test_displaying_routes_for_engines
       engine = Class.new(Rails::Engine) do
-        def self.to_s
+        def self.inspect
           "Blog::Engine"
         end
       end
@@ -136,7 +136,7 @@ def test_rake_routes_shows_route_with_rack_app
 
     def test_rake_routes_shows_route_with_rack_app_nested_with_dynamic_constraints
       constraint = Class.new do
-        def to_s
+        def inspect
           "( my custom constraint )"
         end
       end