diff options
author | Sean McGivern <sean@gitlab.com> | 2018-06-08 11:45:36 +0100 |
---|---|---|
committer | Sean McGivern <sean@gitlab.com> | 2018-06-08 11:45:36 +0100 |
commit | f413c4dd267f3a5906bf6c5b3c4eb3b45f8ac60a (patch) | |
tree | 9c0d229264d01699b415e715ed3a7881e6cfbf21 /spec | |
parent | 2c05c8574f9bdef89008381754aee9566b58ad86 (diff) | |
download | gitlab-ce-f413c4dd267f3a5906bf6c5b3c4eb3b45f8ac60a.tar.gz |
Fix NotificationRecipientService spec for EE
EE checks a license, which needs RequestStore enabled to avoid N+1
queries. However, enabling RequestStore causes Gitaly to complain about N+1
invocations, which we really don't care about here.
Diffstat (limited to 'spec')
-rw-r--r-- | spec/services/notification_recipient_service_spec.rb | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/spec/services/notification_recipient_service_spec.rb b/spec/services/notification_recipient_service_spec.rb index 340d4585e0c..7f536ce4e68 100644 --- a/spec/services/notification_recipient_service_spec.rb +++ b/spec/services/notification_recipient_service_spec.rb @@ -19,8 +19,8 @@ describe NotificationRecipientService do end end - it 'avoids N+1 queries' do - create_watcher + it 'avoids N+1 queries', :request_store do + Gitlab::GitalyClient.allow_n_plus_1_calls { create_watcher } service.build_new_note_recipients(note) @@ -28,7 +28,7 @@ describe NotificationRecipientService do service.build_new_note_recipients(note) end - create_watcher + Gitlab::GitalyClient.allow_n_plus_1_calls { create_watcher } expect { service.build_new_note_recipients(note) }.not_to exceed_query_limit(control_count) end |