summaryrefslogtreecommitdiff
path: root/spec/models/ci/build_spec.rb
diff options
context:
space:
mode:
authorKamil Trzciński <ayufan@ayufan.eu>2019-08-02 13:28:59 +0200
committerKamil Trzciński <ayufan@ayufan.eu>2019-08-02 13:29:04 +0200
commit181b9b3e7328070f6846c9547d166ba0128e415c (patch)
tree382ad3ee9b0d4fb68275247cb098d8cc618dec21 /spec/models/ci/build_spec.rb
parentdb104aaf9236050ea5fd921efab664892a6b1880 (diff)
downloadgitlab-ce-181b9b3e7328070f6846c9547d166ba0128e415c.tar.gz
Respect needs for artifactsrespect-needs-on-artifacts
When `needs:` is defined, the value of it is not respected when returning a list of artifacts to the runner from the job.
Diffstat (limited to 'spec/models/ci/build_spec.rb')
-rw-r--r--spec/models/ci/build_spec.rb17
1 files changed, 15 insertions, 2 deletions
diff --git a/spec/models/ci/build_spec.rb b/spec/models/ci/build_spec.rb
index 8768e914284..0387073cffb 100644
--- a/spec/models/ci/build_spec.rb
+++ b/spec/models/ci/build_spec.rb
@@ -630,12 +630,17 @@ describe Ci::Build do
create(:ci_build,
pipeline: pipeline, name: 'final',
stage_idx: 3, stage: 'deploy', options: {
- dependencies: dependencies,
- needs: needs
+ dependencies: dependencies
}
)
end
+ before do
+ needs.to_a.each do |need|
+ create(:ci_build_need, build: final, name: need)
+ end
+ end
+
subject { final.dependencies }
context 'when depedencies are defined' do
@@ -648,6 +653,14 @@ describe Ci::Build do
let(:needs) { %w(build rspec staging) }
it { is_expected.to contain_exactly(build, rspec_test, staging) }
+
+ context 'when ci_dag_support is disabled' do
+ before do
+ stub_feature_flags(ci_dag_support: false)
+ end
+
+ it { is_expected.to contain_exactly(build, rspec_test, rubocop_test, staging) }
+ end
end
context 'when needs and dependencies are defined' do