Welcome to mirror list, hosted at ThFree Co, Russian Federation.

gitlab.com/gitlab-org/gitlab-foss.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTiago Botelho <tiagonbotelho@hotmail.com>2017-07-18 18:09:14 +0300
committerTiago Botelho <tiagonbotelho@hotmail.com>2017-07-26 14:47:50 +0300
commitb5bdc55d239f3e19f8fe1e59b118da05ac81a0dd (patch)
tree507f7b46386d518cdb7b77fa4048d7f4d77eec37 /spec/workers/project_destroy_worker_spec.rb
parent0aa8249e484ca97cfc28c7301d69077919032c08 (diff)
Move exception handling to execute
Diffstat (limited to 'spec/workers/project_destroy_worker_spec.rb')
-rw-r--r--spec/workers/project_destroy_worker_spec.rb19
1 files changed, 9 insertions, 10 deletions
diff --git a/spec/workers/project_destroy_worker_spec.rb b/spec/workers/project_destroy_worker_spec.rb
index 29f0295de42..f19c9dff941 100644
--- a/spec/workers/project_destroy_worker_spec.rb
+++ b/spec/workers/project_destroy_worker_spec.rb
@@ -21,17 +21,16 @@ describe ProjectDestroyWorker do
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)
+ it 'does not raise error when project could not be found' do
+ expect do
+ subject.perform(-1, project.owner.id, {})
+ end.not_to raise_error
+ end
- expect(Project.all).to include(project)
- expect(project.delete_error).to eq("some error message")
- end
+ it 'does not raise error when user could not be found' do
+ expect do
+ subject.perform(project.id, -1, {})
+ end.not_to raise_error
end
end
end