X-Git-Url: https://git.adam-barratt.org.uk/?a=blobdiff_plain;f=3rdparty%2Fmodules%2Frabbitmq%2Fspec%2Facceptance%2Fpolicy_spec.rb;h=91161a40edd28a22ca06221becfd0fb9fe36c7fc;hb=921e69100a563cf143f56a3905d8362336d939ff;hp=26858ecc5047ef18c22d2be17ff6df9605f280e7;hpb=b54f52d2899c5785923c804fdfbba0782c147da4;p=mirror%2Fdsa-puppet.git

diff --git a/3rdparty/modules/rabbitmq/spec/acceptance/policy_spec.rb b/3rdparty/modules/rabbitmq/spec/acceptance/policy_spec.rb
index 26858ecc5..91161a40e 100644
--- a/3rdparty/modules/rabbitmq/spec/acceptance/policy_spec.rb
+++ b/3rdparty/modules/rabbitmq/spec/acceptance/policy_spec.rb
@@ -1,18 +1,16 @@
 require 'spec_helper_acceptance'
 
 describe 'rabbitmq policy on a vhost:' do
-
-
-  context "create policy resource" do
-    it 'should run successfully' do
+  context 'create policy resource' do
+    it 'runs successfully' do
       pp = <<-EOS
-      if $::osfamily == 'RedHat' {
+      if $facts['os']['family'] == 'RedHat' {
         class { 'erlang': epel_enable => true }
-        Class['erlang'] -> Class['::rabbitmq']
+        Class['erlang'] -> Class['rabbitmq']
       }
-      class { '::rabbitmq':
+      class { 'rabbitmq':
         service_manage    => true,
-        port              => '5672',
+        port              => 5672,
         delete_guest_user => true,
         admin_enable      => true,
       } ->
@@ -30,18 +28,33 @@ describe 'rabbitmq policy on a vhost:' do
           'ha-sync-mode' => 'automatic',
         },
       }
+
+      rabbitmq_policy { 'eu-federation@myhost':
+        pattern    => '^eu\\.',
+        priority   => 0,
+        applyto    => 'all',
+        definition => {
+          'federation-upstream-set' => 'all',
+        },
+      }
       EOS
 
-      apply_manifest(pp, :catch_failures => true)
-      apply_manifest(pp, :catch_changes => true)
+      apply_manifest(pp, catch_failures: true)
+      apply_manifest(pp, catch_changes: true)
+
+      # Apply twice to ensure no changes the second time.
+      apply_manifest(pp, catch_failures: true)
+      expect(apply_manifest(pp, catch_changes: true).exit_code).to be_zero
     end
 
-    it 'should have the policy' do
+    # rubocop:disable RSpec/MultipleExpectations
+    it 'has the policy' do
       shell('rabbitmqctl list_policies -p myhost') do |r|
-        expect(r.stdout).to match(/myhost.*ha-all.*ha-sync-mode/)
+        expect(r.stdout).to match(%r{myhost.*ha-all.*ha-sync-mode})
+        expect(r.stdout).to match(%r{myhost.*eu-federation})
         expect(r.exit_code).to be_zero
       end
     end
-
+    # rubocop:enable RSpec/MultipleExpectations
   end
 end