diff options
author | Tiago Botelho <tiagonbotelho@hotmail.com> | 2017-06-29 12:43:01 +0100 |
---|---|---|
committer | Tiago Botelho <tiagonbotelho@hotmail.com> | 2017-07-20 09:59:56 +0100 |
commit | 3491b19a4e67a9f439c12afac45ef38f3fce0ef5 (patch) | |
tree | a548a111e88c046315e0a2633aba5656be7603ba /spec/workers | |
parent | f0e4e3993b1f5a21ab61aaff95f73ac4e5b88ad3 (diff) | |
download | gitlab-ce-3491b19a4e67a9f439c12afac45ef38f3fce0ef5.tar.gz |
Add specs for ProjectDestroyWorker
Diffstat (limited to 'spec/workers')
-rw-r--r-- | spec/workers/project_destroy_worker_spec.rb | 21 |
1 files changed, 17 insertions, 4 deletions
diff --git a/spec/workers/project_destroy_worker_spec.rb b/spec/workers/project_destroy_worker_spec.rb index 3d135f40c1f..29f0295de42 100644 --- a/spec/workers/project_destroy_worker_spec.rb +++ b/spec/workers/project_destroy_worker_spec.rb @@ -1,24 +1,37 @@ require 'spec_helper' describe ProjectDestroyWorker do - let(:project) { create(:project, :repository) } + let(:project) { create(:project, :repository, pending_delete: true) } let(:path) { project.repository.path_to_repo } subject { described_class.new } - describe "#perform" do - it "deletes the project" do + describe '#perform' do + it 'deletes the project' do subject.perform(project.id, project.owner.id, {}) expect(Project.all).not_to include(project) expect(Dir.exist?(path)).to be_falsey end - it "deletes the project but skips repo deletion" do + it 'deletes the project but skips repo deletion' do subject.perform(project.id, project.owner.id, { "skip_repo" => true }) expect(Project.all).not_to include(project) expect(Dir.exist?(path)).to be_truthy end + + describe 'when StandardError is raised' do + it 'reverts pending_delete attribute with a error message' do + allow_any_instance_of(::Projects::DestroyService).to receive(:execute).and_raise(StandardError, "some error message") + + expect do + subject.perform(project.id, project.owner.id, {}) + end.to change { project.reload.pending_delete }.from(true).to(false) + + expect(Project.all).to include(project) + expect(project.delete_error).to eq("some error message") + end + end end end |