Cop idea: Replace ActiveJob::TestHelper with activejob matchers

[Darhazer]

I'm working on a code base that tests ActiveJobs with assert_enqued_job, assert_enqueued_with, etc, included by the ActiveJob::TestHelper, which is written with minitest. rpsec-rails however provide have_enqueued_job matcher

[backus]

I'm open to PRs for this. I have mentioned this in other PRs but I don't plan on implementing rails specific cops. It would also be nice if these things checked if rails cops were enabled

[Darhazer]

@backus what about capybara-specific checks? I've ran in some wrong usages of capybara matchers

[backus]

As long as capybara support don't get in the way of normal rspec users I guess I'm fine with it. I don't use capybara so I wont be great as a primary reviewer for these.

[Darhazer]

I also seen specs like expect(ActionMailer::MailDeliveryJob).to have_been_enqueued.with(Mailer, :something, :deliver_now) instead of using enqueued_mail matcher

[pirj]

Did that spec fail, @Darhazer ?

[Darhazer]

Nope, but it's overly expressive and leaks internals :)