Skip to content

Commit 862ab91

Browse files
authored
Merge pull request rails#53825 from larouxn/notification_assertions_railties
Migrate applicable `railties` tests to use `NotificationAssertions`
2 parents 65a2a6b + 1947f3f commit 862ab91

File tree

1 file changed

+4
-8
lines changed

1 file changed

+4
-8
lines changed

railties/test/application/initializers/notifications_test.rb

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -45,14 +45,10 @@ def wait
4545
test "rails load_config_initializer event is instrumented" do
4646
app_file "config/initializers/foo.rb", ""
4747

48-
events = []
49-
callback = ->(*_) { events << _ }
50-
ActiveSupport::Notifications.subscribed(callback, "load_config_initializer.railties") do
51-
app
52-
end
53-
54-
assert_equal %w[load_config_initializer.railties], events.map(&:first)
55-
assert_includes events.first.last[:initializer], "config/initializers/foo.rb"
48+
event = capture_notifications("load_config_initializer.railties") { app }.first
49+
50+
assert_equal "load_config_initializer.railties", event.name
51+
assert_match "config/initializers/foo.rb", event.payload[:initializer]
5652
end
5753
end
5854
end

0 commit comments

Comments
 (0)