Commit d249da1c authored by Zeger-Jan van de Weg's avatar Zeger-Jan van de Weg

Specs on force deleting of a user

parent b221d11a
...@@ -12,7 +12,7 @@ class DeleteUserService ...@@ -12,7 +12,7 @@ class DeleteUserService
end end
user.solo_owned_groups.each do |group| user.solo_owned_groups.each do |group|
DeleteGroupService.new(group, current_user).execute DestroyGroupService.new(group, current_user).execute
end end
user.personal_projects.each do |project| user.personal_projects.each do |project|
......
...@@ -7,16 +7,52 @@ describe DeleteUserWorker do ...@@ -7,16 +7,52 @@ describe DeleteUserWorker do
let!(:namespace) { create(:namespace, owner: user) } let!(:namespace) { create(:namespace, owner: user) }
let!(:project) { create(:project, namespace: namespace) } let!(:project) { create(:project, namespace: namespace) }
before do context 'no force flag given' do
DeleteUserWorker.new.perform(current_user.id, user.id) before do
DeleteUserWorker.new.perform(current_user.id, user.id)
end
it 'deletes all personal projects' do
expect { Project.find(project.id) }.to raise_error(ActiveRecord::RecordNotFound)
end
it 'deletes the user' do
expect { User.find(user.id) }.to raise_error(ActiveRecord::RecordNotFound)
end
end end
it 'deletes all personal projects' do context "solo owned groups present" do
expect { Project.find(project.id) }.to raise_error(ActiveRecord::RecordNotFound) let(:solo_owned) { create(:group) }
let(:member) { create(:group_member) }
let(:user) { user = member.user }
before do
solo_owned.group_members = [member]
DeleteUserWorker.new.perform(current_user.id, user.id)
end
it 'does not delete the user' do
expect(User.find(user.id)).to eq user
end
end end
it 'deletes the user' do context "deletions with force" do
expect { User.find(user.id) }.to raise_error(ActiveRecord::RecordNotFound) let(:solo_owned) { create(:group) }
let(:member) { create(:group_member) }
let(:user) { user = member.user }
before do
solo_owned.group_members = [member]
DeleteUserWorker.new.perform(current_user.id, user.id, "force" => true)
end
it 'deletes solo owned groups' do
expect { Project.find(solo_owned.id) }.to raise_error(ActiveRecord::RecordNotFound)
end
it 'deletes the user' do
expect { User.find(user.id) }.to raise_error(ActiveRecord::RecordNotFound)
end
end end
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