openstack/cookbook-openstack-block-storage
Code Issues Proposed changes

Merge "Move all cinder-common tests into cinder-common_spec"

Browse Source
This commit is contained in:
Jenkins
2014-04-28 19:52:49 +00:00
committed by Gerrit Code Review
parent d9eb8eff9f f95c821406
commit 5f8300c216
2 changed files with 24 additions and 40 deletions
Download Patch File Download Diff File Expand all files Collapse all files

40
spec/api_spec.rb
View File

@@ -48,46 +48,6 @@ describe 'openstack-block-storage::api' do
end end
end end
describe 'cinder.conf' do
let(:file) { chef_run.template('/etc/cinder/cinder.conf') }
it 'runs logging recipe if node attributes say to' do
node.set['openstack']['block-storage']['syslog']['use'] = true
expect(chef_run).to render_file(file.name).with_content('log_config = /etc/openstack/logging.conf')
end
context 'rdb driver' do
before do
node.set['openstack']['block-storage']['volume'] = {
'driver' => 'cinder.volume.drivers.rbd.RBDDriver'
}
end
# FIXME(galstrom21): this block needs to check all of the default
# rdb_* configuration options
it 'has default rbd_* options set' do
expect(chef_run).to render_file(file.name).with_content(/^rbd_/)
expect(chef_run).not_to render_file(file.name).with_content(/^netapp_/)
end
end
context 'netapp driver' do
before do
node.set['openstack']['block-storage']['volume'] = {
'driver' => 'cinder.volume.drivers.netapp.NetAppISCSIDriver'
}
end
# FIXME(galstrom21): this block needs to check all of the default
# netapp_* configuration options
it 'has default netapp_* options set' do
expect(chef_run).to render_file(file.name).with_content(/^netapp_/)
expect(chef_run).not_to render_file(file.name).with_content(/^rbd_/)
end
end
end
it 'runs db migrations' do it 'runs db migrations' do
expect(chef_run).to run_execute('cinder-manage db sync') expect(chef_run).to run_execute('cinder-manage db sync')
end end

24
spec/cinder_common_spec.rb
View File

@@ -74,6 +74,30 @@ describe 'openstack-block-storage::cinder-common' do
end end
end end
context 'rdb driver' do
# FIXME(galstrom21): this block needs to check all of the default
# rdb_* configuration options
it 'has default rbd_* options set' do
node.set['openstack']['block-storage']['volume'] = {
'driver' => 'cinder.volume.drivers.rbd.RBDDriver'
}
expect(chef_run).to render_file(file.name).with_content(/^rbd_/)
expect(chef_run).not_to render_file(file.name).with_content(/^netapp_/)
end
end
context 'netapp driver' do
# FIXME(galstrom21): this block needs to check all of the default
# netapp_* configuration options
it 'has default netapp_* options set' do
node.set['openstack']['block-storage']['volume'] = {
'driver' => 'cinder.volume.drivers.netapp.NetAppISCSIDriver'
}
expect(chef_run).to render_file(file.name).with_content(/^netapp_/)
expect(chef_run).not_to render_file(file.name).with_content(/^rbd_/)
end
end
context 'syslog use' do context 'syslog use' do
it 'sets the log_config value when syslog is in use' do it 'sets the log_config value when syslog is in use' do
node.set['openstack']['block-storage']['syslog']['use'] = true node.set['openstack']['block-storage']['syslog']['use'] = true