summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatija Čupić <matteeyah@gmail.com>2018-05-04 12:35:36 +0200
committerMatija Čupić <matteeyah@gmail.com>2018-05-04 12:35:36 +0200
commitc6e394bb8499b470b9d2a0c03c8ce7fd023fa08a (patch)
tree8f3050e8d9a12d809c48ceda8eb0a411ad6259b7
parent980fb6fb26a360894bba7444d0020381ab825c8d (diff)
downloadgitlab-ce-c6e394bb8499b470b9d2a0c03c8ce7fd023fa08a.tar.gz
Improve variable comparison
-rw-r--r--spec/lib/gitlab/ci/pipeline/chain/build_spec.rb3
-rw-r--r--spec/services/ci/create_pipeline_service_spec.rb5
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