Commit 249e6bcf authored by Nick Thomas's avatar Nick Thomas

Fix conflicts in specs

parent 460f60fa
...@@ -118,13 +118,6 @@ feature 'Jobs', :feature do ...@@ -118,13 +118,6 @@ feature 'Jobs', :feature do
before do before do
visit namespace_project_job_path(project.namespace, project, job) visit namespace_project_job_path(project.namespace, project, job)
<<<<<<< HEAD
end
it 'shows status name', :js do
expect(page).to have_css('.ci-status.ci-success', text: 'passed')
=======
>>>>>>> upstream/master
end end
it 'shows commit`s data' do it 'shows commit`s data' do
...@@ -373,11 +366,7 @@ feature 'Jobs', :feature do ...@@ -373,11 +366,7 @@ feature 'Jobs', :feature do
before do before do
job.run! job.run!
visit namespace_project_job_path(project.namespace, project, job) visit namespace_project_job_path(project.namespace, project, job)
<<<<<<< HEAD
find('.js-cancel-job').click() find('.js-cancel-job').click()
=======
click_link "Cancel"
>>>>>>> upstream/master
end end
it 'loads the page and shows all needed controls' do it 'loads the page and shows all needed controls' do
...@@ -385,8 +374,6 @@ feature 'Jobs', :feature do ...@@ -385,8 +374,6 @@ feature 'Jobs', :feature do
expect(page).to have_content 'Retry' expect(page).to have_content 'Retry'
end end
end end
<<<<<<< HEAD
=======
context "Job from other project" do context "Job from other project" do
before do before do
...@@ -397,7 +384,6 @@ feature 'Jobs', :feature do ...@@ -397,7 +384,6 @@ feature 'Jobs', :feature do
it { expect(page.status_code).to eq(404) } it { expect(page.status_code).to eq(404) }
end end
>>>>>>> upstream/master
end end
describe "POST /:project/jobs/:id/retry" do describe "POST /:project/jobs/:id/retry" do
...@@ -405,15 +391,10 @@ feature 'Jobs', :feature do ...@@ -405,15 +391,10 @@ feature 'Jobs', :feature do
before do before do
job.run! job.run!
visit namespace_project_job_path(project.namespace, project, job) visit namespace_project_job_path(project.namespace, project, job)
<<<<<<< HEAD
find('.js-cancel-job').click()
find('.js-retry-button').trigger('click')
=======
click_link 'Cancel' click_link 'Cancel'
page.within('.build-header') do page.within('.build-header') do
click_link 'Retry job' click_link 'Retry job'
end end
>>>>>>> upstream/master
end end
it 'shows the right status and buttons', :js do it 'shows the right status and buttons', :js do
...@@ -424,8 +405,6 @@ feature 'Jobs', :feature do ...@@ -424,8 +405,6 @@ feature 'Jobs', :feature do
end end
end end
<<<<<<< HEAD
=======
context "Job from other project" do context "Job from other project" do
before do before do
job.run! job.run!
...@@ -437,7 +416,6 @@ feature 'Jobs', :feature do ...@@ -437,7 +416,6 @@ feature 'Jobs', :feature do
it { expect(page).to have_http_status(404) } it { expect(page).to have_http_status(404) }
end end
>>>>>>> upstream/master
context "Job that current user is not allowed to retry" do context "Job that current user is not allowed to retry" do
before do before do
job.run! job.run!
...@@ -511,16 +489,6 @@ feature 'Jobs', :feature do ...@@ -511,16 +489,6 @@ feature 'Jobs', :feature do
Capybara.current_session.driver.headers = { 'X-Sendfile-Type' => 'X-Sendfile' } Capybara.current_session.driver.headers = { 'X-Sendfile-Type' => 'X-Sendfile' }
job.run! job.run!
<<<<<<< HEAD
end
context 'when job has trace in file', :js do
=======
allow_any_instance_of(Gitlab::Ci::Trace).to receive(:paths)
.and_return(paths)
visit namespace_project_job_path(project.namespace, project, job)
end end
context 'when job has trace in file', :js do context 'when job has trace in file', :js do
...@@ -528,7 +496,6 @@ feature 'Jobs', :feature do ...@@ -528,7 +496,6 @@ feature 'Jobs', :feature do
[existing_file] [existing_file]
end end
>>>>>>> upstream/master
before do before do
allow_any_instance_of(Gitlab::Ci::Trace) allow_any_instance_of(Gitlab::Ci::Trace)
.to receive(:paths) .to receive(:paths)
......
...@@ -11,11 +11,7 @@ describe API::Jobs, :api do ...@@ -11,11 +11,7 @@ describe API::Jobs, :api do
ref: project.default_branch) ref: project.default_branch)
end end
<<<<<<< HEAD
let!(:job) { create(:ci_build, pipeline: pipeline) }
=======
let(:job) { create(:ci_build, pipeline: pipeline) } let(:job) { create(:ci_build, pipeline: pipeline) }
>>>>>>> upstream/master
let(:user) { create(:user) } let(:user) { create(:user) }
let(:api_user) { user } let(:api_user) { user }
...@@ -30,10 +26,7 @@ describe API::Jobs, :api do ...@@ -30,10 +26,7 @@ describe API::Jobs, :api do
let(:query) { Hash.new } let(:query) { Hash.new }
before do before do
<<<<<<< HEAD
=======
job job
>>>>>>> upstream/master
get api("/projects/#{project.id}/jobs", api_user), query get api("/projects/#{project.id}/jobs", api_user), query
end end
...@@ -97,10 +90,7 @@ describe API::Jobs, :api do ...@@ -97,10 +90,7 @@ describe API::Jobs, :api do
let(:query) { Hash.new } let(:query) { Hash.new }
before do before do
<<<<<<< HEAD
=======
job job
>>>>>>> upstream/master
get api("/projects/#{project.id}/pipelines/#{pipeline.id}/jobs", api_user), query get api("/projects/#{project.id}/pipelines/#{pipeline.id}/jobs", api_user), query
end end
...@@ -203,6 +193,7 @@ describe API::Jobs, :api do ...@@ -203,6 +193,7 @@ describe API::Jobs, :api do
describe 'GET /projects/:id/jobs/:job_id/artifacts' do describe 'GET /projects/:id/jobs/:job_id/artifacts' do
before do before do
stub_artifacts_object_storage stub_artifacts_object_storage
job
get api("/projects/#{project.id}/jobs/#{job.id}/artifacts", api_user) get api("/projects/#{project.id}/jobs/#{job.id}/artifacts", api_user)
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