Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Look into incorporating ActiveSupport::Testing::NotificationsAssertions into our tests #2971

Open
kaylareopelle opened this issue Nov 22, 2024 · 1 comment
Open

Look into incorporating ActiveSupport::Testing::NotificationsAssertions into our tests #2971

kaylareopelle opened this issue Nov 22, 2024 · 1 comment
Labels
needs review rails Issues related to the Ruby on Rails framework

Comments

@kaylareopelle
Copy link
Contributor

kaylareopelle commented Nov 22, 2024 •
edited
Loading

rails/rails#53065 introduces new assertions to simplify testing ActiveSupport::Notifications. We should look into these assertions to see if they can benefit our testing.

Here is a PR where they incorporated these helpers into Rails tests: rails/rails#53700
@kaylareopelle kaylareopelle added the rails Issues related to the Ruby on Rails framework label Nov 22, 2024
@workato-integration
Copy link

https://new-relic.atlassian.net/browse/NR-342999

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
needs review rails Issues related to the Ruby on Rails framework
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kaylareopelle @chynesNR