New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix duplicate test method names #33684
Conversation
r? @kaspth (@rails-bot has picked a reviewer for you, use r? to override) |
@@ -1197,7 +1185,7 @@ def test_assert_performed_jobs_without_block_with_except_and_queue_options | |||
assert_performed_jobs 1, except: HelloJob, queue: :other_queue | |||
end | |||
|
|||
def test_assert_performed_jobs_with_except_and_queue_options_failuree | |||
def test_assert_performed_jobs_with_except_and_queue_options_failure |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you please just change the name to test_assert_performed_jobs_without_block_with_except_and_queue_options_failure
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah.... yes. Fixed.
@utilum Good catch. We should test |
This patch corrects a duplicate method name introduced in rails#33635. Also fixes typo in method names.
f8a7729
to
cade790
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great! Thanks for catching this! ❤️
This patch fixes a duplicate test method name in #33635.
Also fixes typo in method names.