diff options
| author | Matija Čupić <matteeyah@gmail.com> | 2018-05-04 12:35:36 +0200 |
|---|---|---|
| committer | Matija Čupić <matteeyah@gmail.com> | 2018-05-04 12:35:36 +0200 |
| commit | c6e394bb8499b470b9d2a0c03c8ce7fd023fa08a (patch) | |
| tree | 8f3050e8d9a12d809c48ceda8eb0a411ad6259b7 | |
| parent | 980fb6fb26a360894bba7444d0020381ab825c8d (diff) | |
| download | gitlab-ce-c6e394bb8499b470b9d2a0c03c8ce7fd023fa08a.tar.gz | |
Improve variable comparison
| -rw-r--r-- | spec/lib/gitlab/ci/pipeline/chain/build_spec.rb | 3 | ||||
| -rw-r--r-- | spec/services/ci/create_pipeline_service_spec.rb | 5 |
2 files changed, 4 insertions, 4 deletions
diff --git a/spec/lib/gitlab/ci/pipeline/chain/build_spec.rb b/spec/lib/gitlab/ci/pipeline/chain/build_spec.rb index 17f15ac3b27..85d73e5c382 100644 --- a/spec/lib/gitlab/ci/pipeline/chain/build_spec.rb +++ b/spec/lib/gitlab/ci/pipeline/chain/build_spec.rb @@ -44,7 +44,8 @@ describe Gitlab::Ci::Pipeline::Chain::Build do expect(pipeline.tag).to be false expect(pipeline.user).to eq user expect(pipeline.project).to eq project - expect(pipeline.variables.size).to eq variables_attributes.count + expect(pipeline.variables.map { |var| var.slice(:key, :secret_value) }) + .to eq variables_attributes.map(&:with_indifferent_access) end it 'sets a valid config source' do diff --git a/spec/services/ci/create_pipeline_service_spec.rb b/spec/services/ci/create_pipeline_service_spec.rb index 24717898c33..9a0b6efd8a9 100644 --- a/spec/services/ci/create_pipeline_service_spec.rb +++ b/spec/services/ci/create_pipeline_service_spec.rb @@ -557,9 +557,8 @@ describe Ci::CreatePipelineService do subject { execute_service(variables_attributes: variables_attributes) } it 'creates a pipeline with specified variables' do - expect(subject.variables.count).to eq(variables_attributes.count) - expect(subject.variables.first.key).to eq(variables_attributes.first[:key]) - expect(subject.variables.last.secret_value).to eq(variables_attributes.last[:secret_value]) + expect(subject.variables.map { |var| var.slice(:key, :secret_value) }) + .to eq variables_attributes.map(&:with_indifferent_access) end end end |
