From bf83e2607ada039501b161bc8f645a3afb9c3835 Mon Sep 17 00:00:00 2001 From: Brian Ryckbost Date: Fri, 13 Jan 2012 17:22:34 -0500 Subject: [PATCH] Sprucing up a few more specs --- lib/delayed/backend/shared_spec.rb | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/lib/delayed/backend/shared_spec.rb b/lib/delayed/backend/shared_spec.rb index d541ab10b..c83b34cff 100644 --- a/lib/delayed/backend/shared_spec.rb +++ b/lib/delayed/backend/shared_spec.rb @@ -237,8 +237,8 @@ def create_job(opts = {}) end it "should be the instance method that will be called if its a performable method object" do - @job = Story.create(:text => "...").delay.save - @job.name.should == 'Story#save' + job = Story.create(:text => "...").delay.save + job.name.should == 'Story#save' end it "should parse from handler on deserialization error" do @@ -415,10 +415,10 @@ def create_job(opts = {}) begin old_max_run_time = Delayed::Worker.max_run_time Delayed::Worker.max_run_time = 1.second - @job = Delayed::Job.create :payload_object => LongRunningJob.new - worker.run(@job) - @job.reload.last_error.should =~ /expired/ - @job.attempts.should == 1 + job = Delayed::Job.create :payload_object => LongRunningJob.new + worker.run(job) + job.reload.last_error.should =~ /expired/ + job.attempts.should == 1 ensure Delayed::Worker.max_run_time = old_max_run_time end @@ -467,11 +467,11 @@ def create_job(opts = {}) end it 'should re-schedule with handler provided time if present' do - @job = Delayed::Job.enqueue(CustomRescheduleJob.new(99.minutes)) - worker.run(@job) - @job.reload + job = Delayed::Job.enqueue(CustomRescheduleJob.new(99.minutes)) + worker.run(job) + job.reload - (Delayed::Job.db_time_now + 99.minutes - @job.run_at).abs.should < 1 + (Delayed::Job.db_time_now + 99.minutes - job.run_at).abs.should < 1 end it "should not fail when the triggered error doesn't have a message" do