1.9 seems to have a bug involving cloned classes and super. Fix it by not...

1.9 seems to have a bug involving cloned classes and super. Fix it by not cloning (and instead creating classes on demand). The 1.9 bug should be investigated.

actionpack/test/controller/routing_test.rb

@@ -52,11 +52,17 @@ def test_route_generation_allows_passing_non_string_values_to_generated_helper
 end
 
 class MockController
-  def url_for(options)
-    options[:protocol] ||= "http"
-    options[:host] ||= "test.host"
+  def self.build(helpers)
+    Class.new do
+      def url_for(options)
+        options[:protocol] ||= "http"
+        options[:host] ||= "test.host"
 
-    super(options)
+        super(options)
+      end
+
+      include helpers
+    end
   end
 end
 
@@ -250,9 +256,7 @@ def test_optimised_named_route_with_host
   end
 
   def setup_for_named_route
-    inst = MockController.clone.new
-    inst.class.send(:include, rs.url_helpers)
-    inst
+    MockController.build(rs.url_helpers).new
   end
 
   def test_named_route_without_hash
@@ -741,7 +745,7 @@ def setup_named_route_test
       map.users '/admin/users', :controller => 'admin/users', :action => 'index'
     end
 
-    MockController.clone.new.tap { |inst| inst.class.send(:include, set.url_helpers)}
+    MockController.build(set.url_helpers).new
   end
 
   def test_named_route_hash_access_method