Test multiple operating systems for db

This change updates the tests for mistral::db::postgresql and
mistral::db::sync to test multiple operating systems.

Change-Id: Ia362e005c253485db685880ddb5001a0cce5adef

spec/classes/mistral_db_postgresql_spec.rb

@@ -2,6 +2,7 @@ require 'spec_helper'
 
 describe 'mistral::db::postgresql' do
 
+  shared_examples_for 'mistral::db::postgresql' do
     let :req_params do
       { :password => 'pw' }
     end
@@ -10,15 +11,6 @@ describe 'mistral::db::postgresql' do
       'include postgresql::server'
     end
 
-  context 'on a RedHat osfamily' do
-    let :facts do
-      {
-        :osfamily                 => 'RedHat',
-        :operatingsystemrelease   => '7.0',
-        :concat_basedir => '/var/lib/puppet/concat'
-      }
-    end
-
     context 'with only required parameters' do
       let :params do
         req_params
@@ -32,27 +24,19 @@ describe 'mistral::db::postgresql' do
 
   end
 
-  context 'on a Debian osfamily' do
+  on_supported_os({
-    let :facts do
+    :supported_os   => OSDefaults.get_supported_os
-      {
+  }).each do |os,facts|
-        :operatingsystemrelease => '7.8',
+    context "on #{os}" do
-        :operatingsystem        => 'Debian',
+      let (:facts) do
-        :osfamily               => 'Debian',
+        facts.merge(OSDefaults.get_facts({
+          :processorcount => 8,
           :concat_basedir => '/var/lib/puppet/concat'
-      }
+        }))
       end
 
-    context 'with only required parameters' do
+      it_configures 'mistral::db::postgresql'
-      let :params do
-        req_params
     end
-
-      it { is_expected.to contain_postgresql__server__db('mistral').with(
-        :user     => 'mistral',
-        :password => 'md56cc4a9db977897a441d3a498a5d94fce'
-      )}
-    end
-
   end
 
 end

spec/classes/mistral_db_sync_spec.rb

@@ -26,28 +26,19 @@ describe 'mistral::db::sync' do
 
   end
 
-  context 'on a RedHat osfamily' do
+  on_supported_os({
-    let :facts do
+    :supported_os   => OSDefaults.get_supported_os
-      {
+  }).each do |os,facts|
-        :osfamily                 => 'RedHat',
+    context "on #{os}" do
-        :operatingsystemrelease   => '7.0',
+      let (:facts) do
+        facts.merge(OSDefaults.get_facts({
+          :processorcount => 8,
           :concat_basedir => '/var/lib/puppet/concat'
-      }
+        }))
       end
 
       it_configures 'mistral-db-sync'
     end
-
-  context 'on a Debian osfamily' do
-    let :facts do
-      {
-        :operatingsystemrelease => '8.0',
-        :osfamily               => 'Debian',
-        :concat_basedir => '/var/lib/puppet/concat'
-      }
-    end
-
-    it_configures 'mistral-db-sync'
   end
 
 end