openstack
/
puppet-magnum
Code Issues Proposed changes

Merge "Convert to rspec-puppet-facts"

This commit is contained in:
Zuul 2018-11-27 06:58:55 +00:00 committed by Gerrit Code Review
parent c618a4e8c6 72c606f71c
commit a197161a7b
1 changed files with 104 additions and 94 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

114
spec/classes/magnum_keystone_auth_spec.rb
View File

@ -1,87 +1,86 @@
#
# Unit tests for magnum::keystone::auth
#
require 'spec_helper'
describe 'magnum::keystone::auth' do
let :facts do
OSDefaults.get_facts({ :osfamily => 'Debian' })
end
describe 'with default class parameters' do
shared_examples 'magnum::keystone::auth' do
context 'with default class parameters' do
let :params do
{ :password => 'magnum_password',
:tenant => 'foobar' }
{
:password => 'magnum_password',
:tenant => 'foobar'
}
end
it { is_expected.to contain_keystone_user('magnum').with(
it { should contain_keystone_user('magnum').with(
:ensure => 'present',
:password => 'magnum_password',
) }
)}
it { is_expected.to contain_keystone_user_role('magnum@foobar').with(
it { should contain_keystone_user_role('magnum@foobar').with(
:ensure => 'present',
:roles => ['admin']
)}
it { is_expected.to contain_keystone_service('magnum::container-infra').with(
it { should contain_keystone_service('magnum::container-infra').with(
:ensure => 'present',
:description => 'magnum Container Service'
) }
)}
it { is_expected.to contain_keystone_endpoint('RegionOne/magnum::container-infra').with(
it { should contain_keystone_endpoint('RegionOne/magnum::container-infra').with(
:ensure => 'present',
:public_url => 'http://127.0.0.1:9511/v1',
:admin_url => 'http://127.0.0.1:9511/v1',
:internal_url => 'http://127.0.0.1:9511/v1',
) }
)}
end
describe 'when overriding URL paramaters' do
context 'when overriding URL paramaters' do
let :params do
{ :password => 'magnum_password',
{
:password => 'magnum_password',
:public_url => 'https://10.10.10.10:80',
:internal_url => 'http://10.10.10.11:81',
:admin_url => 'http://10.10.10.12:81', }
:admin_url => 'http://10.10.10.12:81',
}
end
it { is_expected.to contain_keystone_endpoint('RegionOne/magnum::container-infra').with(
it { should contain_keystone_endpoint('RegionOne/magnum::container-infra').with(
:ensure => 'present',
:public_url => 'https://10.10.10.10:80',
:internal_url => 'http://10.10.10.11:81',
:admin_url => 'http://10.10.10.12:81',
) }
)}
end
describe 'when overriding auth name' do
context 'when overriding auth name' do
let :params do
{ :password => 'foo',
:auth_name => 'magnumy' }
{
:password => 'foo',
:auth_name => 'magnumy'
}
end
it { is_expected.to contain_keystone_user('magnumy') }
it { is_expected.to contain_keystone_user_role('magnumy@services') }
it { is_expected.to contain_keystone_service('magnumy::container-infra') }
it { is_expected.to contain_keystone_endpoint('RegionOne/magnumy::container-infra') }
it { should contain_keystone_user('magnumy') }
it { should contain_keystone_user_role('magnumy@services') }
it { should contain_keystone_service('magnumy::container-infra') }
it { should contain_keystone_endpoint('RegionOne/magnumy::container-infra') }
end
describe 'when overriding service name' do
context 'when overriding service name' do
let :params do
{ :service_name => 'magnum_service',
{
:service_name => 'magnum_service',
:auth_name => 'magnum',
:password => 'magnum_password' }
:password => 'magnum_password'
}
end
it { is_expected.to contain_keystone_user('magnum') }
it { is_expected.to contain_keystone_user_role('magnum@services') }
it { is_expected.to contain_keystone_service('magnum_service::container-infra') }
it { is_expected.to contain_keystone_endpoint('RegionOne/magnum_service::container-infra') }
it { should contain_keystone_user('magnum') }
it { should contain_keystone_user_role('magnum@services') }
it { should contain_keystone_service('magnum_service::container-infra') }
it { should contain_keystone_endpoint('RegionOne/magnum_service::container-infra') }
end
describe 'when disabling user configuration' do
context 'when disabling user configuration' do
let :params do
{
:password => 'magnum_password',
@ -89,18 +88,17 @@ describe 'magnum::keystone::auth' do
}
end
it { is_expected.not_to contain_keystone_user('magnum') }
it { is_expected.to contain_keystone_user_role('magnum@services') }
it { is_expected.to contain_keystone_service('magnum::container-infra').with(
it { should_not contain_keystone_user('magnum') }
it { should contain_keystone_user_role('magnum@services') }
it { should contain_keystone_service('magnum::container-infra').with(
:ensure => 'present',
:type => 'container-infra',
:description => 'magnum Container Service'
) }
)}
end
describe 'when disabling user and user role configuration' do
context 'when disabling user and user role configuration' do
let :params do
{
:password => 'magnum_password',
@ -109,14 +107,26 @@ describe 'magnum::keystone::auth' do
}
end
it { is_expected.not_to contain_keystone_user('magnum') }
it { is_expected.not_to contain_keystone_user_role('magnum@services') }
it { is_expected.to contain_keystone_service('magnum::container-infra').with(
it { should_not contain_keystone_user('magnum') }
it { should_not contain_keystone_user_role('magnum@services') }
it { should contain_keystone_service('magnum::container-infra').with(
:ensure => 'present',
:type => 'container-infra',
:description => 'magnum Container Service'
) }
)}
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())
end
it_behaves_like 'magnum::keystone::auth'
end
end
end