Merge pull request #303 from dpeters/sort_properties

Sort array properties before comparison

lib/puppet/type/sensu_check.rb

@@ -38,10 +38,16 @@ def initialize(*args)
 
   newproperty(:dependencies, :array_matching => :all) do
     desc "Dependencies of this check"
+    def insync?(is)
+      is.sort == should.sort
+    end
   end
 
   newproperty(:handlers, :array_matching => :all) do
     desc "List of handlers that responds to this check"
+    def insync?(is)
+      is.sort == should.sort
+    end
   end
 
   newproperty(:high_flap_threshold) do
@@ -71,6 +77,9 @@ def initialize(*args)
 
   newproperty(:subscribers, :array_matching => :all) do
     desc "Who is subscribed to this check"
+    def insync?(is)
+      is.sort == should.sort
+    end
   end
 
   newproperty(:custom) do

lib/puppet/type/sensu_client_config.rb

@@ -41,6 +41,9 @@ def initialize(*args)
 
   newproperty(:subscriptions, :array_matching => :all) do
     desc ""
+    def insync?(is)
+      is.sort == should.sort
+    end
   end
 
   newproperty(:bind) do

lib/puppet/type/sensu_handler.rb

@@ -70,14 +70,23 @@ def insync?(is)
 
   newproperty(:filters, :array_matching => :all) do
     desc "Handler filters"
+    def insync?(is)
+      is.sort == should.sort
+    end
   end
 
   newproperty(:severities, :array_matching => :all) do
     desc "Severities applicable to this handler"
+    def insync?(is)
+      is.sort == should.sort
+    end
   end
 
   newproperty(:handlers, :array_matching => :all) do
     desc "Handlers this handler mutexes into"
+    def insync?(is)
+      is.sort == should.sort
+    end
   end
 
   newproperty(:config) do