opuppet/module-inifile
5
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Merge pull request #196 from domcleal/pup-4709-workaround-master

Browse source 
Remove brackets from ini_setting titles to workaround PUP-4709
This commit is contained in:
Bryan Jen 2016-02-09 16:45:31 -07:00
commit b79dd27ac5
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/puppet/parser/functions/create_ini_settings.rb
View file

@ -69,7 +69,7 @@ EOS
end end
settings[section].each do |setting, value| settings[section].each do |setting, value|
res["#{path} [#{section}] #{setting}"] = { res["#{path} #{section} #{setting}"] = {
'ensure' => 'present', 'ensure' => 'present',
'section' => section, 'section' => section,
'setting' => setting, 'setting' => setting,

6
spec/classes/create_ini_settings_test_spec.rb
View file

@ -2,21 +2,21 @@ require 'spec_helper'
# end-to-end test of the create_init_settings function # end-to-end test of the create_init_settings function
describe 'create_ini_settings_test' do describe 'create_ini_settings_test' do
it { should have_ini_setting_resource_count(3) } it { should have_ini_setting_resource_count(3) }
it { should contain_ini_setting('/tmp/foo.ini [section1] setting1').with( it { should contain_ini_setting('/tmp/foo.ini section1 setting1').with(
:ensure => 'present', :ensure => 'present',
:section => 'section1', :section => 'section1',
:setting => 'setting1', :setting => 'setting1',
:value => 'val1', :value => 'val1',
:path => '/tmp/foo.ini' :path => '/tmp/foo.ini'
)} )}
it { should contain_ini_setting('/tmp/foo.ini [section2] setting2').with( it { should contain_ini_setting('/tmp/foo.ini section2 setting2').with(
:ensure => 'present', :ensure => 'present',
:section => 'section2', :section => 'section2',
:setting => 'setting2', :setting => 'setting2',
:value => 'val2', :value => 'val2',
:path => '/tmp/foo.ini' :path => '/tmp/foo.ini'
)} )}
it { should contain_ini_setting('/tmp/foo.ini [section2] setting3').with( it { should contain_ini_setting('/tmp/foo.ini section2 setting3').with(
:ensure => 'absent', :ensure => 'absent',
:section => 'section2', :section => 'section2',
:setting => 'setting3', :setting => 'setting3',