diff options
author | Kamil Trzcinski <ayufan@ayufan.eu> | 2017-04-07 16:02:13 +0200 |
---|---|---|
committer | Kamil Trzcinski <ayufan@ayufan.eu> | 2017-04-07 16:19:11 +0200 |
commit | 21a7aed9a7330382300bc93b8927609e65ee6390 (patch) | |
tree | e5aec1003032a91e43e4eb8af61ec65e47e2ce7e /spec/workers | |
parent | 03088552549ed1631bb16c1bf3d0bef3613ec793 (diff) | |
download | gitlab-ce-21a7aed9a7330382300bc93b8927609e65ee6390.tar.gz |
Update tests to cover trigger schedule
Diffstat (limited to 'spec/workers')
-rw-r--r-- | spec/workers/trigger_schedule_worker_spec.rb | 34 |
1 files changed, 23 insertions, 11 deletions
diff --git a/spec/workers/trigger_schedule_worker_spec.rb b/spec/workers/trigger_schedule_worker_spec.rb index 151e1c2f7b9..fdc638d9070 100644 --- a/spec/workers/trigger_schedule_worker_spec.rb +++ b/spec/workers/trigger_schedule_worker_spec.rb @@ -8,24 +8,39 @@ describe TriggerScheduleWorker do end context 'when there is a scheduled trigger within next_run_at' do + let(:next_run_at) { 2.days.ago } + + let!(:trigger_schedule) do + create(:ci_trigger_schedule, :nightly) + end + before do - trigger_schedule = create(:ci_trigger_schedule, :nightly) - time_future = Time.now + 10.days - allow(Time).to receive(:now).and_return(time_future) - @next_time = Gitlab::Ci::CronParser.new(trigger_schedule.cron, trigger_schedule.cron_timezone).next_time_from(time_future) + trigger_schedule.update_column(:next_run_at, next_run_at) end it 'creates a new trigger request' do - expect { worker.perform }.to change { Ci::TriggerRequest.count }.by(1) + expect { worker.perform }.to change { Ci::TriggerRequest.count } end it 'creates a new pipeline' do - expect { worker.perform }.to change { Ci::Pipeline.count }.by(1) + expect { worker.perform }.to change { Ci::Pipeline.count } expect(Ci::Pipeline.last).to be_pending end it 'updates next_run_at' do - expect { worker.perform }.to change { Ci::TriggerSchedule.last.next_run_at }.to(@next_time) + worker.perform + + expect(trigger_schedule.reload.next_run_at).not_to eq(next_run_at) + end + + context 'inactive schedule' do + before do + trigger_schedule.update(active: false) + end + + it 'does not create a new trigger' do + expect { worker.perform }.not_to change { Ci::TriggerRequest.count } + end end end @@ -42,10 +57,7 @@ describe TriggerScheduleWorker do end context 'when next_run_at is nil' do - before do - trigger_schedule = create(:ci_trigger_schedule, :nightly) - trigger_schedule.update_attribute(:next_run_at, nil) - end + let!(:trigger_schedule) { create(:ci_trigger_schedule, :nightly, next_run_at: nil) } it 'does not create a new pipeline' do expect { worker.perform }.not_to change { Ci::Pipeline.count } |