accepts `.all` as a hook scope as well

1 files changed, 9 insertions, 1 deletions

diff --git a/spec/models/concerns/triggerable_hooks_spec.rb b/spec/models/concerns/triggerable_hooks_spec.rb
index 3ddf477e41e..bea846454de 100644
--- a/spec/models/concerns/triggerable_hooks_spec.rb
+++ b/spec/models/concerns/triggerable_hooks_spec.rb
@@ -26,10 +26,18 @@ RSpec.describe TriggerableHooks do
 
     context 'the model does not have the required trigger scope' do
       it 'returns an empty relation' do
-        TestableHook.create!(url: 'http://example.com', push_events: true)
+        TestableHook.create!(url: 'http://example.com')
 
         expect(TestableHook.hooks_for(:tag_push_hooks)).to eq []
       end
     end
+
+    context 'the stock scope ".all" is accepted' do
+      it 'returns the record' do
+        hook = TestableHook.create!(url: 'http://example.com')
+
+        expect(TestableHook.hooks_for(:all)).to eq [hook]
+      end
+    end
   end
 end