diff options
author | Shinya Maeda <shinya@gitlab.com> | 2018-09-28 14:41:39 +0900 |
---|---|---|
committer | Alessio Caiazza <acaiazza@gitlab.com> | 2018-10-02 17:08:13 +0200 |
commit | 54263dc1d9719f871fb2f23ecac4e0cfcabebe77 (patch) | |
tree | be5a3d1fbbdd3344ca387b576990861af6bd387f /spec/services | |
parent | b5a591d8c2c8fff0136037f6c2a8b635f101e07e (diff) | |
download | gitlab-ce-54263dc1d9719f871fb2f23ecac4e0cfcabebe77.tar.gz |
Fix coding style offence
Diffstat (limited to 'spec/services')
-rw-r--r-- | spec/services/ci/process_pipeline_service_spec.rb | 21 |
1 files changed, 12 insertions, 9 deletions
diff --git a/spec/services/ci/process_pipeline_service_spec.rb b/spec/services/ci/process_pipeline_service_spec.rb index d314d774be4..3ce3785b162 100644 --- a/spec/services/ci/process_pipeline_service_spec.rb +++ b/spec/services/ci/process_pipeline_service_spec.rb @@ -257,7 +257,7 @@ describe Ci::ProcessPipelineService, '#execute' do it 'properly processes the pipeline' do expect(process_pipeline).to be_truthy expect(builds_names_and_statuses).to eq({ 'build': 'pending' }) - + succeed_pending expect(builds_names_and_statuses).to eq({ 'build': 'success', 'rollout10%': 'scheduled' }) @@ -283,7 +283,7 @@ describe Ci::ProcessPipelineService, '#execute' do it 'properly processes the pipeline' do expect(process_pipeline).to be_truthy expect(builds_names_and_statuses).to eq({ 'build': 'pending' }) - + fail_running_or_pending expect(builds_names_and_statuses).to eq({ 'build': 'failed' }) @@ -295,11 +295,11 @@ describe Ci::ProcessPipelineService, '#execute' do it 'properly processes the pipeline' do expect(process_pipeline).to be_truthy expect(builds_names_and_statuses).to eq({ 'build': 'pending' }) - + succeed_pending expect(builds_names_and_statuses).to eq({ 'build': 'success', 'rollout10%': 'scheduled' }) - + unschedule expect(builds_names_and_statuses).to eq({ 'build': 'success', 'rollout10%': 'manual' }) @@ -324,11 +324,11 @@ describe Ci::ProcessPipelineService, '#execute' do it 'properly processes the pipeline' do expect(process_pipeline).to be_truthy expect(builds_names_and_statuses).to eq({ 'build': 'pending' }) - + succeed_pending expect(builds_names_and_statuses).to eq({ 'build': 'success', 'rollout10%': 'scheduled' }) - + enqueue_scheduled('rollout10%') fail_running_or_pending @@ -354,11 +354,11 @@ describe Ci::ProcessPipelineService, '#execute' do it 'properly processes the pipeline' do expect(process_pipeline).to be_truthy expect(builds_names_and_statuses).to eq({ 'build': 'pending' }) - + succeed_pending expect(builds_names_and_statuses).to eq({ 'build': 'success', 'rollout10%': 'scheduled' }) - + play_manual_action('rollout10%') expect(builds_names_and_statuses).to eq({ 'build': 'success', 'rollout10%': 'pending' }) @@ -718,7 +718,10 @@ describe Ci::ProcessPipelineService, '#execute' do end def builds_names_and_statuses - builds.inject({}) { |h, b| h[b.name.to_sym] = b.status; h } + builds.each_with_object({}) do |h, b| + h[b.name.to_sym] = b.status + h + end end def all_builds_names |