Merge pull request #538 from cwjohnston/fix/allow-delete-without-vali…

…dation

[sensu_check] validate subscribers/standalone on create action only

resources/check.rb

@@ -23,5 +23,7 @@ def after_created
     raise Chef::Exceptions::ValidationFailed, "Sensu check #{name}: name cannot contain spaces or special characters"
   end
 
-  raise Chef::Exceptions::ValidationFailed, "Sensu check #{name}: must either define subscribers, or has to be standalone." unless (subscribers || standalone)
+  if action == :create
+     raise Chef::Exceptions::ValidationFailed, "Sensu check #{name}: must either define subscribers, or has to be standalone." unless (subscribers || standalone)
+  end
 end

test/cookbooks/sensu-test/recipes/good_checks.rb

@@ -1,4 +1,3 @@
-
 sensu_check "valid_standalone_check" do
   interval 20
   command 'true'
@@ -10,3 +9,7 @@
   command 'true'
   subscribers ['all']
 end
+
+sensu_check "removed_check" do
+  action :delete
+end

test/unit/check_spec.rb

@@ -15,6 +15,13 @@
     expect(chef_run).to create_sensu_check("valid_pubsub_check").with(:subscribers => ['all'])
   end
 
+  it "deletes removed_check without specifying subscriptions/standalone" do
+    expect(chef_run).to delete_sensu_check("removed_check").with(
+      :subscribers => nil,
+      :standalone => nil
+    )
+  end
+
 end
 
 describe 'sensu-test::bad_check_name' do