Commit 88aacaa7 authored by Lin Jen-Shin's avatar Lin Jen-Shin

Reuse those two methods

parent a7713232
...@@ -1115,19 +1115,23 @@ describe Project, models: true do ...@@ -1115,19 +1115,23 @@ describe Project, models: true do
end end
describe '#latest_successful_builds_for' do describe '#latest_successful_builds_for' do
let(:project) { create(:project) } def create_pipeline
let(:pipeline) do
create(:ci_pipeline, project: project, create(:ci_pipeline, project: project,
sha: project.commit.id, sha: project.commit.sha,
ref: project.default_branch, ref: project.default_branch,
status: 'success') status: 'success')
end end
let(:build) do def create_build(new_pipeline = pipeline, name = 'test')
create(:ci_build, :artifacts, :success, pipeline: pipeline) create(:ci_build, :success, :artifacts,
pipeline: new_pipeline,
name: name)
end end
let(:project) { create(:project) }
let(:pipeline) { create_pipeline }
let(:build) { create_build }
context 'with succeeded pipeline' do context 'with succeeded pipeline' do
context 'standalone pipeline' do context 'standalone pipeline' do
before do before do
...@@ -1163,19 +1167,6 @@ describe Project, models: true do ...@@ -1163,19 +1167,6 @@ describe Project, models: true do
expect(latest_builds).to contain_exactly(@build2_p2, @build1_p2) expect(latest_builds).to contain_exactly(@build2_p2, @build1_p2)
end end
def create_pipeline
create(:ci_pipeline, project: project,
sha: project.commit.sha,
ref: project.default_branch,
status: 'success')
end
def create_build(pipe, name = 'test')
create(:ci_build, :success, :artifacts,
pipeline: pipe,
name: name)
end
end end
context 'with multiple pipelines and builds' do context 'with multiple pipelines and builds' do
......
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