2019-04-11 12:17:24 +00:00
|
|
|
# frozen_string_literal: true
|
|
|
|
|
2018-05-29 09:51:43 +00:00
|
|
|
require 'spec_helper'
|
|
|
|
|
|
|
|
describe MergeRequests::SquashService do
|
2018-10-02 03:21:46 +00:00
|
|
|
include GitHelpers
|
|
|
|
|
2019-02-06 12:33:11 +00:00
|
|
|
let(:service) { described_class.new(project, user, { merge_request: merge_request }) }
|
2018-05-29 09:51:43 +00:00
|
|
|
let(:user) { project.owner }
|
|
|
|
let(:project) { create(:project, :repository) }
|
|
|
|
let(:repository) { project.repository.raw }
|
|
|
|
let(:log_error) { "Failed to squash merge request #{merge_request.to_reference(full: true)}:" }
|
|
|
|
let(:squash_dir_path) do
|
|
|
|
File.join(Gitlab.config.shared.path, 'tmp/squash', repository.gl_repository, merge_request.id.to_s)
|
|
|
|
end
|
|
|
|
let(:merge_request_with_one_commit) do
|
|
|
|
create(:merge_request,
|
|
|
|
source_branch: 'feature', source_project: project,
|
|
|
|
target_branch: 'master', target_project: project)
|
|
|
|
end
|
|
|
|
|
|
|
|
let(:merge_request_with_only_new_files) do
|
|
|
|
create(:merge_request,
|
|
|
|
source_branch: 'video', source_project: project,
|
|
|
|
target_branch: 'master', target_project: project)
|
|
|
|
end
|
|
|
|
|
|
|
|
let(:merge_request_with_large_files) do
|
|
|
|
create(:merge_request,
|
|
|
|
source_branch: 'squash-large-files', source_project: project,
|
|
|
|
target_branch: 'master', target_project: project)
|
|
|
|
end
|
|
|
|
|
|
|
|
shared_examples 'the squash succeeds' do
|
|
|
|
it 'returns the squashed commit SHA' do
|
2019-02-06 12:33:11 +00:00
|
|
|
result = service.execute
|
2018-05-29 09:51:43 +00:00
|
|
|
|
|
|
|
expect(result).to match(status: :success, squash_sha: a_string_matching(/\h{40}/))
|
|
|
|
expect(result[:squash_sha]).not_to eq(merge_request.diff_head_sha)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'cleans up the temporary directory' do
|
2019-02-06 12:33:11 +00:00
|
|
|
service.execute
|
2018-05-29 09:51:43 +00:00
|
|
|
|
|
|
|
expect(File.exist?(squash_dir_path)).to be(false)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not keep the branch push event' do
|
2019-02-06 12:33:11 +00:00
|
|
|
expect { service.execute }.not_to change { Event.count }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when there is a single commit in the merge request' do
|
|
|
|
before do
|
|
|
|
expect(merge_request).to receive(:commits_count).at_least(:once).and_return(1)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'will skip performing the squash, as the outcome would be the same' do
|
|
|
|
expect(merge_request.target_project.repository).not_to receive(:squash)
|
|
|
|
|
|
|
|
service.execute
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'will still perform the squash when a custom squash commit message has been provided' do
|
|
|
|
service = described_class.new(project, user, { merge_request: merge_request, squash_commit_message: 'A custom commit message' })
|
|
|
|
|
|
|
|
expect(merge_request.target_project.repository).to receive(:squash).and_return('sha')
|
|
|
|
|
|
|
|
service.execute
|
|
|
|
end
|
2018-05-29 09:51:43 +00:00
|
|
|
end
|
|
|
|
|
|
|
|
context 'the squashed commit' do
|
2019-02-06 12:33:11 +00:00
|
|
|
let(:squash_sha) { service.execute[:squash_sha] }
|
2018-05-29 09:51:43 +00:00
|
|
|
let(:squash_commit) { project.repository.commit(squash_sha) }
|
|
|
|
|
2019-02-06 12:33:11 +00:00
|
|
|
it 'copies the author info from the merge request' do
|
2018-05-29 09:51:43 +00:00
|
|
|
expect(squash_commit.author_name).to eq(merge_request.author.name)
|
|
|
|
expect(squash_commit.author_email).to eq(merge_request.author.email)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'sets the current user as the committer' do
|
|
|
|
expect(squash_commit.committer_name).to eq(user.name.chomp('.'))
|
|
|
|
expect(squash_commit.committer_email).to eq(user.email)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'has the same diff as the merge request, but a different SHA' do
|
2018-10-02 03:21:46 +00:00
|
|
|
rugged = rugged_repo(project.repository)
|
2018-05-29 09:51:43 +00:00
|
|
|
mr_diff = rugged.diff(merge_request.diff_base_sha, merge_request.diff_head_sha)
|
|
|
|
squash_diff = rugged.diff(merge_request.diff_start_sha, squash_sha)
|
|
|
|
|
|
|
|
expect(squash_diff.patch.length).to eq(mr_diff.patch.length)
|
|
|
|
expect(squash_commit.sha).not_to eq(merge_request.diff_head_sha)
|
|
|
|
end
|
2019-02-06 12:33:11 +00:00
|
|
|
|
|
|
|
it 'has a default squash commit message if no message was provided' do
|
|
|
|
expect(squash_commit.message.chomp).to eq(merge_request.default_squash_commit_message.chomp)
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'if a message was provided' do
|
|
|
|
let(:service) { described_class.new(project, user, { merge_request: merge_request, squash_commit_message: message }) }
|
|
|
|
let(:message) { 'My custom message' }
|
|
|
|
let(:squash_sha) { service.execute[:squash_sha] }
|
|
|
|
|
|
|
|
it 'has the same message as the message provided' do
|
|
|
|
expect(squash_commit.message.chomp).to eq(message)
|
|
|
|
end
|
|
|
|
end
|
2018-05-29 09:51:43 +00:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#execute' do
|
|
|
|
context 'when there is only one commit in the merge request' do
|
2019-02-06 12:33:11 +00:00
|
|
|
let(:merge_request) { merge_request_with_one_commit }
|
|
|
|
|
2018-05-29 09:51:43 +00:00
|
|
|
it 'returns that commit SHA' do
|
2019-02-06 12:33:11 +00:00
|
|
|
result = service.execute
|
2018-05-29 09:51:43 +00:00
|
|
|
|
2019-02-06 12:33:11 +00:00
|
|
|
expect(result).to match(status: :success, squash_sha: merge_request.diff_head_sha)
|
2018-05-29 09:51:43 +00:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not perform any git actions' do
|
|
|
|
expect(repository).not_to receive(:popen)
|
|
|
|
|
2019-02-06 12:33:11 +00:00
|
|
|
service.execute
|
2018-05-29 09:51:43 +00:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when squashing only new files' do
|
|
|
|
let(:merge_request) { merge_request_with_only_new_files }
|
|
|
|
|
|
|
|
include_examples 'the squash succeeds'
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when squashing with files too large to display' do
|
|
|
|
let(:merge_request) { merge_request_with_large_files }
|
|
|
|
|
|
|
|
include_examples 'the squash succeeds'
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'git errors' do
|
|
|
|
let(:merge_request) { merge_request_with_only_new_files }
|
|
|
|
let(:error) { 'A test error' }
|
|
|
|
|
|
|
|
context 'with gitaly enabled' do
|
|
|
|
before do
|
|
|
|
allow(repository.gitaly_operation_client).to receive(:user_squash)
|
|
|
|
.and_raise(Gitlab::Git::Repository::GitError, error)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'logs the stage and output' do
|
|
|
|
expect(service).to receive(:log_error).with(log_error)
|
|
|
|
expect(service).to receive(:log_error).with(error)
|
|
|
|
|
2019-02-06 12:33:11 +00:00
|
|
|
service.execute
|
2018-05-29 09:51:43 +00:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns an error' do
|
2019-02-06 12:33:11 +00:00
|
|
|
expect(service.execute).to match(status: :error, message: a_string_including('squash'))
|
2018-05-29 09:51:43 +00:00
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when any other exception is thrown' do
|
|
|
|
let(:merge_request) { merge_request_with_only_new_files }
|
|
|
|
let(:error) { 'A test error' }
|
|
|
|
|
|
|
|
before do
|
2019-02-06 12:33:11 +00:00
|
|
|
allow(merge_request.target_project.repository).to receive(:squash).and_raise(error)
|
2018-05-29 09:51:43 +00:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'logs the MR reference and exception' do
|
|
|
|
expect(service).to receive(:log_error).with(a_string_including("#{project.full_path}#{merge_request.to_reference}"))
|
|
|
|
expect(service).to receive(:log_error).with(error)
|
|
|
|
|
2019-02-06 12:33:11 +00:00
|
|
|
service.execute
|
2018-05-29 09:51:43 +00:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns an error' do
|
2019-02-06 12:33:11 +00:00
|
|
|
expect(service.execute).to match(status: :error, message: a_string_including('squash'))
|
2018-05-29 09:51:43 +00:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'cleans up the temporary directory' do
|
2019-02-06 12:33:11 +00:00
|
|
|
service.execute
|
2018-05-29 09:51:43 +00:00
|
|
|
|
|
|
|
expect(File.exist?(squash_dir_path)).to be(false)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|