Commit e4584330 authored by Douwe Maan's avatar Douwe Maan

Merge branch 'backport-destroy-service-spec-fixes' into 'master'

Backport from EE: Fixed DestroyService spec to actually test if removal was successful

See merge request !12437
parents 4d6ed39b b6118afd
...@@ -15,8 +15,9 @@ describe Projects::DestroyService, services: true do ...@@ -15,8 +15,9 @@ describe Projects::DestroyService, services: true do
shared_examples 'deleting the project' do shared_examples 'deleting the project' do
it 'deletes the project' do it 'deletes the project' do
expect(Project.unscoped.all).not_to include(project) expect(Project.unscoped.all).not_to include(project)
expect(Dir.exist?(path)).to be_falsey
expect(Dir.exist?(remove_path)).to be_falsey expect(project.gitlab_shell.exists?(project.repository_storage_path, path + '.git')).to be_falsey
expect(project.gitlab_shell.exists?(project.repository_storage_path, remove_path + '.git')).to be_falsey
end end
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment