Improve instrumentation tests

Don't build AS::Notifications::Event manually, similar to 95b6fbd00f
Also assert that options get passed.

actionpack/test/controller/caching_test.rb

@@ -257,8 +257,8 @@ Ciao
   def test_fragment_cache_instrumentation
     payload = nil
 
-    subscriber = proc do |_, _, _, _, event_payload|
-      payload = event_payload
+    subscriber = proc do |event|
+      payload = event.payload
     end
 
     ActiveSupport::Notifications.subscribed(subscriber, "read_fragment.action_controller") do

actionpack/test/controller/redirect_test.rb

@@ -604,8 +604,7 @@ class RedirectTest < ActionController::TestCase
   def test_redirect_to_instrumentation
     payload = nil
 
-    subscriber = proc do |*args|
-      event = ActiveSupport::Notifications::Event.new(*args)
+    subscriber = proc do |event|
       payload = event.payload
     end
 

actionpack/test/controller/send_file_test.rb

@@ -208,10 +208,10 @@ class SendFileTest < ActionController::TestCase
   end
 
   def test_send_file_instrumentation
+    @controller.options = { disposition: :inline }
     payload = nil
 
-    subscriber = proc do |*args|
-      event = ActiveSupport::Notifications::Event.new(*args)
+    subscriber = proc do |event|
       payload = event.payload
     end
 
@@ -220,13 +220,14 @@ class SendFileTest < ActionController::TestCase
     end
 
     assert_equal __FILE__, payload[:path]
+    assert_equal :inline, payload[:disposition]
   end
 
   def test_send_data_instrumentation
+    @controller.options = { content_type: "application/x-ruby" }
     payload = nil
 
-    subscriber = proc do |*args|
-      event = ActiveSupport::Notifications::Event.new(*args)
+    subscriber = proc do |event|
       payload = event.payload
     end
 
@@ -234,7 +235,7 @@ class SendFileTest < ActionController::TestCase
       process("data")
     end
 
-    assert_equal({}, payload)
+    assert_equal("application/x-ruby", payload[:content_type])
   end
 
   %w(file data).each do |method|