From 180bfecb39fe0fd9da54a100532da8fb932c7cf2 Mon Sep 17 00:00:00 2001 From: Juan Antonio Osorio Robles Date: Fri, 10 Mar 2017 13:47:48 +0200 Subject: [PATCH] rabbitmq SSL: check puppet resource instead of actual config The configuration is done via the oslo::messaging::rabbit resource from puppet-oslo. We should be checking for the resource that we're using instead of the underlying configuration file. Change-Id: I2f0a392e4b0a7a7ab61cc1e853fca44c3b9cbe97 --- spec/classes/ceilometer_init_spec.rb | 32 +++++++++++++++------------- 1 file changed, 17 insertions(+), 15 deletions(-) diff --git a/spec/classes/ceilometer_init_spec.rb b/spec/classes/ceilometer_init_spec.rb index 3a3e2a3..011d9bb 100644 --- a/spec/classes/ceilometer_init_spec.rb +++ b/spec/classes/ceilometer_init_spec.rb @@ -271,11 +271,13 @@ describe 'ceilometer' do shared_examples_for 'rabbit with SSL support' do context "with default parameters" do - it { is_expected.to contain_ceilometer_config('oslo_messaging_rabbit/rabbit_use_ssl').with_value('') } - it { is_expected.to contain_ceilometer_config('oslo_messaging_rabbit/kombu_ssl_ca_certs').with_value('') } - it { is_expected.to contain_ceilometer_config('oslo_messaging_rabbit/kombu_ssl_certfile').with_value('') } - it { is_expected.to contain_ceilometer_config('oslo_messaging_rabbit/kombu_ssl_keyfile').with_value('') } - it { is_expected.to contain_ceilometer_config('oslo_messaging_rabbit/kombu_ssl_version').with_value('') } + it { is_expected.to contain_oslo__messaging__rabbit('ceilometer_config').with( + :rabbit_use_ssl => '', + :kombu_ssl_ca_certs => '', + :kombu_ssl_certfile => '', + :kombu_ssl_keyfile => '', + :kombu_ssl_version => '', + )} end context "with SSL enabled with kombu" do @@ -287,11 +289,13 @@ describe 'ceilometer' do :kombu_ssl_version => 'TLSv1' ) } - it { is_expected.to contain_ceilometer_config('oslo_messaging_rabbit/rabbit_use_ssl').with_value(true) } - it { is_expected.to contain_ceilometer_config('oslo_messaging_rabbit/kombu_ssl_ca_certs').with_value('/path/to/ca.crt') } - it { is_expected.to contain_ceilometer_config('oslo_messaging_rabbit/kombu_ssl_certfile').with_value('/path/to/cert.crt') } - it { is_expected.to contain_ceilometer_config('oslo_messaging_rabbit/kombu_ssl_keyfile').with_value('/path/to/cert.key') } - it { is_expected.to contain_ceilometer_config('oslo_messaging_rabbit/kombu_ssl_version').with_value('TLSv1') } + it { is_expected.to contain_oslo__messaging__rabbit('ceilometer_config').with( + :rabbit_use_ssl => true, + :kombu_ssl_ca_certs => '/path/to/ca.crt', + :kombu_ssl_certfile => '/path/to/cert.crt', + :kombu_ssl_keyfile => '/path/to/cert.key', + :kombu_ssl_version => 'TLSv1' + )} end context "with SSL enabled without kombu" do @@ -299,11 +303,9 @@ describe 'ceilometer' do :rabbit_use_ssl => true ) } - it { is_expected.to contain_ceilometer_config('oslo_messaging_rabbit/rabbit_use_ssl').with_value(true) } - it { is_expected.to contain_ceilometer_config('oslo_messaging_rabbit/kombu_ssl_ca_certs').with_value('') } - it { is_expected.to contain_ceilometer_config('oslo_messaging_rabbit/kombu_ssl_certfile').with_value('') } - it { is_expected.to contain_ceilometer_config('oslo_messaging_rabbit/kombu_ssl_keyfile').with_value('') } - it { is_expected.to contain_ceilometer_config('oslo_messaging_rabbit/kombu_ssl_version').with_value('') } + it { is_expected.to contain_oslo__messaging__rabbit('ceilometer_config').with( + :rabbit_use_ssl => true, + )} end context "with SSL wrongly configured" do -- 2.45.2