diff options
author | Yorick Peterse <yorickpeterse@gmail.com> | 2017-04-19 14:53:04 +0300 |
---|---|---|
committer | James Lopez <james@jameslopez.es> | 2017-04-19 16:27:19 +0300 |
commit | 930e8c03a2da6c6dd55dd67ebbd2be3589e47b02 (patch) | |
tree | e08b138a5671241b9f9226b739e22f615b111313 /spec/services | |
parent | 31f94d25c215d6d8377306e56140f1845944e528 (diff) |
Merge branch 'fix/orphan-notification-settings' into 'master'
Fix orphaned notification settings
Closes #29688
See merge request !10763
Diffstat (limited to 'spec/services')
-rw-r--r-- | spec/services/groups/destroy_service_spec.rb | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/spec/services/groups/destroy_service_spec.rb b/spec/services/groups/destroy_service_spec.rb index 98c560ffb26..a37257d1bf4 100644 --- a/spec/services/groups/destroy_service_spec.rb +++ b/spec/services/groups/destroy_service_spec.rb @@ -6,7 +6,8 @@ describe Groups::DestroyService, services: true do let!(:user) { create(:user) } let!(:group) { create(:group) } let!(:nested_group) { create(:group, parent: group) } - let!(:project) { create(:project, namespace: group) } + let!(:project) { create(:empty_project, namespace: group) } + let!(:notification_setting) { create(:notification_setting, source: group)} let!(:gitlab_shell) { Gitlab::Shell.new } let!(:remove_path) { group.path + "+#{group.id}+deleted" } @@ -23,6 +24,7 @@ describe Groups::DestroyService, services: true do it { expect(Group.unscoped.all).not_to include(group) } it { expect(Group.unscoped.all).not_to include(nested_group) } it { expect(Project.unscoped.all).not_to include(project) } + it { expect(NotificationSetting.unscoped.all).not_to include(notification_setting) } end context 'file system' do |