openstack/puppet-cinder
Code Issues Proposed changes

Fix unit test for class cinder::backup

Browse Source 
Change-Id: I68921e52e3f1d911c24e259ebbd05970f77fe34f
This commit is contained in:
ZhongShengping 2017-05-27 09:41:23 +08:00
parent d67297944d
commit 6d786cf1cb
1 changed files with 12 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
spec/classes/cinder_backup_spec.rb
View File

@ -53,8 +53,11 @@ describe 'cinder::backup' do
it 'ensure cinder backup service is running' do
is_expected.to contain_service('cinder-backup').with(
'hasstatus' => true,
'tag' => 'cinder-service',
:ensure => 'running',
:name => platform_params[:backup_service],
:enable => true,
:hasstatus => true,
:tag => 'cinder-service',
)
end
@ -78,7 +81,12 @@ describe 'cinder::backup' do
params.merge!(:manage_service => false)
end
it 'should not change the state of the service' do
is_expected.to contain_service('cinder-backup').without_ensure
is_expected.to contain_service('cinder-backup').with(
:ensure => nil,
:name => platform_params[:backup_service],
:hasstatus => true,
:tag => 'cinder-service',
)
end
end
@ -97,7 +105,7 @@ describe 'cinder::backup' do
{ :backup_package => 'cinder-backup',
:backup_service => 'cinder-backup' }
else
{ :backup_service => 'opentack-cinder-backup' }
{ :backup_service => 'openstack-cinder-backup' }
end
end