Commit eaebe30b authored by Nick Thomas's avatar Nick Thomas

Resolve most spec conflicts

parent f549b01a
require 'spec_helper'
describe 'seed production settings', lib: true do
include StubENV
context 'GITLAB_SHARED_RUNNERS_REGISTRATION_TOKEN is set in the environment' do
before do
stub_env('GITLAB_SHARED_RUNNERS_REGISTRATION_TOKEN', '013456789')
end
it 'writes the token to the database' do
load(File.join(__dir__, '../../../db/fixtures/production/010_settings.rb'))
expect(ApplicationSetting.current.runners_registration_token).to eq('013456789')
end
end
end
...@@ -42,15 +42,12 @@ FactoryGirl.define do ...@@ -42,15 +42,12 @@ FactoryGirl.define do
import_status :failed import_status :failed
end end
<<<<<<< HEAD
trait :mirror do trait :mirror do
mirror true mirror true
import_url { generate(:url) } import_url { generate(:url) }
mirror_user_id { creator_id } mirror_user_id { creator_id }
end end
=======
>>>>>>> ce/master
trait :archived do trait :archived do
archived true archived true
end end
......
require 'spec_helper' require 'spec_helper'
<<<<<<< HEAD feature 'Protected Branches', feature: true, js: true do
feature 'Projected Branches', feature: true, js: true do
include ProtectedBranchHelpers include ProtectedBranchHelpers
=======
feature 'Protected Branches', feature: true, js: true do
>>>>>>> ce/master
let(:user) { create(:user, :admin) } let(:user) { create(:user, :admin) }
let(:project) { create(:project, :repository) } let(:project) { create(:project, :repository) }
......
...@@ -68,19 +68,11 @@ describe Gitlab::EtagCaching::Router do ...@@ -68,19 +68,11 @@ describe Gitlab::EtagCaching::Router do
end end
it 'matches build endpoint' do it 'matches build endpoint' do
<<<<<<< HEAD
env = build_env(
'/my-group/my-project/builds/234.json'
)
result = described_class.match(env)
=======
request = build_request( request = build_request(
'/my-group/my-project/builds/234.json' '/my-group/my-project/builds/234.json'
) )
result = described_class.match(request) result = described_class.match(request)
>>>>>>> ce/master
expect(result).to be_present expect(result).to be_present
expect(result.name).to eq 'project_build' expect(result.name).to eq 'project_build'
......
...@@ -7,11 +7,7 @@ describe Gitlab::GitAccess, lib: true do ...@@ -7,11 +7,7 @@ describe Gitlab::GitAccess, lib: true do
let(:project) { create(:project, :repository) } let(:project) { create(:project, :repository) }
let(:user) { create(:user) } let(:user) { create(:user) }
let(:actor) { user } let(:actor) { user }
<<<<<<< HEAD
=======
let(:protocol) { 'ssh' } let(:protocol) { 'ssh' }
>>>>>>> ce/master
let(:authentication_abilities) do let(:authentication_abilities) do
[ [
:read_project, :read_project,
...@@ -302,11 +298,7 @@ describe Gitlab::GitAccess, lib: true do ...@@ -302,11 +298,7 @@ describe Gitlab::GitAccess, lib: true do
let(:project) { create(:project, :repository, namespace: user.namespace) } let(:project) { create(:project, :repository, namespace: user.namespace) }
context 'pull code' do context 'pull code' do
<<<<<<< HEAD
it { expect { subject }.not_to raise_error }
=======
it { expect { pull_access_check }.not_to raise_error } it { expect { pull_access_check }.not_to raise_error }
>>>>>>> ce/master
end end
end end
...@@ -314,11 +306,7 @@ describe Gitlab::GitAccess, lib: true do ...@@ -314,11 +306,7 @@ describe Gitlab::GitAccess, lib: true do
before { project.team << [user, :reporter] } before { project.team << [user, :reporter] }
context 'pull code' do context 'pull code' do
<<<<<<< HEAD
it { expect { subject }.not_to raise_error }
=======
it { expect { pull_access_check }.not_to raise_error } it { expect { pull_access_check }.not_to raise_error }
>>>>>>> ce/master
end end
end end
...@@ -329,21 +317,13 @@ describe Gitlab::GitAccess, lib: true do ...@@ -329,21 +317,13 @@ describe Gitlab::GitAccess, lib: true do
before { project.team << [user, :reporter] } before { project.team << [user, :reporter] }
context 'pull code' do context 'pull code' do
<<<<<<< HEAD
it { expect { subject }.not_to raise_error }
=======
it { expect { pull_access_check }.not_to raise_error } it { expect { pull_access_check }.not_to raise_error }
>>>>>>> ce/master
end end
end end
context 'when is not member of the project' do context 'when is not member of the project' do
context 'pull code' do context 'pull code' do
<<<<<<< HEAD
it { expect { subject }.not_to raise_error }
=======
it { expect { pull_access_check }.to raise_unauthorized('You are not allowed to download code from this project.') } it { expect { pull_access_check }.to raise_unauthorized('You are not allowed to download code from this project.') }
>>>>>>> ce/master
end end
end end
end end
......
...@@ -109,18 +109,15 @@ pipelines: ...@@ -109,18 +109,15 @@ pipelines:
- artifacts - artifacts
- pipeline_schedule - pipeline_schedule
- merge_requests - merge_requests
<<<<<<< HEAD
- source_pipeline - source_pipeline
- sourced_pipelines - sourced_pipelines
- triggered_by_pipeline - triggered_by_pipeline
- triggered_pipelines - triggered_pipelines
=======
stages: stages:
- project - project
- pipeline - pipeline
- statuses - statuses
- builds - builds
>>>>>>> ce/master
statuses: statuses:
- project - project
- pipeline - pipeline
......
...@@ -3,10 +3,7 @@ require 'spec_helper' ...@@ -3,10 +3,7 @@ require 'spec_helper'
describe ForkedProjectLink, "add link on fork" do describe ForkedProjectLink, "add link on fork" do
let(:project_from) { create(:project, :repository) } let(:project_from) { create(:project, :repository) }
let(:user) { create(:user) } let(:user) { create(:user) }
<<<<<<< HEAD
=======
let(:namespace) { user.namespace } let(:namespace) { user.namespace }
>>>>>>> ce/master
before do before do
create(:project_member, :reporter, user: user, project: project_from) create(:project_member, :reporter, user: user, project: project_from)
......
...@@ -1890,7 +1890,6 @@ describe Project, models: true do ...@@ -1890,7 +1890,6 @@ describe Project, models: true do
project.add_import_job project.add_import_job
end end
<<<<<<< HEAD
context 'without mirror' do context 'without mirror' do
it 'returns nil' do it 'returns nil' do
project = create(:project) project = create(:project)
...@@ -1902,13 +1901,6 @@ describe Project, models: true do ...@@ -1902,13 +1901,6 @@ describe Project, models: true do
context 'without repository' do context 'without repository' do
it 'schedules RepositoryImportWorker' do it 'schedules RepositoryImportWorker' do
project = create(:empty_project, import_url: generate(:url)) project = create(:empty_project, import_url: generate(:url))
=======
context 'not forked' do
it 'schedules a RepositoryImportWorker job' do
project = create(:empty_project, import_url: generate(:url))
expect(RepositoryImportWorker).to receive(:perform_async).with(project.id)
>>>>>>> ce/master
expect(RepositoryImportWorker).to receive(:perform_async).with(project.id) expect(RepositoryImportWorker).to receive(:perform_async).with(project.id)
...@@ -1926,6 +1918,19 @@ describe Project, models: true do ...@@ -1926,6 +1918,19 @@ describe Project, models: true do
end end
end end
end end
context 'not forked' do
it 'schedules a RepositoryImportWorker job' do
project = create(:empty_project, import_url: generate(:url))
expect(RepositoryImportWorker).to receive(:perform_async).with(project.id)
expect(RepositoryImportWorker).to receive(:perform_async).with(project.id)
project.add_import_job
end
end
end
end end
describe '#gitlab_project_import?' do describe '#gitlab_project_import?' do
......
...@@ -15,13 +15,10 @@ describe User, models: true do ...@@ -15,13 +15,10 @@ describe User, models: true do
describe 'delegations' do describe 'delegations' do
it { is_expected.to delegate_method(:path).to(:namespace).with_prefix } it { is_expected.to delegate_method(:path).to(:namespace).with_prefix }
<<<<<<< HEAD
# EE # EE
it { is_expected.to delegate_method(:shared_runners_minutes_limit).to(:namespace) } it { is_expected.to delegate_method(:shared_runners_minutes_limit).to(:namespace) }
it { is_expected.to delegate_method(:shared_runners_minutes_limit=).to(:namespace).with_arguments(133) } it { is_expected.to delegate_method(:shared_runners_minutes_limit=).to(:namespace).with_arguments(133) }
=======
>>>>>>> ce/master
end end
describe 'associations' do describe 'associations' do
......
...@@ -457,7 +457,6 @@ describe API::Users do ...@@ -457,7 +457,6 @@ describe API::Users do
expect(response).to have_http_status(403) expect(response).to have_http_status(403)
end end
<<<<<<< HEAD
it "cannot update their own shared_runners_minutes_limit" do it "cannot update their own shared_runners_minutes_limit" do
expect do expect do
...@@ -466,8 +465,6 @@ describe API::Users do ...@@ -466,8 +465,6 @@ describe API::Users do
expect(response).to have_http_status(403) expect(response).to have_http_status(403)
end end
=======
>>>>>>> ce/master
end end
it "returns 404 for non-existing user" do it "returns 404 for non-existing user" do
......
...@@ -116,7 +116,6 @@ describe PipelineDetailsEntity do ...@@ -116,7 +116,6 @@ describe PipelineDetailsEntity do
expect(subject[:flags][:yaml_errors]).to be false expect(subject[:flags][:yaml_errors]).to be false
end end
end end
<<<<<<< HEAD
context 'when pipeline is triggered by other pipeline' do context 'when pipeline is triggered by other pipeline' do
let(:pipeline) { create(:ci_empty_pipeline) } let(:pipeline) { create(:ci_empty_pipeline) }
...@@ -152,7 +151,5 @@ describe PipelineDetailsEntity do ...@@ -152,7 +151,5 @@ describe PipelineDetailsEntity do
expect(subject[:triggered].first[:project]).not_to be_nil expect(subject[:triggered].first[:project]).not_to be_nil
end end
end end
=======
>>>>>>> ce/master
end end
end end
...@@ -174,17 +174,9 @@ describe CreateDeploymentService, services: true do ...@@ -174,17 +174,9 @@ describe CreateDeploymentService, services: true do
{ environment: { name: 'production', url: 'http://gitlab.com' } } { environment: { name: 'production', url: 'http://gitlab.com' } }
end end
<<<<<<< HEAD
context 'when pipeline succeeds' do
it_behaves_like 'does create environment and deployment' do
let(:deployable) { build }
subject { build.success }
=======
context 'when job succeeds' do context 'when job succeeds' do
it_behaves_like 'creates deployment' do it_behaves_like 'creates deployment' do
subject { job.success } subject { job.success }
>>>>>>> ce/master
end end
end end
......
...@@ -3,11 +3,7 @@ require 'spec_helper' ...@@ -3,11 +3,7 @@ require 'spec_helper'
describe Projects::ForkService, services: true do describe Projects::ForkService, services: true do
describe 'fork by user' do describe 'fork by user' do
before do before do
<<<<<<< HEAD
@from_user = create(:user )
=======
@from_user = create(:user) @from_user = create(:user)
>>>>>>> ce/master
@from_namespace = @from_user.namespace @from_namespace = @from_user.namespace
avatar = fixture_file_upload(Rails.root + "spec/fixtures/dk.png", "image/png") avatar = fixture_file_upload(Rails.root + "spec/fixtures/dk.png", "image/png")
@from_project = create(:project, @from_project = create(:project,
......
...@@ -56,11 +56,8 @@ RSpec.configure do |config| ...@@ -56,11 +56,8 @@ RSpec.configure do |config|
config.include StubGitlabCalls config.include StubGitlabCalls
config.include StubGitlabData config.include StubGitlabData
config.include ApiHelpers, :api config.include ApiHelpers, :api
<<<<<<< HEAD
config.include Rails.application.routes.url_helpers, type: :routing
=======
config.include MigrationsHelpers, :migration config.include MigrationsHelpers, :migration
>>>>>>> ce/master config.include Rails.application.routes.url_helpers, type: :routing
config.infer_spec_type_from_file_location! config.infer_spec_type_from_file_location!
......
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