openstack/puppet-sahara
Code Issues Proposed changes

Test multiple operating systems for db

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

Change-Id: Ie4a3383f2857300f637d605e94d03d539156c863
This commit is contained in:
ZhongShengping 2016-04-30 11:57:49 +08:00
parent b4290d316c
commit e5f369e5a7
2 changed files with 27 additions and 54 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

36
spec/classes/sahara_db_postgresql_spec.rb
View File

@ -2,6 +2,7 @@ require 'spec_helper'
describe 'sahara::db::postgresql' do
shared_examples_for 'sahara::db::postgresql' do
let :req_params do
{ :password => 'pw' }
end
@ -10,15 +11,6 @@ describe 'sahara::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 'sahara::db::postgresql' do
end
context 'on a Debian osfamily' do
let :facts do
{
:operatingsystemrelease => '7.8',
:operatingsystem => 'Debian',
: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({
:processorcount => 8,
:concat_basedir => '/var/lib/puppet/concat'
}
}))
end
context 'with only required parameters' do
let :params do
req_params
it_configures 'sahara::db::postgresql'
end
it { is_expected.to contain_postgresql__server__db('sahara').with(
:user => 'sahara',
:password => 'md59b1dd0cc439677764ef5a848112ef0ab'
)}
end
end
end

27
spec/classes/sahara_db_sync_spec.rb
View File

@ -34,30 +34,19 @@ describe 'sahara::db::sync' do
end
context 'on a RedHat osfamily' do
let :facts do
{
:osfamily => 'RedHat',
:operatingsystemrelease => '7.0',
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({
:processorcount => 8,
:concat_basedir => '/var/lib/puppet/concat'
}
}))
end
it_configures 'sahara-dbsync'
end
context 'on a Debian osfamily' do
let :facts do
{
:operatingsystemrelease => '7.8',
:operatingsystem => 'Debian',
:osfamily => 'Debian',
:concat_basedir => '/var/lib/puppet/concat'
}
end
it_configures 'sahara-dbsync'
end
end