summaryrefslogtreecommitdiff
path: root/spec/models/ci/trigger_spec.rb
diff options
context:
space:
mode:
authorFabio Pitino <fpitino@gitlab.com>2019-06-25 12:41:11 +0100
committerFabio Pitino <fpitino@gitlab.com>2019-07-15 15:05:22 +0200
commit42d99460d3bbae88fdd9d8415970ecbfffc85a01 (patch)
tree0539b3150651f2a18567a58cf9b7a7aa87abf46d /spec/models/ci/trigger_spec.rb
parent0e8af2525f16d871510c24e6e15f1bc80f133edd (diff)
downloadgitlab-ce-42d99460d3bbae88fdd9d8415970ecbfffc85a01.tar.gz
Allow use of legacy triggers with feature flag
Keep feature flag disabled by default and turn off all functionality related to legacy triggers. * Block legacy triggers from creating pipeline * Highlight legacy triggers to be invalid via the UI * Make legacy triggers invalid in the model
Diffstat (limited to 'spec/models/ci/trigger_spec.rb')
-rw-r--r--spec/models/ci/trigger_spec.rb20
1 files changed, 16 insertions, 4 deletions
diff --git a/spec/models/ci/trigger_spec.rb b/spec/models/ci/trigger_spec.rb
index fde8375f2a5..5b5d6f51b33 100644
--- a/spec/models/ci/trigger_spec.rb
+++ b/spec/models/ci/trigger_spec.rb
@@ -54,19 +54,31 @@ describe Ci::Trigger do
end
describe '#can_access_project?' do
+ let(:owner) { create(:user) }
let(:trigger) { create(:ci_trigger, owner: owner, project: project) }
context 'when owner is blank' do
- let(:owner) { nil }
+ before do
+ stub_feature_flags(use_legacy_pipeline_triggers: false)
+ trigger.update_attribute(:owner, nil)
+ end
subject { trigger.can_access_project? }
- it { is_expected.to eq(true) }
+ it { is_expected.to eq(false) }
+
+ context 'when :use_legacy_pipeline_triggers feature flag is enabled' do
+ before do
+ stub_feature_flags(use_legacy_pipeline_triggers: true)
+ end
+
+ subject { trigger.can_access_project? }
+
+ it { is_expected.to eq(true) }
+ end
end
context 'when owner is set' do
- let(:owner) { create(:user) }
-
subject { trigger.can_access_project? }
context 'and is member of the project' do