2019-04-11 08:17:24 -04:00
|
|
|
# frozen_string_literal: true
|
|
|
|
|
2017-12-20 10:57:27 -05:00
|
|
|
require "spec_helper"
|
|
|
|
|
2020-06-24 11:08:50 -04:00
|
|
|
RSpec.describe Files::DeleteService do
|
2017-12-20 10:57:27 -05:00
|
|
|
subject { described_class.new(project, user, commit_params) }
|
|
|
|
|
|
|
|
let(:project) { create(:project, :repository) }
|
2018-09-18 05:04:15 -04:00
|
|
|
let(:user) { create(:user, :commit_email) }
|
2017-12-20 10:57:27 -05:00
|
|
|
let(:file_path) { 'files/ruby/popen.rb' }
|
|
|
|
let(:branch_name) { project.default_branch }
|
|
|
|
let(:last_commit_sha) { nil }
|
2018-09-17 09:14:58 -04:00
|
|
|
let(:commit) { project.repository.head_commit }
|
2017-12-20 10:57:27 -05:00
|
|
|
|
|
|
|
let(:commit_params) do
|
|
|
|
{
|
|
|
|
file_path: file_path,
|
|
|
|
commit_message: "Delete File",
|
|
|
|
last_commit_sha: last_commit_sha,
|
|
|
|
start_project: project,
|
|
|
|
start_branch: project.default_branch,
|
|
|
|
branch_name: branch_name
|
|
|
|
}
|
|
|
|
end
|
|
|
|
|
|
|
|
shared_examples 'successfully deletes the file' do
|
|
|
|
it 'returns a hash with the :success status' do
|
|
|
|
results = subject.execute
|
|
|
|
|
|
|
|
expect(results[:status]).to match(:success)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'deletes the file' do
|
|
|
|
subject.execute
|
|
|
|
|
|
|
|
blob = project.repository.blob_at_branch(project.default_branch, file_path)
|
|
|
|
|
|
|
|
expect(blob).to be_nil
|
|
|
|
end
|
2018-09-17 09:14:58 -04:00
|
|
|
|
|
|
|
it 'uses the commit email' do
|
|
|
|
subject.execute
|
|
|
|
|
2018-09-18 05:04:15 -04:00
|
|
|
expect(user.commit_email).not_to eq(user.email)
|
2018-09-17 09:14:58 -04:00
|
|
|
expect(commit.author_email).to eq(user.commit_email)
|
|
|
|
expect(commit.committer_email).to eq(user.commit_email)
|
|
|
|
end
|
2017-12-20 10:57:27 -05:00
|
|
|
end
|
|
|
|
|
|
|
|
before do
|
2018-07-11 10:36:08 -04:00
|
|
|
project.add_maintainer(user)
|
2017-12-20 10:57:27 -05:00
|
|
|
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" }
|
|
|
|
|
2020-12-09 13:09:48 -05:00
|
|
|
it "returns a hash with the correct error message and a :error status" do
|
2017-12-20 10:57:27 -05:00
|
|
|
expect { subject.execute }
|
|
|
|
.to raise_error(Files::UpdateService::FileChangedError,
|
|
|
|
"You are attempting to delete a file that has been previously updated.")
|
|
|
|
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_behaves_like 'successfully deletes the file'
|
|
|
|
end
|
|
|
|
|
|
|
|
context "when the last_commit_sha is not supplied" do
|
|
|
|
it_behaves_like 'successfully deletes the file'
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|