diff options
| author | Sean McGivern <sean@gitlab.com> | 2017-06-08 19:10:35 +0300 |
|---|---|---|
| committer | Sean McGivern <sean@gitlab.com> | 2017-06-15 17:15:13 +0300 |
| commit | e94c1028c1e829f899b0d0f56313cc62df6f9a0a (patch) | |
| tree | d69f40f69e1ed69ce5c2c6d7fe2367ccca7cdce1 /spec/services/notification_service_spec.rb | |
| parent | f4b5fcbca100769b89e6b06e0df180012d16a7a8 (diff) | |
Deserialise existing custom notification settings
Create a post-deployment migration to update all existing notification settings
with at least one custom level enabled to the new format. Also handle the same
conversion when updating settings, to catch any stragglers.
Diffstat (limited to 'spec/services/notification_service_spec.rb')
| -rw-r--r-- | spec/services/notification_service_spec.rb | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/spec/services/notification_service_spec.rb b/spec/services/notification_service_spec.rb index de3bbc6b6a1..f1e00c1163b 100644 --- a/spec/services/notification_service_spec.rb +++ b/spec/services/notification_service_spec.rb @@ -1539,8 +1539,7 @@ describe NotificationService, services: true do # When resource is nil it means global notification def update_custom_notification(event, user, resource: nil, value: true) setting = user.notification_settings_for(resource) - setting.events[event] = value - setting.save + setting.update!(event => value) end def add_users_with_subscription(project, issuable) |