Commit 45b61a9e authored by Douwe Maan's avatar Douwe Maan

Merge branch 'blackst0ne-update-push-new-merge-request-url' into 'master'

Make new merge request URL more friendly when pushing code

Closes #53012

See merge request gitlab-org/gitlab-ce!22526
parents e4e40b96 72ff187c
...@@ -86,10 +86,10 @@ module CreatesCommit ...@@ -86,10 +86,10 @@ module CreatesCommit
def new_merge_request_path def new_merge_request_path
project_new_merge_request_path( project_new_merge_request_path(
@project_to_commit_into, @project_to_commit_into,
merge_request_source_branch: @branch_name,
merge_request: { merge_request: {
source_project_id: @project_to_commit_into.id, source_project_id: @project_to_commit_into.id,
target_project_id: @project.id, target_project_id: @project.id,
source_branch: @branch_name,
target_branch: @start_branch target_branch: @start_branch
} }
) )
......
...@@ -89,6 +89,8 @@ class Projects::MergeRequests::CreationsController < Projects::MergeRequests::Ap ...@@ -89,6 +89,8 @@ class Projects::MergeRequests::CreationsController < Projects::MergeRequests::Ap
def build_merge_request def build_merge_request
params[:merge_request] ||= ActionController::Parameters.new(source_project: @project) params[:merge_request] ||= ActionController::Parameters.new(source_project: @project)
params[:merge_request][:source_branch] ||= params[:merge_request_source_branch].presence
@merge_request = ::MergeRequests::BuildService.new(project, current_user, merge_request_params.merge(diff_options: diff_options)).execute @merge_request = ::MergeRequests::BuildService.new(project, current_user, merge_request_params.merge(diff_options: diff_options)).execute
end end
......
...@@ -13,8 +13,8 @@ module CompareHelper ...@@ -13,8 +13,8 @@ module CompareHelper
def create_mr_path(from = params[:from], to = params[:to], project = @project) def create_mr_path(from = params[:from], to = params[:to], project = @project)
project_new_merge_request_path( project_new_merge_request_path(
project, project,
merge_request_source_branch: to,
merge_request: { merge_request: {
source_branch: to,
target_branch: from target_branch: from
} }
) )
......
...@@ -11,10 +11,10 @@ module MergeRequestsHelper ...@@ -11,10 +11,10 @@ module MergeRequestsHelper
def new_mr_from_push_event(event, target_project) def new_mr_from_push_event(event, target_project)
{ {
merge_request_source_branch: event.branch_name,
merge_request: { merge_request: {
source_project_id: event.project.id, source_project_id: event.project.id,
target_project_id: target_project.id, target_project_id: target_project.id,
source_branch: event.branch_name,
target_branch: target_project.repository.root_ref target_branch: target_project.repository.root_ref
} }
} }
...@@ -51,10 +51,10 @@ module MergeRequestsHelper ...@@ -51,10 +51,10 @@ module MergeRequestsHelper
def mr_change_branches_path(merge_request) def mr_change_branches_path(merge_request)
project_new_merge_request_path( project_new_merge_request_path(
@project, @project,
merge_request_source_branch: merge_request.source_branch,
merge_request: { merge_request: {
source_project_id: merge_request.source_project_id, source_project_id: merge_request.source_project_id,
target_project_id: merge_request.target_project_id, target_project_id: merge_request.target_project_id,
source_branch: merge_request.source_branch,
target_branch: merge_request.target_branch target_branch: merge_request.target_branch
}, },
change_branches: true change_branches: true
......
...@@ -50,8 +50,8 @@ module MergeRequests ...@@ -50,8 +50,8 @@ module MergeRequests
end end
def url_for_new_merge_request(branch_name) def url_for_new_merge_request(branch_name)
merge_request_params = { source_branch: branch_name } url = Gitlab::Routing.url_helpers.project_new_merge_request_url(project, branch_name)
url = Gitlab::Routing.url_helpers.project_new_merge_request_url(project, merge_request: merge_request_params)
{ branch_name: branch_name, url: url, new_merge_request: true } { branch_name: branch_name, url: url, new_merge_request: true }
end end
......
---
title: Make new merge request URL more friendly when pushing code
merge_request: 22526
author: "@blackst0ne"
type: changed
...@@ -149,9 +149,7 @@ constraints(::Constraints::ProjectUrlConstrainer.new) do ...@@ -149,9 +149,7 @@ constraints(::Constraints::ProjectUrlConstrainer.new) do
scope path: 'merge_requests', controller: 'merge_requests/creations' do scope path: 'merge_requests', controller: 'merge_requests/creations' do
post '', action: :create, as: nil post '', action: :create, as: nil
scope path: 'new', as: :new_merge_request do scope path: 'new/(:merge_request_source_branch)', as: :new_merge_request do
get '', action: :new
scope constraints: { format: nil }, action: :new do scope constraints: { format: nil }, action: :new do
get :diffs, defaults: { tab: 'diffs' } get :diffs, defaults: { tab: 'diffs' }
get :pipelines, defaults: { tab: 'pipelines' } get :pipelines, defaults: { tab: 'pipelines' }
...@@ -165,6 +163,7 @@ constraints(::Constraints::ProjectUrlConstrainer.new) do ...@@ -165,6 +163,7 @@ constraints(::Constraints::ProjectUrlConstrainer.new) do
get :diff_for_path get :diff_for_path
get :branch_from get :branch_from
get :branch_to get :branch_to
get '', action: :new
end end
end end
......
...@@ -331,10 +331,10 @@ describe Projects::BlobController do ...@@ -331,10 +331,10 @@ describe Projects::BlobController do
expect(response).to redirect_to( expect(response).to redirect_to(
project_new_merge_request_path( project_new_merge_request_path(
forked_project, forked_project,
merge_request_source_branch: "fork-test-1",
merge_request: { merge_request: {
source_project_id: forked_project.id, source_project_id: forked_project.id,
target_project_id: project.id, target_project_id: project.id,
source_branch: "fork-test-1",
target_branch: "master" target_branch: "master"
} }
) )
......
...@@ -147,10 +147,12 @@ describe 'Dashboard Projects' do ...@@ -147,10 +147,12 @@ describe 'Dashboard Projects' do
end end
context 'last push widget', :use_clean_rails_memory_store_caching do context 'last push widget', :use_clean_rails_memory_store_caching do
let(:ref) { "feature" }
before do before do
event = create(:push_event, project: project, author: user) event = create(:push_event, project: project, author: user)
create(:push_event_payload, event: event, ref: 'feature', action: :created) create(:push_event_payload, event: event, ref: ref, action: :created)
Users::LastPushEventService.new(user).cache_last_push_event(event) Users::LastPushEventService.new(user).cache_last_push_event(event)
...@@ -165,9 +167,9 @@ describe 'Dashboard Projects' do ...@@ -165,9 +167,9 @@ describe 'Dashboard Projects' do
end end
expect(page).to have_selector('.merge-request-form') expect(page).to have_selector('.merge-request-form')
expect(current_path).to eq project_new_merge_request_path(project) expect(current_path).to eq project_new_merge_request_path(project, merge_request_source_branch: ref)
expect(find('#merge_request_target_project_id', visible: false).value).to eq project.id.to_s expect(find('#merge_request_target_project_id', visible: false).value).to eq project.id.to_s
expect(find('input#merge_request_source_branch', visible: false).value).to eq 'feature' expect(find('input#merge_request_source_branch', visible: false).value).to eq ref
expect(find('input#merge_request_target_branch', visible: false).value).to eq 'master' expect(find('input#merge_request_target_branch', visible: false).value).to eq 'master'
end end
end end
......
...@@ -9,10 +9,10 @@ describe 'create a merge request, allowing commits from members who can merge to ...@@ -9,10 +9,10 @@ describe 'create a merge request, allowing commits from members who can merge to
def visit_new_merge_request def visit_new_merge_request
visit project_new_merge_request_path( visit project_new_merge_request_path(
source_project, source_project,
merge_request_source_branch: 'fix',
merge_request: { merge_request: {
source_project_id: source_project.id, source_project_id: source_project.id,
target_project_id: target_project.id, target_project_id: target_project.id,
source_branch: 'fix',
target_branch: 'master' target_branch: 'master'
}) })
end end
......
...@@ -20,10 +20,10 @@ describe 'Merge request > User sees merge widget', :js do ...@@ -20,10 +20,10 @@ describe 'Merge request > User sees merge widget', :js do
before do before do
visit project_new_merge_request_path( visit project_new_merge_request_path(
project, project,
merge_request_source_branch: 'feature',
merge_request: { merge_request: {
source_project_id: project.id, source_project_id: project.id,
target_project_id: project.id, target_project_id: project.id,
source_branch: 'feature',
target_branch: 'master' target_branch: 'master'
}) })
end end
......
...@@ -13,10 +13,10 @@ describe 'Merge request > User sees WIP help message' do ...@@ -13,10 +13,10 @@ describe 'Merge request > User sees WIP help message' do
it 'shows a specific WIP hint' do it 'shows a specific WIP hint' do
visit project_new_merge_request_path( visit project_new_merge_request_path(
project, project,
merge_request_source_branch: 'wip',
merge_request: { merge_request: {
source_project_id: project.id, source_project_id: project.id,
target_project_id: project.id, target_project_id: project.id,
source_branch: 'wip',
target_branch: 'master' target_branch: 'master'
}) })
...@@ -32,10 +32,10 @@ describe 'Merge request > User sees WIP help message' do ...@@ -32,10 +32,10 @@ describe 'Merge request > User sees WIP help message' do
it 'shows the regular WIP message' do it 'shows the regular WIP message' do
visit project_new_merge_request_path( visit project_new_merge_request_path(
project, project,
merge_request_source_branch: 'fix',
merge_request: { merge_request: {
source_project_id: project.id, source_project_id: project.id,
target_project_id: project.id, target_project_id: project.id,
source_branch: 'fix',
target_branch: 'master' target_branch: 'master'
}) })
......
...@@ -109,13 +109,13 @@ describe 'Merge request > User selects branches for new MR', :js do ...@@ -109,13 +109,13 @@ describe 'Merge request > User selects branches for new MR', :js do
end end
it 'populates source branch button' do it 'populates source branch button' do
visit project_new_merge_request_path(project, change_branches: true, merge_request: { target_branch: 'master', source_branch: 'fix' }) visit project_new_merge_request_path(project, change_branches: true, merge_request_source_branch: 'fix', merge_request: { target_branch: 'master' })
expect(find('.js-source-branch')).to have_content('fix') expect(find('.js-source-branch')).to have_content('fix')
end end
it 'allows to change the diff view' do it 'allows to change the diff view' do
visit project_new_merge_request_path(project, merge_request: { target_branch: 'master', source_branch: 'fix' }) visit project_new_merge_request_path(project, merge_request_source_branch: 'fix', merge_request: { target_branch: 'master' })
click_link 'Changes' click_link 'Changes'
...@@ -131,7 +131,7 @@ describe 'Merge request > User selects branches for new MR', :js do ...@@ -131,7 +131,7 @@ describe 'Merge request > User selects branches for new MR', :js do
end end
it 'does not allow non-existing branches' do it 'does not allow non-existing branches' do
visit project_new_merge_request_path(project, merge_request: { target_branch: 'non-exist-target', source_branch: 'non-exist-source' }) visit project_new_merge_request_path(project, merge_request_source_branch: 'non-exist-source', merge_request: { target_branch: 'non-exist-target' })
expect(page).to have_content('The form contains the following errors') expect(page).to have_content('The form contains the following errors')
expect(page).to have_content('Source branch "non-exist-source" does not exist') expect(page).to have_content('Source branch "non-exist-source" does not exist')
...@@ -140,7 +140,7 @@ describe 'Merge request > User selects branches for new MR', :js do ...@@ -140,7 +140,7 @@ describe 'Merge request > User selects branches for new MR', :js do
context 'when a branch contains commits that both delete and add the same image' do context 'when a branch contains commits that both delete and add the same image' do
it 'renders the diff successfully' do it 'renders the diff successfully' do
visit project_new_merge_request_path(project, merge_request: { target_branch: 'master', source_branch: 'deleted-image-test' }) visit project_new_merge_request_path(project, merge_request_source_branch: 'deleted-image-test', merge_request: { target_branch: 'master' })
click_link "Changes" click_link "Changes"
...@@ -165,7 +165,8 @@ describe 'Merge request > User selects branches for new MR', :js do ...@@ -165,7 +165,8 @@ describe 'Merge request > User selects branches for new MR', :js do
it 'shows pipelines for a new merge request' do it 'shows pipelines for a new merge request' do
visit project_new_merge_request_path( visit project_new_merge_request_path(
project, project,
merge_request: { target_branch: 'master', source_branch: 'fix' }) merge_request_source_branch: 'fix',
merge_request: { target_branch: 'master' })
page.within('.merge-request') do page.within('.merge-request') do
click_link 'Pipelines' click_link 'Pipelines'
......
...@@ -144,7 +144,7 @@ describe 'Merge request > User uses quick actions', :js do ...@@ -144,7 +144,7 @@ describe 'Merge request > User uses quick actions', :js do
describe '/target_branch command in merge request' do describe '/target_branch command in merge request' do
let(:another_project) { create(:project, :public, :repository) } let(:another_project) { create(:project, :public, :repository) }
let(:new_url_opts) { { merge_request: { source_branch: 'feature' } } } let(:new_url_opts) { { merge_request_source_branch: 'feature' } }
before do before do
another_project.add_maintainer(user) another_project.add_maintainer(user)
......
...@@ -65,7 +65,7 @@ describe 'User squashes a merge request', :js do ...@@ -65,7 +65,7 @@ describe 'User squashes a merge request', :js do
context 'when squash is enabled on merge request creation' do context 'when squash is enabled on merge request creation' do
before do before do
visit project_new_merge_request_path(project, merge_request: { target_branch: 'master', source_branch: source_branch }) visit project_new_merge_request_path(project, merge_request_source_branch: source_branch, merge_request: { target_branch: 'master' })
check 'merge_request[squash]' check 'merge_request[squash]'
click_on 'Submit merge request' click_on 'Submit merge request'
wait_for_requests wait_for_requests
...@@ -95,7 +95,7 @@ describe 'User squashes a merge request', :js do ...@@ -95,7 +95,7 @@ describe 'User squashes a merge request', :js do
context 'when squash is not enabled on merge request creation' do context 'when squash is not enabled on merge request creation' do
before do before do
visit project_new_merge_request_path(project, merge_request: { target_branch: 'master', source_branch: source_branch }) visit project_new_merge_request_path(project, merge_request_source_branch: source_branch, merge_request: { target_branch: 'master' })
click_on 'Submit merge request' click_on 'Submit merge request'
wait_for_requests wait_for_requests
end end
......
...@@ -57,7 +57,7 @@ describe 'Projects > Files > User creates a directory', :js do ...@@ -57,7 +57,7 @@ describe 'Projects > Files > User creates a directory', :js do
expect(page).to have_content('From new-feature into master') expect(page).to have_content('From new-feature into master')
expect(page).to have_content('Add new directory') expect(page).to have_content('Add new directory')
expect(current_path).to eq(project_new_merge_request_path(project)) expect(current_path).to eq(project_new_merge_request_path(project, merge_request_source_branch: "new-feature"))
end end
end end
...@@ -80,8 +80,7 @@ describe 'Projects > Files > User creates a directory', :js do ...@@ -80,8 +80,7 @@ describe 'Projects > Files > User creates a directory', :js do
click_button('Create directory') click_button('Create directory')
fork = user.fork_of(project2.reload) fork = user.fork_of(project2.reload)
expect(current_path).to eq(project_new_merge_request_path(fork, merge_request_source_branch: "patch-1"))
expect(current_path).to eq(project_new_merge_request_path(fork))
end end
end end
end end
...@@ -144,7 +144,7 @@ describe 'Projects > Files > User creates files' do ...@@ -144,7 +144,7 @@ describe 'Projects > Files > User creates files' do
fill_in(:branch_name, with: 'new_branch_name', visible: true) fill_in(:branch_name, with: 'new_branch_name', visible: true)
click_button('Commit changes') click_button('Commit changes')
expect(current_path).to eq(project_new_merge_request_path(project)) expect(current_path).to eq(project_new_merge_request_path(project, merge_request_source_branch: "new_branch_name"))
click_link('Changes') click_link('Changes')
...@@ -182,7 +182,7 @@ describe 'Projects > Files > User creates files' do ...@@ -182,7 +182,7 @@ describe 'Projects > Files > User creates files' do
fork = user.fork_of(project2.reload) fork = user.fork_of(project2.reload)
expect(current_path).to eq(project_new_merge_request_path(fork)) expect(current_path).to eq(project_new_merge_request_path(fork, merge_request_source_branch: "patch-1"))
expect(page).to have_content('New commit message') expect(page).to have_content('New commit message')
end end
end end
......
...@@ -63,7 +63,7 @@ describe 'Projects > Files > User deletes files', :js do ...@@ -63,7 +63,7 @@ describe 'Projects > Files > User deletes files', :js do
fork = user.fork_of(project2.reload) fork = user.fork_of(project2.reload)
expect(current_path).to eq(project_new_merge_request_path(fork)) expect(current_path).to eq(project_new_merge_request_path(fork, merge_request_source_branch: "patch-1"))
expect(page).to have_content('New commit message') expect(page).to have_content('New commit message')
end end
end end
......
...@@ -86,7 +86,7 @@ describe 'Projects > Files > User edits files', :js do ...@@ -86,7 +86,7 @@ describe 'Projects > Files > User edits files', :js do
fill_in(:branch_name, with: 'new_branch_name', visible: true) fill_in(:branch_name, with: 'new_branch_name', visible: true)
click_button('Commit changes') click_button('Commit changes')
expect(current_path).to eq(project_new_merge_request_path(project)) expect(current_path).to eq(project_new_merge_request_path(project, merge_request_source_branch: "new_branch_name"))
click_link('Changes') click_link('Changes')
...@@ -155,7 +155,7 @@ describe 'Projects > Files > User edits files', :js do ...@@ -155,7 +155,7 @@ describe 'Projects > Files > User edits files', :js do
fork = user.fork_of(project2.reload) fork = user.fork_of(project2.reload)
expect(current_path).to eq(project_new_merge_request_path(fork)) expect(current_path).to eq(project_new_merge_request_path(fork, merge_request_source_branch: "patch-1"))
wait_for_requests wait_for_requests
...@@ -183,7 +183,7 @@ describe 'Projects > Files > User edits files', :js do ...@@ -183,7 +183,7 @@ describe 'Projects > Files > User edits files', :js do
fork = user.fork_of(project2) fork = user.fork_of(project2)
expect(current_path).to eq(project_new_merge_request_path(fork)) expect(current_path).to eq(project_new_merge_request_path(fork, merge_request_source_branch: "patch-1"))
wait_for_requests wait_for_requests
......
...@@ -78,7 +78,7 @@ describe 'Projects > Files > User replaces files', :js do ...@@ -78,7 +78,7 @@ describe 'Projects > Files > User replaces files', :js do
fork = user.fork_of(project2.reload) fork = user.fork_of(project2.reload)
expect(current_path).to eq(project_new_merge_request_path(fork)) expect(current_path).to eq(project_new_merge_request_path(fork, merge_request_source_branch: "undefined"))
click_link('Changes') click_link('Changes')
......
...@@ -36,7 +36,7 @@ describe 'Projects > Files > User uploads files' do ...@@ -36,7 +36,7 @@ describe 'Projects > Files > User uploads files' do
click_button('Upload file') click_button('Upload file')
expect(page).to have_content('New commit message') expect(page).to have_content('New commit message')
expect(current_path).to eq(project_new_merge_request_path(project)) expect(current_path).to eq(project_new_merge_request_path(project, merge_request_source_branch: "new_branch_name"))
click_link('Changes') click_link('Changes')
find("a[data-action='diffs']", text: 'Changes').click find("a[data-action='diffs']", text: 'Changes').click
...@@ -92,7 +92,7 @@ describe 'Projects > Files > User uploads files' do ...@@ -92,7 +92,7 @@ describe 'Projects > Files > User uploads files' do
fork = user.fork_of(project2.reload) fork = user.fork_of(project2.reload)
expect(current_path).to eq(project_new_merge_request_path(fork)) expect(current_path).to eq(project_new_merge_request_path(fork, merge_request_source_branch: "undefined"))
find("a[data-action='diffs']", text: 'Changes').click find("a[data-action='diffs']", text: 'Changes').click
......
...@@ -22,8 +22,8 @@ describe 'Merge Request button' do ...@@ -22,8 +22,8 @@ describe 'Merge Request button' do
it 'shows Create merge request button' do it 'shows Create merge request button' do
href = project_new_merge_request_path(project, href = project_new_merge_request_path(project,
merge_request: { source_branch: 'feature', merge_request_source_branch: 'feature',
target_branch: 'master' }) merge_request: { target_branch: 'master' })
visit url visit url
...@@ -77,8 +77,8 @@ describe 'Merge Request button' do ...@@ -77,8 +77,8 @@ describe 'Merge Request button' do
it 'shows Create merge request button' do it 'shows Create merge request button' do
href = project_new_merge_request_path(forked_project, href = project_new_merge_request_path(forked_project,
merge_request: { source_branch: 'feature', merge_request_source_branch: 'feature',
target_branch: 'master' }) merge_request: { target_branch: 'master' })
visit fork_url visit fork_url
......
...@@ -683,7 +683,7 @@ describe API::Internal do ...@@ -683,7 +683,7 @@ describe API::Internal do
expect(json_response).to match [{ expect(json_response).to match [{
"branch_name" => "new_branch", "branch_name" => "new_branch",
"url" => "http://#{Gitlab.config.gitlab.host}/#{project.namespace.name}/#{project.path}/merge_requests/new?merge_request%5Bsource_branch%5D=new_branch", "url" => "http://#{Gitlab.config.gitlab.host}/#{project.namespace.name}/#{project.path}/merge_requests/new/new_branch",
"new_merge_request" => true "new_merge_request" => true
}] }]
end end
...@@ -704,7 +704,7 @@ describe API::Internal do ...@@ -704,7 +704,7 @@ describe API::Internal do
expect(json_response).to match [{ expect(json_response).to match [{
"branch_name" => "new_branch", "branch_name" => "new_branch",
"url" => "http://#{Gitlab.config.gitlab.host}/#{project.namespace.name}/#{project.path}/merge_requests/new?merge_request%5Bsource_branch%5D=new_branch", "url" => "http://#{Gitlab.config.gitlab.host}/#{project.namespace.name}/#{project.path}/merge_requests/new/new_branch",
"new_merge_request" => true "new_merge_request" => true
}] }]
end end
...@@ -837,7 +837,7 @@ describe API::Internal do ...@@ -837,7 +837,7 @@ describe API::Internal do
expect(json_response['merge_request_urls']).to match [{ expect(json_response['merge_request_urls']).to match [{
"branch_name" => "new_branch", "branch_name" => "new_branch",
"url" => "http://#{Gitlab.config.gitlab.host}/#{project.namespace.name}/#{project.path}/merge_requests/new?merge_request%5Bsource_branch%5D=new_branch", "url" => "http://#{Gitlab.config.gitlab.host}/#{project.namespace.name}/#{project.path}/merge_requests/new/new_branch",
"new_merge_request" => true "new_merge_request" => true
}] }]
end end
......
...@@ -6,7 +6,7 @@ describe MergeRequests::GetUrlsService do ...@@ -6,7 +6,7 @@ describe MergeRequests::GetUrlsService do
let(:project) { create(:project, :public, :repository) } let(:project) { create(:project, :public, :repository) }
let(:service) { described_class.new(project) } let(:service) { described_class.new(project) }
let(:source_branch) { "merge-test" } let(:source_branch) { "merge-test" }
let(:new_merge_request_url) { "http://#{Gitlab.config.gitlab.host}/#{project.namespace.name}/#{project.path}/merge_requests/new?merge_request%5Bsource_branch%5D=#{source_branch}" } let(:new_merge_request_url) { "http://#{Gitlab.config.gitlab.host}/#{project.namespace.name}/#{project.path}/merge_requests/new/#{source_branch}" }
let(:show_merge_request_url) { "http://#{Gitlab.config.gitlab.host}/#{project.namespace.name}/#{project.path}/merge_requests/#{merge_request.iid}" } let(:show_merge_request_url) { "http://#{Gitlab.config.gitlab.host}/#{project.namespace.name}/#{project.path}/merge_requests/#{merge_request.iid}" }
let(:new_branch_changes) { "#{Gitlab::Git::BLANK_SHA} 570e7b2abdd848b95f2f578043fc23bd6f6fd24d refs/heads/#{source_branch}" } let(:new_branch_changes) { "#{Gitlab::Git::BLANK_SHA} 570e7b2abdd848b95f2f578043fc23bd6f6fd24d refs/heads/#{source_branch}" }
let(:deleted_branch_changes) { "d14d6c0abdd253381df51a723d58691b2ee1ab08 #{Gitlab::Git::BLANK_SHA} refs/heads/#{source_branch}" } let(:deleted_branch_changes) { "d14d6c0abdd253381df51a723d58691b2ee1ab08 #{Gitlab::Git::BLANK_SHA} refs/heads/#{source_branch}" }
...@@ -117,7 +117,7 @@ describe MergeRequests::GetUrlsService do ...@@ -117,7 +117,7 @@ describe MergeRequests::GetUrlsService do
let(:new_branch_changes) { "#{Gitlab::Git::BLANK_SHA} 570e7b2abdd848b95f2f578043fc23bd6f6fd24d refs/heads/new_branch" } let(:new_branch_changes) { "#{Gitlab::Git::BLANK_SHA} 570e7b2abdd848b95f2f578043fc23bd6f6fd24d refs/heads/new_branch" }
let(:existing_branch_changes) { "d14d6c0abdd253381df51a723d58691b2ee1ab08 570e7b2abdd848b95f2f578043fc23bd6f6fd24d refs/heads/markdown" } let(:existing_branch_changes) { "d14d6c0abdd253381df51a723d58691b2ee1ab08 570e7b2abdd848b95f2f578043fc23bd6f6fd24d refs/heads/markdown" }
let(:changes) { "#{new_branch_changes}\n#{existing_branch_changes}" } let(:changes) { "#{new_branch_changes}\n#{existing_branch_changes}" }
let(:new_merge_request_url) { "http://#{Gitlab.config.gitlab.host}/#{project.namespace.name}/#{project.path}/merge_requests/new?merge_request%5Bsource_branch%5D=new_branch" } let(:new_merge_request_url) { "http://#{Gitlab.config.gitlab.host}/#{project.namespace.name}/#{project.path}/merge_requests/new/new_branch" }
it 'returns 2 urls for both creating new and showing merge request' do it 'returns 2 urls for both creating new and showing merge request' do
result = service.execute(changes) result = service.execute(changes)
......
...@@ -17,10 +17,10 @@ RSpec.shared_examples 'a creatable merge request' do ...@@ -17,10 +17,10 @@ RSpec.shared_examples 'a creatable merge request' do
sign_in(user) sign_in(user)
visit project_new_merge_request_path( visit project_new_merge_request_path(
target_project, target_project,
merge_request_source_branch: 'fix',
merge_request: { merge_request: {
source_project_id: source_project.id, source_project_id: source_project.id,
target_project_id: target_project.id, target_project_id: target_project.id,
source_branch: 'fix',
target_branch: 'master' target_branch: 'master'
}) })
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