Default should be a list.

sensu-plugins · Jul 5, 2016 · 8c36ba9 · 8c36ba9
1 parent 59ae2ef
commit 8c36ba9
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/bin/handler-pagerduty.rb b/bin/handler-pagerduty.rb
@@ -68,7 +68,7 @@ def proxy_settings
   end
 
   def contexts
-    @contexts ||= @event['check']['pagerduty_contexts'] || {}
+    @contexts ||= @event['check']['pagerduty_contexts'] || []
   end
 
   def handle(pd_client = nil)

diff --git a/test/bin/handler-pagerduty_spec.rb b/test/bin/handler-pagerduty_spec.rb
@@ -80,6 +80,12 @@ def settings
   end
 
   describe '#contexts' do
+    it 'should return list as default' do
+      io_obj = fixture('minimal_create.json')
+      @handler.read_event(io_obj)
+      expect(@handler.contexts).to eq([])
+    end
+
     it 'should return contexts hash from check' do
       io_obj = fixture('check_with_contexts.json')
       @handler.read_event(io_obj)