Fix invalid arguments passed to `find_by`.
This commit is contained in:
parent
7b7432990a
commit
00a9cb1752
|
@ -3,7 +3,7 @@ module Jobs
|
|||
sidekiq_options queue: 'low'
|
||||
|
||||
def execute(args)
|
||||
user = User.find_by(args[:user_id])
|
||||
user = User.find_by(id: args[:user_id])
|
||||
raise Discourse::InvalidParameters.new(:user_id) if user.nil?
|
||||
|
||||
PendingAssignsReminder.new.remind(user)
|
||||
|
|
|
@ -0,0 +1,11 @@
|
|||
require 'rails_helper'
|
||||
|
||||
RSpec.describe Jobs::RemindUser do
|
||||
describe '#execute' do
|
||||
it "should raise the right error when user_id is invalid" do
|
||||
expect do
|
||||
described_class.new.execute({})
|
||||
end.to raise_error(Discourse::InvalidParameters)
|
||||
end
|
||||
end
|
||||
end
|
Loading…
Reference in New Issue