From: Mykyta Karpin <mkarpin@mirantis.com>
Date: Fri, 11 Nov 2016 17:03:11 +0000 (+0200)
Subject: Switch ceilometer to rspec-puppet-facts
X-Git-Tag: 10.0.0~3
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=52ebcc955d0b7c9f737710c266dc60be8ba559ea;p=puppet-modules%2Fpuppet-ceilometer.git

Switch ceilometer to rspec-puppet-facts

Change-Id: I5befcadb924f0896457f3db0b712f6de157e926c
---

diff --git a/spec/classes/ceilometer_agent_auth_spec.rb b/spec/classes/ceilometer_agent_auth_spec.rb
index 72a4d81..97dc0d6 100644
--- a/spec/classes/ceilometer_agent_auth_spec.rb
+++ b/spec/classes/ceilometer_agent_auth_spec.rb
@@ -49,20 +49,16 @@ describe 'ceilometer::agent::auth' do
 
   end
 
-  context 'on Debian platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'Debian' })
-    end
-
-    it_configures 'ceilometer-agent-auth'
-  end
+  on_supported_os({
+    :supported_os => OSDefaults.get_supported_os
+  }).each do |os,facts|
+    context "on #{os}" do
+      let (:facts) do
+        facts.merge!(OSDefaults.get_facts())
+      end
 
-  context 'on RedHat platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'RedHat' })
+      it_behaves_like 'ceilometer-agent-auth'
     end
-
-    it_configures 'ceilometer-agent-auth'
   end
 
 end
diff --git a/spec/classes/ceilometer_agent_central_spec.rb b/spec/classes/ceilometer_agent_central_spec.rb
index ddf87d1..b22024c 100644
--- a/spec/classes/ceilometer_agent_central_spec.rb
+++ b/spec/classes/ceilometer_agent_central_spec.rb
@@ -77,29 +77,27 @@ describe 'ceilometer::agent::central' do
 
   end
 
-  context 'on Debian platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'Debian' })
-    end
-
-    let :platform_params do
-      { :agent_package_name => 'ceilometer-agent-central',
-        :agent_service_name => 'ceilometer-agent-central' }
-    end
-
-    it_configures 'ceilometer-agent-central'
-  end
+  on_supported_os({
+    :supported_os => OSDefaults.get_supported_os
+  }).each do |os,facts|
+    context "on #{os}" do
+      let (:facts) do
+        facts.merge!(OSDefaults.get_facts())
+      end
 
-  context 'on RedHat platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'RedHat' })
-    end
+      let :platform_params do
+        case facts[:osfamily]
+        when 'Debian'
+          { :agent_package_name => 'ceilometer-agent-central',
+            :agent_service_name => 'ceilometer-agent-central' }
+        when 'RedHat'
+          { :agent_package_name => 'openstack-ceilometer-central',
+            :agent_service_name => 'openstack-ceilometer-central' }
+        end
+      end
 
-    let :platform_params do
-      { :agent_package_name => 'openstack-ceilometer-central',
-        :agent_service_name => 'openstack-ceilometer-central' }
+      it_behaves_like 'ceilometer-agent-central'
     end
-
-    it_configures 'ceilometer-agent-central'
   end
+
 end
diff --git a/spec/classes/ceilometer_agent_compute_spec.rb b/spec/classes/ceilometer_agent_compute_spec.rb
index 7649729..5dd6ad1 100644
--- a/spec/classes/ceilometer_agent_compute_spec.rb
+++ b/spec/classes/ceilometer_agent_compute_spec.rb
@@ -88,44 +88,34 @@ describe 'ceilometer::agent::compute' do
 
   end
 
-  context 'on Debian platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'Debian' })
-    end
-
-    let :platform_params do
-      { :agent_package_name => 'ceilometer-agent-compute',
-        :agent_service_name => 'ceilometer-agent-compute' }
-    end
-
-    context 'on Ubuntu operating systems' do
-      before do
-        facts.merge!( :operatingsystem => 'Ubuntu' )
-        platform_params.merge!( :libvirt_group => 'libvirtd' )
+  on_supported_os({
+    :supported_os => OSDefaults.get_supported_os
+  }).each do |os,facts|
+    context "on #{os}" do
+      let (:facts) do
+        facts.merge!(OSDefaults.get_facts())
       end
 
-      it_configures 'ceilometer-agent-compute'
-    end
-
-    context 'on other operating systems' do
-      before do
-        platform_params.merge!( :libvirt_group => 'libvirt' )
+      let :platform_params do
+        case facts[:osfamily]
+        when 'Debian'
+          if facts[:operatingsystem] == 'Ubuntu'
+            { :agent_package_name => 'ceilometer-agent-compute',
+              :agent_service_name => 'ceilometer-agent-compute',
+              :libvirt_group      => 'libvirtd' }
+          else
+            { :agent_package_name => 'ceilometer-agent-compute',
+              :agent_service_name => 'ceilometer-agent-compute',
+              :libvirt_group      => 'libvirt' }
+          end
+        when 'RedHat'
+          { :agent_package_name => 'openstack-ceilometer-compute',
+            :agent_service_name => 'openstack-ceilometer-compute' }
+        end
       end
 
-      it_configures 'ceilometer-agent-compute'
+      it_behaves_like 'ceilometer-agent-compute'
     end
   end
 
-  context 'on RedHat platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'RedHat' })
-    end
-
-    let :platform_params do
-      { :agent_package_name => 'openstack-ceilometer-compute',
-        :agent_service_name => 'openstack-ceilometer-compute' }
-    end
-
-    it_configures 'ceilometer-agent-compute'
-  end
 end
diff --git a/spec/classes/ceilometer_agent_notification_spec.rb b/spec/classes/ceilometer_agent_notification_spec.rb
index c172234..f66858a 100644
--- a/spec/classes/ceilometer_agent_notification_spec.rb
+++ b/spec/classes/ceilometer_agent_notification_spec.rb
@@ -172,94 +172,27 @@ describe 'ceilometer::agent::notification' do
     end
   end
 
-  context 'on Debian platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'Debian' })
-    end
-
-    let :platform_params do
-      { :agent_notification_package_name => 'ceilometer-agent-notification',
-        :agent_notification_service_name => 'ceilometer-agent-notification' }
-    end
-
-    it_configures 'ceilometer-agent-notification'
-  end
-
-  context 'on RedHat platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'RedHat' })
-    end
-
-    let :platform_params do
-      { :agent_notification_package_name => 'openstack-ceilometer-notification',
-        :agent_notification_service_name => 'openstack-ceilometer-notification' }
-    end
-
-    it_configures 'ceilometer-agent-notification'
-  end
-
-  context 'on RHEL 7' do
-    let :facts do
-      @default_facts.merge({ :osfamily                  => 'RedHat',
-        :operatingsystem           => 'RedHat',
-        :operatingsystemmajrelease => 7
-      })
-    end
-
-    let :platform_params do
-      { :agent_notification_package_name => 'openstack-ceilometer-notification',
-        :agent_notification_service_name => 'openstack-ceilometer-notification' }
-    end
-
-    it_configures 'ceilometer-agent-notification'
-  end
-
-  context 'on CentOS 7' do
-    let :facts do
-      @default_facts.merge({ :osfamily                  => 'RedHat',
-        :operatingsystem           => 'CentOS',
-        :operatingsystemmajrelease => 7
-      })
-    end
-
-    let :platform_params do
-      { :agent_notification_package_name => 'openstack-ceilometer-notification',
-        :agent_notification_service_name => 'openstack-ceilometer-notification' }
-    end
-
-    it_configures 'ceilometer-agent-notification'
-  end
-
-  context 'on Scientific 7' do
-    let :facts do
-      @default_facts.merge({ :osfamily                  => 'RedHat',
-        :operatingsystem           => 'Scientific',
-        :operatingsystemmajrelease => 7
-      })
-    end
-
-    let :platform_params do
-      { :agent_notification_package_name => 'openstack-ceilometer-notification',
-        :agent_notification_service_name => 'openstack-ceilometer-notification' }
-    end
-
-    it_configures 'ceilometer-agent-notification'
-  end
+  on_supported_os({
+    :supported_os => OSDefaults.get_supported_os
+  }).each do |os,facts|
+    context "on #{os}" do
+      let (:facts) do
+        facts.merge!(OSDefaults.get_facts())
+      end
 
-  context 'on Fedora 20' do
-    let :facts do
-      @default_facts.merge({ :osfamily               => 'RedHat',
-        :operatingsystem        => 'Fedora',
-        :operatingsystemrelease => 20
-      })
-    end
+      let :platform_params do
+        case facts[:osfamily]
+        when 'Debian'
+          { :agent_notification_package_name => 'ceilometer-agent-notification',
+            :agent_notification_service_name => 'ceilometer-agent-notification' }
+        when 'RedHat'
+          { :agent_notification_package_name => 'openstack-ceilometer-notification',
+            :agent_notification_service_name => 'openstack-ceilometer-notification' }
+        end
+      end
 
-    let :platform_params do
-      { :agent_notification_package_name => 'openstack-ceilometer-notification',
-        :agent_notification_service_name => 'openstack-ceilometer-notification' }
+      it_behaves_like 'ceilometer-agent-notification'
     end
-
-    it_configures 'ceilometer-agent-notification'
   end
 
 end
diff --git a/spec/classes/ceilometer_agent_polling_spec.rb b/spec/classes/ceilometer_agent_polling_spec.rb
index 3280a17..1846855 100644
--- a/spec/classes/ceilometer_agent_polling_spec.rb
+++ b/spec/classes/ceilometer_agent_polling_spec.rb
@@ -114,45 +114,34 @@ describe 'ceilometer::agent::polling' do
 
   end
 
-  context 'on Debian platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'Debian' })
-    end
-
-    let :platform_params do
-      { :agent_package_name => 'ceilometer-polling',
-        :agent_service_name => 'ceilometer-polling' }
-    end
-
-    context 'on Ubuntu operating systems' do
-      before do
-        facts.merge!( :operatingsystem => 'Ubuntu' )
-        platform_params.merge!( :libvirt_group => 'libvirtd' )
+  on_supported_os({
+    :supported_os => OSDefaults.get_supported_os
+  }).each do |os,facts|
+    context "on #{os}" do
+      let (:facts) do
+        facts.merge!(OSDefaults.get_facts())
       end
 
-      it_configures 'ceilometer-polling'
-    end
-
-    context 'on other operating systems' do
-      before do
-        platform_params.merge!( :libvirt_group => 'libvirt' )
+      let :platform_params do
+        case facts[:osfamily]
+        when 'Debian'
+          if facts[:operatingsystem] == 'Ubuntu'
+            { :agent_package_name => 'ceilometer-polling',
+              :agent_service_name => 'ceilometer-polling',
+              :libvirt_group      => 'libvirtd' }
+          else
+            { :agent_package_name => 'ceilometer-polling',
+              :agent_service_name => 'ceilometer-polling',
+              :libvirt_group      => 'libvirt' }
+          end
+        when 'RedHat'
+            { :agent_package_name => 'openstack-ceilometer-polling',
+              :agent_service_name => 'openstack-ceilometer-polling' }
+        end
       end
 
-      it_configures 'ceilometer-polling'
+      it_behaves_like 'ceilometer-polling'
     end
-
   end
 
-  context 'on RedHat platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'RedHat' })
-    end
-
-    let :platform_params do
-      { :agent_package_name => 'openstack-ceilometer-polling',
-        :agent_service_name => 'openstack-ceilometer-polling' }
-    end
-
-    it_configures 'ceilometer-polling'
-  end
 end
diff --git a/spec/classes/ceilometer_api_spec.rb b/spec/classes/ceilometer_api_spec.rb
index e0ccda5..bfb1de6 100644
--- a/spec/classes/ceilometer_api_spec.rb
+++ b/spec/classes/ceilometer_api_spec.rb
@@ -149,41 +149,30 @@ describe 'ceilometer::api' do
     end
   end
 
-  context 'on Debian platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily               => 'Debian',
-        :operatingsystem        => 'Debian',
-        :operatingsystemrelease => '8.0',
-        :concat_basedir         => '/var/lib/puppet/concat',
-        :fqdn                   => 'some.host.tld',
-      })
-    end
-
-    let :platform_params do
-      { :api_package_name => 'ceilometer-api',
-        :api_service_name => 'ceilometer-api' }
-    end
-
-    it_configures 'ceilometer-api'
-  end
+  on_supported_os({
+    :supported_os => OSDefaults.get_supported_os
+  }).each do |os,facts|
+    context "on #{os}" do
+      let (:facts) do
+        facts.merge!(OSDefaults.get_facts({
+          :concat_basedir         => '/var/lib/puppet/concat',
+          :fqdn                   => 'some.host.tld',
+        }))
+      end
 
-  context 'on RedHat platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily               => 'RedHat',
-        :operatingsystem        => 'RedHat',
-        :operatingsystemrelease => '7.1',
-        :operatingsystemmajrelease => '7',
-        :fqdn                   => 'some.host.tld',
-        :concat_basedir         => '/var/lib/puppet/concat',
-      })
-    end
+      let :platform_params do
+        case facts[:osfamily]
+        when 'Debian'
+          { :api_package_name => 'ceilometer-api',
+            :api_service_name => 'ceilometer-api' }
+        when 'RedHat'
+          { :api_package_name => 'openstack-ceilometer-api',
+            :api_service_name => 'openstack-ceilometer-api' }
+        end
+      end
 
-    let :platform_params do
-      { :api_package_name => 'openstack-ceilometer-api',
-        :api_service_name => 'openstack-ceilometer-api' }
+      it_behaves_like 'ceilometer-api'
     end
-
-    it_configures 'ceilometer-api'
   end
 
 end
diff --git a/spec/classes/ceilometer_client_spec.rb b/spec/classes/ceilometer_client_spec.rb
index 99f2e29..ce4927e 100644
--- a/spec/classes/ceilometer_client_spec.rb
+++ b/spec/classes/ceilometer_client_spec.rb
@@ -15,27 +15,20 @@ describe 'ceilometer::client' do
     end
   end
 
-  context 'on Debian platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'Debian' })
+  on_supported_os({
+    :supported_os => OSDefaults.get_supported_os
+  }).each do |os,facts|
+    context "on #{os}" do
+      let (:facts) do
+        facts.merge!(OSDefaults.get_facts())
+      end
+
+      let :platform_params do
+        { :client_package_name => 'python-ceilometerclient' }
+      end
+
+      it_behaves_like 'ceilometer client'
     end
-
-    let :platform_params do
-      { :client_package_name => 'python-ceilometerclient' }
-    end
-
-    it_configures 'ceilometer client'
   end
 
-  context 'on RedHat platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'RedHat' })
-    end
-
-    let :platform_params do
-      { :client_package_name => 'python-ceilometerclient' }
-    end
-
-    it_configures 'ceilometer client'
-  end
 end
diff --git a/spec/classes/ceilometer_collector_spec.rb b/spec/classes/ceilometer_collector_spec.rb
index 4763ae8..1316dfb 100644
--- a/spec/classes/ceilometer_collector_spec.rb
+++ b/spec/classes/ceilometer_collector_spec.rb
@@ -113,29 +113,27 @@ describe 'ceilometer::collector' do
     end
   end
 
-  context 'on Debian platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'Debian' })
+  on_supported_os({
+    :supported_os => OSDefaults.get_supported_os
+  }).each do |os,facts|
+    context "on #{os}" do
+      let (:facts) do
+        facts.merge!(OSDefaults.get_facts())
+      end
+
+      let :platform_params do
+        case facts[:osfamily]
+        when 'Debian'
+          { :collector_package_name => 'ceilometer-collector',
+            :collector_service_name => 'ceilometer-collector' }
+        when 'RedHat'
+          { :collector_package_name => 'openstack-ceilometer-collector',
+            :collector_service_name => 'openstack-ceilometer-collector' }
+        end
+      end
+
+      it_behaves_like 'ceilometer-collector'
     end
-
-    let :platform_params do
-      { :collector_package_name => 'ceilometer-collector',
-        :collector_service_name => 'ceilometer-collector' }
-    end
-
-    it_configures 'ceilometer-collector'
   end
 
-  context 'on RedHat platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'RedHat' })
-    end
-
-    let :platform_params do
-      { :collector_package_name => 'openstack-ceilometer-collector',
-        :collector_service_name => 'openstack-ceilometer-collector' }
-    end
-
-    it_configures 'ceilometer-collector'
-  end
 end
diff --git a/spec/classes/ceilometer_db_mysql_spec.rb b/spec/classes/ceilometer_db_mysql_spec.rb
index 610f4fc..fde1038 100644
--- a/spec/classes/ceilometer_db_mysql_spec.rb
+++ b/spec/classes/ceilometer_db_mysql_spec.rb
@@ -31,60 +31,48 @@ describe 'ceilometer::db::mysql' do
         :charset       => params[:charset]
       )
     end
-  end
-
-  context 'on Debian platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'Debian' })
-    end
 
-    it_configures 'ceilometer mysql database'
-  end
+    describe "overriding allowed_hosts param to array" do
+      let :params do
+        {
+          :password       => 'ceilometerpass',
+          :allowed_hosts  => ['localhost','%']
+        }
+      end
 
-  context 'on RedHat platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'RedHat' })
     end
 
-    it_configures 'ceilometer mysql database'
-  end
+    describe "overriding allowed_hosts param to string" do
+      let :params do
+        {
+          :password       => 'ceilometerpass2',
+          :allowed_hosts  => '192.168.1.1'
+        }
+      end
 
-  describe "overriding allowed_hosts param to array" do
-    let :facts do
-      @default_facts.merge({ :osfamily => "Debian" })
-    end
-    let :params do
-      {
-        :password       => 'ceilometerpass',
-        :allowed_hosts  => ['localhost','%']
-      }
     end
 
-  end
+    describe "overriding allowed_hosts param equals to host param " do
+      let :params do
+        {
+          :password       => 'ceilometerpass2',
+          :allowed_hosts  => 'localhost'
+        }
+      end
 
-  describe "overriding allowed_hosts param to string" do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'RedHat' })
     end
-    let :params do
-      {
-        :password       => 'ceilometerpass2',
-        :allowed_hosts  => '192.168.1.1'
-      }
-    end
-
   end
 
-  describe "overriding allowed_hosts param equals to host param " do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'RedHat' })
-    end
-    let :params do
-      {
-        :password       => 'ceilometerpass2',
-        :allowed_hosts  => 'localhost'
-      }
-    end
+  on_supported_os({
+    :supported_os => OSDefaults.get_supported_os
+  }).each do |os,facts|
+    context "on #{os}" do
+      let (:facts) do
+        facts.merge!(OSDefaults.get_facts())
+      end
 
+      it_behaves_like 'ceilometer mysql database'
+    end
   end
+
 end
diff --git a/spec/classes/ceilometer_db_spec.rb b/spec/classes/ceilometer_db_spec.rb
index 1da5b16..3b098c3 100644
--- a/spec/classes/ceilometer_db_spec.rb
+++ b/spec/classes/ceilometer_db_spec.rb
@@ -91,63 +91,66 @@ describe 'ceilometer::db' do
 
   end
 
-  context 'on Debian platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'Debian',
-        :operatingsystem => 'Debian',
-        :operatingsystemrelease => 'jessie',
-      })
-    end
-
-    it_configures 'ceilometer::db'
-
-    context 'using pymysql driver' do
+  shared_examples_for 'ceilometer::db on Debian' do
+    context 'with sqlite backend' do
       let :params do
-        { :database_connection     => 'mysql+pymysql:///ceilometer:ceilometer@localhost/ceilometer', }
+        { :database_connection     => 'sqlite:///var/lib/ceilometer.db', }
       end
 
       it 'install the proper backend package' do
         is_expected.to contain_package('db_backend_package').with(
           :ensure => 'present',
-          :name   => 'python-pymysql',
+          :name   => 'python-pysqlite2',
           :tag    => 'openstack'
         )
       end
     end
 
-    context 'with sqlite backend' do
+    context 'using pymysql driver' do
       let :params do
-        { :database_connection     => 'sqlite:///var/lib/ceilometer.db', }
+        { :database_connection => 'mysql+pymysql:///ceilometer:ceilometer@localhost/ceilometer', }
       end
 
       it 'install the proper backend package' do
         is_expected.to contain_package('db_backend_package').with(
           :ensure => 'present',
-          :name   => 'python-pysqlite2',
+          :name   => 'python-pymysql',
           :tag    => 'openstack'
         )
       end
-
     end
   end
 
-  context 'on Redhat platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'RedHat',
-        :operatingsystemrelease => '7.1',
-      })
-    end
-
-    it_configures 'ceilometer::db'
-
+  shared_examples_for 'ceilometer::db on RedHat' do
     context 'using pymysql driver' do
       let :params do
-        { :database_connection     => 'mysql+pymysql:///ceilometer:ceilometer@localhost/ceilometer', }
+        { :database_connection => 'mysql+pymysql:///ceilometer:ceilometer@localhost/ceilometer', }
       end
 
       it { is_expected.not_to contain_package('db_backend_package') }
     end
   end
 
-end
+  on_supported_os({
+    :supported_os => OSDefaults.get_supported_os
+  }).each do |os,facts|
+    context "on #{os}" do
+      let (:facts) do
+        facts.merge!(OSDefaults.get_facts({
+          :concat_basedir         => '/var/lib/puppet/concat',
+          :fqdn                   => 'some.host.tld',
+        }))
+      end
 
+      case facts[:osfamily]
+      when 'Debian'
+        it_behaves_like 'ceilometer::db on Debian'
+      when 'RedHat'
+        it_behaves_like 'ceilometer::db on RedHat'
+      end
+
+      it_behaves_like 'ceilometer::db'
+    end
+  end
+
+end
diff --git a/spec/classes/ceilometer_db_sync_spec.rb b/spec/classes/ceilometer_db_sync_spec.rb
index f4b37b4..6a9239f 100644
--- a/spec/classes/ceilometer_db_sync_spec.rb
+++ b/spec/classes/ceilometer_db_sync_spec.rb
@@ -43,7 +43,7 @@ describe 'ceilometer::db::sync' do
         }))
       end
 
-      it_configures 'ceilometer-dbsync'
+      it_behaves_like 'ceilometer-dbsync'
     end
   end
 
diff --git a/spec/classes/ceilometer_dispatcher_gnocchi_spec.rb b/spec/classes/ceilometer_dispatcher_gnocchi_spec.rb
index e455400..a5b6444 100644
--- a/spec/classes/ceilometer_dispatcher_gnocchi_spec.rb
+++ b/spec/classes/ceilometer_dispatcher_gnocchi_spec.rb
@@ -35,20 +35,16 @@ describe 'ceilometer::dispatcher::gnocchi' do
     end
   end
 
-  context 'on Debian platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'Debian' })
-    end
-
-    it_configures 'ceilometer-gnocchi-dispatcher'
-  end
+  on_supported_os({
+    :supported_os   => OSDefaults.get_supported_os
+  }).each do |os,facts|
+    context "on #{os}" do
+      let (:facts) do
+        facts.merge(OSDefaults.get_facts())
+      end
 
-  context 'on RedHat platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'RedHat' })
+      it_behaves_like 'ceilometer-gnocchi-dispatcher'
     end
-
-    it_configures 'ceilometer-gnocchi-dispatcher'
   end
 
 end
diff --git a/spec/classes/ceilometer_expirer_spec.rb b/spec/classes/ceilometer_expirer_spec.rb
index 3bf6fe0..8c9b0a6 100644
--- a/spec/classes/ceilometer_expirer_spec.rb
+++ b/spec/classes/ceilometer_expirer_spec.rb
@@ -64,28 +64,25 @@ describe 'ceilometer::expirer' do
 
   end
 
-  context 'on Debian platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'Debian' })
-    end
-
-    let :platform_params do
-      { :common_package_name => 'ceilometer-common' }
-    end
-
-    it_configures 'ceilometer-expirer'
-  end
+  on_supported_os({
+    :supported_os => OSDefaults.get_supported_os
+  }).each do |os,facts|
+    context "on #{os}" do
+      let (:facts) do
+        facts.merge!(OSDefaults.get_facts())
+      end
 
-  context 'on RedHat platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'RedHat' })
-    end
+      let :platform_params do
+        case facts[:osfamily]
+        when 'Debian'
+          { :common_package_name => 'ceilometer-common' }
+        when 'RedHat'
+          { :common_package_name => 'openstack-ceilometer-common' }
+        end
+      end
 
-    let :platform_params do
-      { :common_package_name => 'openstack-ceilometer-common' }
+      it_behaves_like 'ceilometer-expirer'
     end
-
-    it_configures 'ceilometer-expirer'
   end
 
 end
diff --git a/spec/classes/ceilometer_init_spec.rb b/spec/classes/ceilometer_init_spec.rb
index 36af30d..b5cd7e5 100644
--- a/spec/classes/ceilometer_init_spec.rb
+++ b/spec/classes/ceilometer_init_spec.rb
@@ -375,27 +375,25 @@ describe 'ceilometer' do
     end
   end
 
-  context 'on Debian platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'Debian' })
-    end
-
-    let :platform_params do
-      { :common_package_name => 'ceilometer-common' }
-    end
-
-    it_configures 'ceilometer'
-  end
+  on_supported_os({
+    :supported_os => OSDefaults.get_supported_os
+  }).each do |os,facts|
+    context "on #{os}" do
+      let (:facts) do
+        facts.merge!(OSDefaults.get_facts())
+      end
 
-  context 'on RedHat platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'RedHat' })
-    end
+      let :platform_params do
+        case facts[:osfamily]
+        when 'Debian'
+          { :common_package_name => 'ceilometer-common' }
+        when 'RedHat'
+          { :common_package_name => 'openstack-ceilometer-common' }
+        end
+      end
 
-    let :platform_params do
-      { :common_package_name => 'openstack-ceilometer-common' }
+      it_behaves_like 'ceilometer'
     end
-
-    it_configures 'ceilometer'
   end
+
 end
diff --git a/spec/classes/ceilometer_keystone_auth_spec.rb b/spec/classes/ceilometer_keystone_auth_spec.rb
index 84177f6..4110ca7 100644
--- a/spec/classes/ceilometer_keystone_auth_spec.rb
+++ b/spec/classes/ceilometer_keystone_auth_spec.rb
@@ -169,20 +169,16 @@ describe 'ceilometer::keystone::auth' do
 
   end
 
-  context 'on Debian platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'Debian' })
-    end
-
-    it_configures 'ceilometer keystone auth'
-  end
+  on_supported_os({
+    :supported_os   => OSDefaults.get_supported_os
+  }).each do |os,facts|
+    context "on #{os}" do
+      let (:facts) do
+        facts.merge(OSDefaults.get_facts())
+      end
 
-  context 'on RedHat platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'RedHat' })
+      it_behaves_like 'ceilometer keystone auth'
     end
-
-    it_configures 'ceilometer keystone auth'
   end
 
 end
diff --git a/spec/classes/ceilometer_keystone_authtoken_spec.rb b/spec/classes/ceilometer_keystone_authtoken_spec.rb
index 17c1f6d..ce6caa3 100644
--- a/spec/classes/ceilometer_keystone_authtoken_spec.rb
+++ b/spec/classes/ceilometer_keystone_authtoken_spec.rb
@@ -138,7 +138,7 @@ describe 'ceilometer::keystone::authtoken' do
         facts.merge!(OSDefaults.get_facts())
       end
 
-      it_configures 'ceilometer authtoken'
+      it_behaves_like 'ceilometer authtoken'
     end
   end
 
diff --git a/spec/classes/ceilometer_logging_spec.rb b/spec/classes/ceilometer_logging_spec.rb
index e96456f..37e0c9d 100644
--- a/spec/classes/ceilometer_logging_spec.rb
+++ b/spec/classes/ceilometer_logging_spec.rb
@@ -120,20 +120,16 @@ describe 'ceilometer::logging' do
       }
   end
 
-  context 'on Debian platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'Debian' })
+  on_supported_os({
+    :supported_os   => OSDefaults.get_supported_os
+  }).each do |os,facts|
+    context "on #{os}" do
+      let (:facts) do
+        facts.merge(OSDefaults.get_facts())
+      end
+
+      it_behaves_like 'ceilometer-logging'
     end
-
-    it_configures 'ceilometer-logging'
-  end
-
-  context 'on RedHat platforms' do
-    let :facts do
-      @default_facts.merge({ :osfamily => 'RedHat' })
-    end
-
-    it_configures 'ceilometer-logging'
   end
 
 end
diff --git a/spec/classes/ceilometer_policy_spec.rb b/spec/classes/ceilometer_policy_spec.rb
index 5bb5473..c57d367 100644
--- a/spec/classes/ceilometer_policy_spec.rb
+++ b/spec/classes/ceilometer_policy_spec.rb
@@ -32,7 +32,7 @@ describe 'ceilometer::policy' do
         facts.merge!(OSDefaults.get_facts())
       end
 
-      it_configures 'ceilometer policies'
+      it_behaves_like 'ceilometer policies'
     end
   end
 
diff --git a/spec/classes/ceilometer_wsgi_apache_spec.rb b/spec/classes/ceilometer_wsgi_apache_spec.rb
index 64908b5..4b4543b 100644
--- a/spec/classes/ceilometer_wsgi_apache_spec.rb
+++ b/spec/classes/ceilometer_wsgi_apache_spec.rb
@@ -85,44 +85,37 @@ describe 'ceilometer::wsgi::apache' do
     end
   end
 
-  context 'on RedHat platforms' do
-    let :facts do
-      @default_facts.merge(global_facts.merge({
-        :osfamily               => 'RedHat',
-        :operatingsystemrelease => '7.0'
-      }))
-    end
-
-    let :platform_parameters do
-      {
-        :httpd_service_name => 'httpd',
-        :httpd_ports_file   => '/etc/httpd/conf/ports.conf',
-        :wsgi_script_path   => '/var/www/cgi-bin/ceilometer',
-        :wsgi_script_source => '/usr/lib/python2.7/site-packages/ceilometer/api/app.wsgi'
-      }
-    end
-
-    it_configures 'apache serving ceilometer with mod_wsgi'
-  end
+  on_supported_os({
+    :supported_os   => OSDefaults.get_supported_os
+  }).each do |os,facts|
+    context "on #{os}" do
+      let (:facts) do
+        facts.merge!(OSDefaults.get_facts({
+          :fqdn => 'some.host.tld'
+        }))
+      end
 
-  context 'on Debian platforms' do
-    let :facts do
-      @default_facts.merge(global_facts.merge({
-        :osfamily               => 'Debian',
-        :operatingsystem        => 'Debian',
-        :operatingsystemrelease => '7.0'
-      }))
-    end
+      let :platform_parameters do
+        case facts[:osfamily]
+        when 'Debian'
+         {
+            :httpd_service_name => 'apache2',
+            :httpd_ports_file   => '/etc/apache2/ports.conf',
+            :wsgi_script_path   => '/usr/lib/cgi-bin/ceilometer',
+            :wsgi_script_source => '/usr/lib/python2.7/dist-packages/ceilometer/api/app.wsgi'
+          }
+        when 'RedHat'
+          {
+            :httpd_service_name => 'httpd',
+            :httpd_ports_file   => '/etc/httpd/conf/ports.conf',
+            :wsgi_script_path   => '/var/www/cgi-bin/ceilometer',
+            :wsgi_script_source => '/usr/lib/python2.7/site-packages/ceilometer/api/app.wsgi'
+          }
+        end
+      end
 
-    let :platform_parameters do
-      {
-        :httpd_service_name => 'apache2',
-        :httpd_ports_file   => '/etc/apache2/ports.conf',
-        :wsgi_script_path   => '/usr/lib/cgi-bin/ceilometer',
-        :wsgi_script_source => '/usr/lib/python2.7/dist-packages/ceilometer/api/app.wsgi'
-      }
+      it_behaves_like 'apache serving ceilometer with mod_wsgi'
     end
-
-    it_configures 'apache serving ceilometer with mod_wsgi'
   end
+
 end