openstack/puppet-keystone
Code Issues Proposed changes

Move keystone ldap class to correct file.

Browse Source 
The dirname did not match the class namespace.

Now it does. And I added some tests so that its actually
being tested for stuff like this.
This commit is contained in:
Dan Bode 2012-11-06 14:54:08 -08:00
parent ea17c786ec
commit 18a6c6a99f
2 changed files with 20 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

0
manifests/config/ldap.pp → manifests/ldap.pp
View File

20
spec/classes/keystone_ldap_spec.rb Normal file
View File

@ -0,0 +1,20 @@
require 'spec_helper'
describe 'keystone::ldap' do
describe 'with default params' do
it 'should contain default params' do
should contain_keystone_config('ldap/url').with_value('ldap://localhost')
should contain_keystone_config('ldap/user').with_value('dc=Manager,dc=example,dc=com')
should contain_keystone_config('ldap/password').with_value('None')
should contain_keystone_config('ldap/suffix').with_value('cn=example,cn=com')
should contain_keystone_config('ldap/user_tree_dn').with_value('ou=Users,dc=example,dc=com')
should contain_keystone_config('ldap/tenant_tree_dn').with_value('ou=Roles,dc=example,dc=com')
should contain_keystone_config('ldap/role_tree_dn').with_value('dc=example,dc=com')
end
end
end