Commit 452a4399 authored by Markus Doits's avatar Markus Doits

add specs for create pipeline service and new retry when feature

parent 0db50a80
...@@ -435,6 +435,7 @@ describe Ci::CreatePipelineService do ...@@ -435,6 +435,7 @@ describe Ci::CreatePipelineService do
end end
context 'when builds with auto-retries are configured' do context 'when builds with auto-retries are configured' do
context 'as an integer' do
before do before do
config = YAML.dump(rspec: { script: 'rspec', retry: 2 }) config = YAML.dump(rspec: { script: 'rspec', retry: 2 })
stub_ci_pipeline_yaml_file(config) stub_ci_pipeline_yaml_file(config)
...@@ -445,6 +446,23 @@ describe Ci::CreatePipelineService do ...@@ -445,6 +446,23 @@ describe Ci::CreatePipelineService do
expect(pipeline).to be_persisted expect(pipeline).to be_persisted
expect(pipeline.builds.find_by(name: 'rspec').retries_max).to eq 2 expect(pipeline.builds.find_by(name: 'rspec').retries_max).to eq 2
expect(pipeline.builds.find_by(name: 'rspec').retry_when).to eq ['always']
end
end
context 'as hash' do
before do
config = YAML.dump(rspec: { script: 'rspec', retry: { max: 2, when: 'runner_system_failure' } })
stub_ci_pipeline_yaml_file(config)
end
it 'correctly creates builds with auto-retry value configured' do
pipeline = execute_service
expect(pipeline).to be_persisted
expect(pipeline.builds.find_by(name: 'rspec').retries_max).to eq 2
expect(pipeline.builds.find_by(name: 'rspec').retry_when).to eq ['runner_system_failure']
end
end end
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment