diff options
Diffstat (limited to 'spec/lib/gitlab/ci/config/entry/bridge_spec.rb')
-rw-r--r-- | spec/lib/gitlab/ci/config/entry/bridge_spec.rb | 227 |
1 files changed, 227 insertions, 0 deletions
diff --git a/spec/lib/gitlab/ci/config/entry/bridge_spec.rb b/spec/lib/gitlab/ci/config/entry/bridge_spec.rb new file mode 100644 index 00000000000..07590556db8 --- /dev/null +++ b/spec/lib/gitlab/ci/config/entry/bridge_spec.rb @@ -0,0 +1,227 @@ +# frozen_string_literal: true + +require 'spec_helper' + +describe Gitlab::Ci::Config::Entry::Bridge do + subject { described_class.new(config, name: :my_bridge) } + + it_behaves_like 'with inheritable CI config' do + let(:inheritable_key) { 'default' } + let(:inheritable_class) { Gitlab::Ci::Config::Entry::Default } + + # These are entries defined in Default + # that we know that we don't want to inherit + # as they do not have sense in context of Bridge + let(:ignored_inheritable_columns) do + %i[before_script after_script image services cache interruptible timeout + retry tags artifacts] + end + end + + describe '.matching?' do + subject { described_class.matching?(name, config) } + + context 'when config is not a hash' do + let(:name) { :my_trigger } + let(:config) { 'string' } + + it { is_expected.to be_falsey } + end + + context 'when config is a regular job' do + let(:name) { :my_trigger } + let(:config) do + { script: 'ls -al' } + end + + it { is_expected.to be_falsey } + + context 'with rules' do + let(:config) do + { + script: 'ls -al', + rules: [{ if: '$VAR == "value"', when: 'always' }] + } + end + + it { is_expected.to be_falsey } + end + end + + context 'when config is a bridge job' do + let(:name) { :my_trigger } + let(:config) do + { trigger: 'other-project' } + end + + it { is_expected.to be_truthy } + + context 'with rules' do + let(:config) do + { + trigger: 'other-project', + rules: [{ if: '$VAR == "value"', when: 'always' }] + } + end + + it { is_expected.to be_truthy } + end + end + + context 'when config is a hidden job' do + let(:name) { '.my_trigger' } + let(:config) do + { trigger: 'other-project' } + end + + it { is_expected.to be_falsey } + end + end + + describe '.new' do + before do + subject.compose! + end + + let(:base_config) do + { + trigger: { project: 'some/project', branch: 'feature' }, + extends: '.some-key', + stage: 'deploy', + variables: { VARIABLE: '123' } + } + end + + context 'when trigger config is a non-empty string' do + let(:config) { { trigger: 'some/project' } } + + describe '#valid?' do + it { is_expected.to be_valid } + end + + describe '#value' do + it 'is returns a bridge job configuration' do + expect(subject.value).to eq(name: :my_bridge, + trigger: { project: 'some/project' }, + ignore: false, + stage: 'test', + only: { refs: %w[branches tags] }) + end + end + end + + context 'when bridge trigger is a hash' do + let(:config) do + { trigger: { project: 'some/project', branch: 'feature' } } + end + + describe '#valid?' do + it { is_expected.to be_valid } + end + + describe '#value' do + it 'is returns a bridge job configuration hash' do + expect(subject.value).to eq(name: :my_bridge, + trigger: { project: 'some/project', + branch: 'feature' }, + ignore: false, + stage: 'test', + only: { refs: %w[branches tags] }) + end + end + end + + context 'when bridge configuration contains trigger, when, extends, stage, only, except, and variables' do + let(:config) do + base_config.merge({ + when: 'always', + only: { variables: %w[$SOMEVARIABLE] }, + except: { refs: %w[feature] } + }) + end + + it { is_expected.to be_valid } + end + + context 'when bridge configuration uses rules' do + let(:config) { base_config.merge({ rules: [{ if: '$VAR == null', when: 'never' }] }) } + + it { is_expected.to be_valid } + end + + context 'when bridge configuration uses rules with job:when' do + let(:config) do + base_config.merge({ + when: 'always', + rules: [{ if: '$VAR == null', when: 'never' }] + }) + end + + it { is_expected.not_to be_valid } + end + + context 'when bridge configuration uses rules with only' do + let(:config) do + base_config.merge({ + only: { variables: %w[$SOMEVARIABLE] }, + rules: [{ if: '$VAR == null', when: 'never' }] + }) + end + + it { is_expected.not_to be_valid } + end + + context 'when bridge configuration uses rules with except' do + let(:config) do + base_config.merge({ + except: { refs: %w[feature] }, + rules: [{ if: '$VAR == null', when: 'never' }] + }) + end + + it { is_expected.not_to be_valid } + end + + context 'when bridge has only job needs' do + let(:config) do + { + needs: ['some_job'] + } + end + + describe '#valid?' do + it { is_expected.not_to be_valid } + end + end + + context 'when bridge config contains unknown keys' do + let(:config) { { unknown: 123 } } + + describe '#valid?' do + it { is_expected.not_to be_valid } + end + + describe '#errors' do + it 'is returns an error about unknown config key' do + expect(subject.errors.first) + .to match /config contains unknown keys: unknown/ + end + end + end + + context 'when bridge config contains build-specific attributes' do + let(:config) { { script: 'something' } } + + describe '#valid?' do + it { is_expected.not_to be_valid } + end + + describe '#errors' do + it 'returns an error message' do + expect(subject.errors.first) + .to match /contains unknown keys: script/ + end + end + end + end +end |