From ef61829b586ea2581d37d7c3ca0db60e01c8aec6 Mon Sep 17 00:00:00 2001 From: Shinya Maeda Date: Fri, 26 May 2017 21:41:32 +0900 Subject: [PATCH] Add a test for deletion of importing_or_inactive? --- spec/models/ci/pipeline_schedule_spec.rb | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/spec/models/ci/pipeline_schedule_spec.rb b/spec/models/ci/pipeline_schedule_spec.rb index 822b98c5f6c..b00e7a73571 100644 --- a/spec/models/ci/pipeline_schedule_spec.rb +++ b/spec/models/ci/pipeline_schedule_spec.rb @@ -25,6 +25,14 @@ describe Ci::PipelineSchedule, models: true do expect(pipeline_schedule).not_to be_valid end + + context 'when active is false' do + it 'does not allow nullified ref' do + pipeline_schedule = build(:ci_pipeline_schedule, :inactive, ref: nil) + + expect(pipeline_schedule).not_to be_valid + end + end end describe '#set_next_run_at' do