Commit 73be775c authored by Kamil Trzcinski's avatar Kamil Trzcinski

Fix ci_build trace trait

parent 6c6bb794
...@@ -538,7 +538,7 @@ describe Ci::Build, models: true do ...@@ -538,7 +538,7 @@ describe Ci::Build, models: true do
end end
context 'build is erasable' do context 'build is erasable' do
let!(:build) { create(:ci_build_with_trace, :success, :artifacts) } let!(:build) { create(:ci_build, :trace, :success, :artifacts) }
describe '#erase' do describe '#erase' do
before { build.erase(erased_by: user) } before { build.erase(erased_by: user) }
...@@ -570,7 +570,7 @@ describe Ci::Build, models: true do ...@@ -570,7 +570,7 @@ describe Ci::Build, models: true do
end end
describe '#erased?' do describe '#erased?' do
let!(:build) { create(:ci_build_with_trace, :success, :artifacts) } let!(:build) { create(:ci_build, :trace, :success, :artifacts) }
subject { build.erased? } subject { build.erased? }
context 'build has not been erased' do context 'build has not been erased' do
......
...@@ -218,7 +218,7 @@ describe API::API, api: true do ...@@ -218,7 +218,7 @@ describe API::API, api: true do
end end
context 'build is erasable' do context 'build is erasable' do
let(:build) { create(:ci_build_with_trace, :artifacts, :success, project: project, commit: commit) } let(:build) { create(:ci_build, :trace, :artifacts, :success, project: project, commit: commit) }
it 'should erase build content' do it 'should erase build content' do
expect(response.status).to eq 201 expect(response.status).to eq 201
...@@ -234,7 +234,7 @@ describe API::API, api: true do ...@@ -234,7 +234,7 @@ describe API::API, api: true do
end end
context 'build is not erasable' do context 'build is not erasable' do
let(:build) { create(:ci_build_with_trace, project: project, commit: commit) } let(:build) { create(:ci_build, :trace, project: project, commit: commit) }
it 'should respond with forbidden' do it 'should respond with forbidden' do
expect(response.status).to eq 403 expect(response.status).to eq 403
......
...@@ -132,7 +132,7 @@ describe Ci::API::API do ...@@ -132,7 +132,7 @@ describe Ci::API::API do
describe "PUT /builds/:id" do describe "PUT /builds/:id" do
let(:commit) {create(:ci_commit, project: project)} let(:commit) {create(:ci_commit, project: project)}
let(:build) { create(:ci_build_with_trace, commit: commit, runner_id: runner.id) } let(:build) { create(:ci_build, :trace, commit: commit, runner_id: runner.id) }
before do before do
build.run! build.run!
......
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