diff options
author | Rémy Coutable <remy@rymai.me> | 2017-09-13 13:28:56 +0000 |
---|---|---|
committer | Rémy Coutable <remy@rymai.me> | 2017-09-13 13:28:56 +0000 |
commit | 7bd66b7612dde47a85ca19d478fdd32810eb55de (patch) | |
tree | af5943aad5e01592a409f2bb0d233c8738146933 /spec/services/ci | |
parent | 1f5ae8cd8b3663ea2de883ec4605dc4100465bf4 (diff) | |
parent | 6fe19ce96104e19677ae6583868016e3116687f3 (diff) | |
download | gitlab-ce-7bd66b7612dde47a85ca19d478fdd32810eb55de.tar.gz |
Merge branch 'backstage/gb/optimize-some-specs-with-rspec-set' into 'master'
Optimize some tests with RSpec Set
See merge request !14047
Diffstat (limited to 'spec/services/ci')
-rw-r--r-- | spec/services/ci/retry_build_service_spec.rb | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/spec/services/ci/retry_build_service_spec.rb b/spec/services/ci/retry_build_service_spec.rb index bbc3a8c79f5..fbb3213f42b 100644 --- a/spec/services/ci/retry_build_service_spec.rb +++ b/spec/services/ci/retry_build_service_spec.rb @@ -1,9 +1,10 @@ require 'spec_helper' describe Ci::RetryBuildService do - let(:user) { create(:user) } - let(:project) { create(:project) } - let(:pipeline) { create(:ci_pipeline, project: project) } + set(:user) { create(:user) } + set(:project) { create(:project) } + set(:pipeline) { create(:ci_pipeline, project: project) } + let(:build) { create(:ci_build, pipeline: pipeline) } let(:service) do @@ -37,7 +38,7 @@ describe Ci::RetryBuildService do :queued, :coverage, :tags, :allowed_to_fail, :on_tag, :triggered, :trace, :teardown_environment, description: 'my-job', stage: 'test', pipeline: pipeline, - auto_canceled_by: create(:ci_empty_pipeline)) do |build| + auto_canceled_by: create(:ci_empty_pipeline, project: project)) do |build| ## # TODO, workaround for FactoryGirl limitation when having both # stage (text) and stage_id (integer) columns in the table. |