Merge remote-tracking branch 'upstream/pull/5180'

[rails.git] / test / integration / cors_test.rb

diff --git a/test/integration/cors_test.rb b/test/integration/cors_test.rb
index fdecfc256c4eb666de1e244d090365da36555ed9..c35f73d349d7c90f415d73c6e7935feabafeb8c2 100644 (file)
--- a/test/integration/cors_test.rb
+++ b/test/integration/cors_test.rb
@@ -1,32 +1,27 @@
 require "test_helper"
 
 class CORSTest < ActionDispatch::IntegrationTest
 require "test_helper"
 
 class CORSTest < ActionDispatch::IntegrationTest

-  # Rails 4 adds a built-in `options` method. When we upgrade, we can remove
-  # this definition.
-  unless instance_methods.include?(:options)
-    def options(*args)
-      reset! unless integration_session
-      @html_document = nil
-      integration_session.send(:process, :options, *args).tap do
-        copy_session_variables!
-      end
-    end
-  end
-

   def test_api_routes_allow_cross_origin_requests
   def test_api_routes_allow_cross_origin_requests

-    options "/api/capabilities", nil,
-            "HTTP_ORIGIN" => "http://www.example.com",
-            "HTTP_ACCESS_CONTROL_REQUEST_METHOD" => "GET"
+    process :options, "/api/capabilities", :headers => {
+      "HTTP_ORIGIN" => "http://www.example.com",
+      "HTTP_ACCESS_CONTROL_REQUEST_METHOD" => "GET"
+    }

 
     assert_response :success
 
     assert_response :success

-    assert_equal "http://www.example.com", response.headers["Access-Control-Allow-Origin"]
+    assert_equal "*", response.headers["Access-Control-Allow-Origin"]
+    assert_nil response.media_type
+    assert_equal "", response.body

   end
 
   def test_non_api_routes_dont_allow_cross_origin_requests
   end
 
   def test_non_api_routes_dont_allow_cross_origin_requests

-    assert_raises ActionController::RoutingError do
-      options "/", nil,
-              "HTTP_ORIGIN" => "http://www.example.com",
-              "HTTP_ACCESS_CONTROL_REQUEST_METHOD" => "GET"
-    end
+    process :options, "/", :headers => {
+      "HTTP_ORIGIN" => "http://www.example.com",
+      "HTTP_ACCESS_CONTROL_REQUEST_METHOD" => "GET"
+    }
+
+    assert_response :success
+    assert_nil response.headers["Access-Control-Allow-Origin"]
+    assert_nil response.media_type
+    assert_equal "", response.body

   end
 end
   end
 end