2016-09-13 17:45:13 +05:30
|
|
|
require "spec_helper"
|
|
|
|
|
|
|
|
describe Files::UpdateService do
|
|
|
|
subject { described_class.new(project, user, commit_params) }
|
|
|
|
|
2017-08-17 22:00:37 +05:30
|
|
|
let(:project) { create(:project, :repository) }
|
2016-09-13 17:45:13 +05:30
|
|
|
let(:user) { create(:user) }
|
|
|
|
let(:file_path) { 'files/ruby/popen.rb' }
|
2017-08-17 22:00:37 +05:30
|
|
|
let(:new_contents) { 'New Content' }
|
|
|
|
let(:branch_name) { project.default_branch }
|
|
|
|
let(:last_commit_sha) { nil }
|
|
|
|
|
2016-09-13 17:45:13 +05:30
|
|
|
let(:commit_params) do
|
|
|
|
{
|
|
|
|
file_path: file_path,
|
|
|
|
commit_message: "Update File",
|
|
|
|
file_content: new_contents,
|
|
|
|
file_content_encoding: "text",
|
|
|
|
last_commit_sha: last_commit_sha,
|
2017-08-17 22:00:37 +05:30
|
|
|
start_project: project,
|
|
|
|
start_branch: project.default_branch,
|
|
|
|
branch_name: branch_name
|
2016-09-13 17:45:13 +05:30
|
|
|
}
|
|
|
|
end
|
|
|
|
|
|
|
|
before do
|
|
|
|
project.team << [user, :master]
|
|
|
|
end
|
|
|
|
|
|
|
|
describe "#execute" do
|
|
|
|
context "when the file's last commit sha does not match the supplied last_commit_sha" do
|
|
|
|
let(:last_commit_sha) { "foo" }
|
|
|
|
|
|
|
|
it "returns a hash with the correct error message and a :error status " do
|
2017-09-10 17:25:29 +05:30
|
|
|
expect { subject.execute }
|
|
|
|
.to raise_error(Files::UpdateService::FileChangedError,
|
2016-09-13 17:45:13 +05:30
|
|
|
"You are attempting to update a file that has changed since you started editing it.")
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context "when the file's last commit sha does match the supplied last_commit_sha" do
|
|
|
|
let(:last_commit_sha) { Gitlab::Git::Commit.last_for_path(project.repository, project.default_branch, file_path).sha }
|
|
|
|
|
|
|
|
it "returns a hash with the :success status " do
|
|
|
|
results = subject.execute
|
|
|
|
|
2016-11-03 12:29:30 +05:30
|
|
|
expect(results[:status]).to match(:success)
|
2016-09-13 17:45:13 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
it "updates the file with the new contents" do
|
|
|
|
subject.execute
|
|
|
|
|
|
|
|
results = project.repository.blob_at_branch(project.default_branch, file_path)
|
|
|
|
|
|
|
|
expect(results.data).to eq(new_contents)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context "when the last_commit_sha is not supplied" do
|
|
|
|
it "returns a hash with the :success status " do
|
|
|
|
results = subject.execute
|
|
|
|
|
2016-11-03 12:29:30 +05:30
|
|
|
expect(results[:status]).to match(:success)
|
2016-09-13 17:45:13 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
it "updates the file with the new contents" do
|
|
|
|
subject.execute
|
|
|
|
|
|
|
|
results = project.repository.blob_at_branch(project.default_branch, file_path)
|
|
|
|
|
|
|
|
expect(results.data).to eq(new_contents)
|
|
|
|
end
|
|
|
|
end
|
2017-08-17 22:00:37 +05:30
|
|
|
|
|
|
|
context 'when target branch is different than source branch' do
|
|
|
|
let(:branch_name) { "#{project.default_branch}-new" }
|
|
|
|
|
|
|
|
it 'fires hooks only once' do
|
|
|
|
expect(GitHooksService).to receive(:new).once.and_call_original
|
|
|
|
|
|
|
|
subject.execute
|
|
|
|
end
|
|
|
|
end
|
2016-09-13 17:45:13 +05:30
|
|
|
end
|
|
|
|
end
|