1234567891011121314151617181920212223242526272829303132333435363738394041424344454647484950515253545556575859606162636465666768697071727374757677787980818283848586878889909192939495969798991001011021031041051061071081091101111121131141151161171181191201211221231241251261271281291301311321331341351361371381391401411421431441451461471481491501511521531541551561571581591601611621631641651661671681691701711721731741751761771781791801811821831841851861871881891901911921931941951961971981992002012022032042052062072082092102112122132142152162172182192202212222232242252262272282292302312322332342352362372382392402412422432442452462472482492502512522532542552562572582592602612622632642652662672682692702712722732742752762772782792802812822832842852862872882892902912922932942952962972982993003013023033043053063073083093103113123133143153163173183193203213223233243253263273283293303313323333343353363373383393403413423433443453463473483493503513523533543553563573583593603613623633643653663673683693703713723733743753763773783793803813823833843853863873883893903913923933943953963973983994004014024034044054064074084094104114124134144154164174184194204214224234244254264274284294304314324334344354364374384394404414424434444454464474484494504514524534544554564574584594604614624634644654664674684694704714724734744754764774784794804814824834844854864874884894904914924934944954964974984995005015025035045055065075085095105115125135145155165175185195205215225235245255265275285295305315325335345355365375385395405415425435445455465475485495505515525535545555565575585595605615625635645655665675685695705715725735745755765775785795805815825835845855865875885895905915925935945955965975985996006016026036046056066076086096106116126136146156166176186196206216226236246256266276286296306316326336346356366376386396406416426436446456466476486496506516526536546556566576586596606616626636646656666676686696706716726736746756766776786796806816826836846856866876886896906916926936946956966976986997007017027037047057067077087097107117127137147157167177187197207217227237247257267277287297307317327337347357367377387397407417427437447457467477487497507517527537547557567577587597607617627637647657667677687697707717727737747757767777787797807817827837847857867877887897907917927937947957967977987998008018028038048058068078088098108118128138148158168178188198208218228238248258268278288298308318328338348358368378388398408418428438448458468478488498508518528538548558568578588598608618628638648658668678688698708718728738748758768778788798808818828838848858868878888898908918928938948958968978988999009019029039049059069079089099109119129139149159169179189199209219229239249259269279289299309319329339349359369379389399409419429439449459469479489499509519529539549559569579589599609619629639649659669679689699709719729739749759769779789799809819829839849859869879889899909919929939949959969979989991000100110021003100410051006100710081009101010111012101310141015101610171018101910201021102210231024102510261027102810291030103110321033103410351036103710381039104010411042104310441045104610471048104910501051105210531054105510561057105810591060106110621063106410651066106710681069107010711072107310741075107610771078107910801081108210831084108510861087108810891090109110921093109410951096109710981099110011011102110311041105110611071108110911101111111211131114111511161117111811191120112111221123112411251126112711281129113011311132113311341135113611371138113911401141114211431144114511461147114811491150115111521153115411551156115711581159116011611162116311641165116611671168116911701171117211731174117511761177117811791180118111821183118411851186118711881189119011911192119311941195119611971198119912001201120212031204120512061207120812091210121112121213121412151216121712181219122012211222122312241225122612271228122912301231123212331234123512361237123812391240124112421243124412451246124712481249125012511252125312541255125612571258125912601261126212631264126512661267126812691270127112721273127412751276127712781279128012811282128312841285128612871288128912901291129212931294129512961297129812991300130113021303130413051306130713081309131013111312131313141315131613171318131913201321132213231324132513261327132813291330133113321333133413351336133713381339134013411342134313441345134613471348134913501351135213531354135513561357135813591360136113621363136413651366136713681369137013711372137313741375137613771378137913801381138213831384138513861387138813891390139113921393139413951396139713981399140014011402140314041405140614071408140914101411141214131414141514161417141814191420142114221423142414251426142714281429143014311432143314341435143614371438143914401441144214431444144514461447144814491450145114521453145414551456145714581459146014611462146314641465146614671468146914701471147214731474147514761477147814791480148114821483148414851486148714881489149014911492149314941495149614971498 |
- require 'spec_helper'
- require 'puppet'
- provider_class = Puppet::Type.type(:ini_setting).provider(:ruby)
- describe provider_class do
- include PuppetlabsSpec::Files
- let(:tmpfile) { tmpfilename("ini_setting_test") }
- let(:emptyfile) { tmpfilename("ini_setting_test_empty") }
- let(:common_params) { {
- :title => 'ini_setting_ensure_present_test',
- :path => tmpfile,
- :section => 'section2',
- } }
- def validate_file(expected_content,tmpfile = tmpfile)
- File.read(tmpfile).should == expected_content
- end
- before :each do
- File.open(tmpfile, 'w') do |fh|
- fh.write(orig_content)
- end
- File.open(emptyfile, 'w') do |fh|
- fh.write("")
- end
- end
- context 'when calling instances' do
- let :orig_content do
- ''
- end
- it 'should fail when file path is not set' do
- expect {
- provider_class.instances
- }.to raise_error(Puppet::Error, 'Ini_settings only support collecting instances when a file path is hard coded')
- end
- context 'when file path is set by a child class' do
- it 'should return [] when file is empty' do
- child_one = Class.new(provider_class) do
- def self.file_path
- emptyfile
- end
- end
- child_one.stubs(:file_path).returns(emptyfile)
- child_one.instances.should == []
- end
- it 'should override the provider instances file_path' do
- child_two = Class.new(provider_class) do
- def self.file_path
- '/some/file/path'
- end
- end
- resource = Puppet::Type::Ini_setting.new(common_params)
- provider = child_two.new(resource)
- provider.file_path.should == '/some/file/path'
- end
- context 'when file has contecnts' do
- let(:orig_content) {
- <<-EOS
- # This is a comment
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- EOS
- }
- it 'should be able to parse the results' do
- child_three = Class.new(provider_class) do
- def self.file_path
- '/some/file/path'
- end
- end
- child_three.stubs(:file_path).returns(tmpfile)
- child_three.instances.size.should eq(7)
- expected_array = [
- {:name => 'section1/foo', :value => 'foovalue' },
- {:name => 'section1/bar', :value => 'barvalue' },
- {:name => 'section1/master', :value => 'true' },
- {:name => 'section2/foo', :value => 'foovalue2' },
- {:name => 'section2/baz', :value => 'bazvalue' },
- {:name => 'section2/url', :value => 'http://192.168.1.1:8080' },
- {:name => 'section:sub/subby', :value => 'bar' }
- ]
- real_array = []
- ensure_array = []
- child_three.instances.each do |x|
- prop_hash = x.instance_variable_get(:@property_hash)
- ensure_value = prop_hash.delete(:ensure)
- ensure_array.push(ensure_value)
- real_array.push(prop_hash)
- end
- ensure_array.uniq.should eq([:present])
- ((real_array - expected_array) && (expected_array - real_array)).should == []
- end
- end
- end
- end
- context "when ensuring that a setting is present" do
- let(:orig_content) {
- <<-EOS
- # This is a comment
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- -nonstandard-
- shoes = purple
- EOS
- }
- it "should add a missing setting to the correct section" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :setting => 'yahoo', :value => 'yippee'))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- # This is a comment
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- yahoo = yippee
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- -nonstandard-
- shoes = purple
- EOS
- )
- end
- it "should add a missing setting to the correct section with pre/suffix" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'nonstandard',
- :setting => 'yahoo', :value => 'yippee',
- :section_prefix => '-', :section_suffix => '-'))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- # This is a comment
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- -nonstandard-
- shoes = purple
- yahoo = yippee
- EOS
- )
- end
- it "should add a missing setting to the correct section with colon" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'section:sub', :setting => 'yahoo', :value => 'yippee'))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- # This is a comment
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- -nonstandard-
- shoes = purple
- yahoo = yippee
- EOS
- )
- end
- it "should modify an existing setting with a different value" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :setting => 'baz', :value => 'bazvalue2'))
- provider = described_class.new(resource)
- provider.exists?.should be true
- provider.value=('bazvalue2')
- validate_file(<<-EOS
- # This is a comment
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue2
- url = http://192.168.1.1:8080
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- -nonstandard-
- shoes = purple
- EOS
- )
- end
- it "should modify an existing setting with pre/suffix with a different value" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'nonstandard',
- :setting => 'shoes', :value => 'orange',
- :section_prefix => '-', :section_suffix => '-' ))
- provider = described_class.new(resource)
- provider.exists?.should be true
- provider.value=('orange')
- validate_file(<<-EOS
- # This is a comment
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- -nonstandard-
- shoes = orange
- EOS
- )
- end
- it "should modify an existing setting with a different value - with colon in section" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'section:sub', :setting => 'subby', :value => 'foo'))
- provider = described_class.new(resource)
- provider.exists?.should be true
- provider.value.should eq('bar')
- provider.value=('foo')
- validate_file(<<-EOS
- # This is a comment
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section:sub]
- subby=foo
- #another comment
- ; yet another comment
- -nonstandard-
- shoes = purple
- EOS
- )
- end
- it "should be able to handle settings with non alphanumbering settings " do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :setting => 'url', :value => 'http://192.168.0.1:8080'))
- provider = described_class.new(resource)
- provider.exists?.should be true
- provider.value.should eq('http://192.168.1.1:8080')
- provider.value=('http://192.168.0.1:8080')
- validate_file( <<-EOS
- # This is a comment
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.0.1:8080
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- -nonstandard-
- shoes = purple
- EOS
- )
- end
- it "should be able to handle settings with pre/suffix with non alphanumbering settings " do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'nonstandard',
- :setting => 'shoes', :value => 'http://192.168.0.1:8080',
- :section_prefix => '-', :section_suffix => '-' ))
- provider = described_class.new(resource)
- provider.exists?.should be true
- provider.value.should eq('purple')
- provider.value=('http://192.168.0.1:8080')
- validate_file( <<-EOS
- # This is a comment
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- -nonstandard-
- shoes = http://192.168.0.1:8080
- EOS
- )
- end
- it "should recognize an existing setting with the specified value" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :setting => 'baz', :value => 'bazvalue'))
- provider = described_class.new(resource)
- provider.exists?.should be true
- end
- it "should recognize an existing setting with pre/suffix with the specified value" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'nonstandard',
- :setting => 'shoes', :value => 'purple',
- :section_prefix => '-', :section_suffix => '-' ))
- provider = described_class.new(resource)
- provider.exists?.should be true
- end
- it "should add a new section if the section does not exist" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => "section3", :setting => 'huzzah', :value => 'shazaam'))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- # This is a comment
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- -nonstandard-
- shoes = purple
- [section3]
- huzzah = shazaam
- EOS
- )
- end
- it "should add a new section with pre/suffix if the section does not exist" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => "section3", :setting => 'huzzah', :value => 'shazaam',
- :section_prefix => '-', :section_suffix => '-' ))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- # This is a comment
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- -nonstandard-
- shoes = purple
- -section3-
- huzzah = shazaam
- EOS
- )
- end
- it "should add a new section if the section does not exist - with colon" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => "section:subsection", :setting => 'huzzah', :value => 'shazaam'))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- # This is a comment
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- -nonstandard-
- shoes = purple
- [section:subsection]
- huzzah = shazaam
- EOS
- )
- end
- it "should add a new section with pre/suffix if the section does not exist - with colon" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => "section:subsection", :setting => 'huzzah', :value => 'shazaam',
- :section_prefix => '-', :section_suffix => '-' ))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- # This is a comment
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- -nonstandard-
- shoes = purple
- -section:subsection-
- huzzah = shazaam
- EOS
- )
- end
- it "should add a new section if no sections exists" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => "section1", :setting => 'setting1', :value => 'hellowworld', :path => emptyfile))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file("
- [section1]
- setting1 = hellowworld
- ", emptyfile)
- end
- it "should add a new section with pre/suffix if no sections exists" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => "section1", :setting => 'setting1', :value => 'hellowworld', :path => emptyfile,
- :section_prefix => '-', :section_suffix => '-' ))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file("
- -section1-
- setting1 = hellowworld
- ", emptyfile)
- end
- it "should add a new section with colon if no sections exists" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => "section:subsection", :setting => 'setting1', :value => 'hellowworld', :path => emptyfile))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file("
- [section:subsection]
- setting1 = hellowworld
- ", emptyfile)
- end
- it "should add a new section with pre/suffix with colon if no sections exists" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => "section:subsection", :setting => 'setting1', :value => 'hellowworld', :path => emptyfile,
- :section_prefix => '-', :section_suffix => '-' ))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file("
- -section:subsection-
- setting1 = hellowworld
- ", emptyfile)
- end
- it "should be able to handle variables of any type" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => "section1", :setting => 'master', :value => true))
- provider = described_class.new(resource)
- provider.exists?.should be true
- provider.value.should == 'true'
- end
- end
- context "when dealing with a global section" do
- let(:orig_content) {
- <<-EOS
- # This is a comment
- foo=blah
- [section2]
- foo = http://192.168.1.1:8080
- ; yet another comment
- EOS
- }
- it "should add a missing setting if it doesn't exist" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => '', :setting => 'bar', :value => 'yippee'))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- # This is a comment
- foo=blah
- bar = yippee
- [section2]
- foo = http://192.168.1.1:8080
- ; yet another comment
- EOS
- )
- end
- it "should modify an existing setting with a different value" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => '', :setting => 'foo', :value => 'yippee'))
- provider = described_class.new(resource)
- provider.exists?.should be true
- provider.value.should eq('blah')
- provider.value=('yippee')
- validate_file(<<-EOS
- # This is a comment
- foo=yippee
- [section2]
- foo = http://192.168.1.1:8080
- ; yet another comment
- EOS
- )
- end
- it "should recognize an existing setting with the specified value" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => '', :setting => 'foo', :value => 'blah'))
- provider = described_class.new(resource)
- provider.exists?.should be true
- end
- end
- context "when the first line of the file is a section" do
- let(:orig_content) {
- <<-EOS
- [section2]
- foo = http://192.168.1.1:8080
- EOS
- }
- it "should be able to add a global setting" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => '', :setting => 'foo', :value => 'yippee'))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- foo = yippee
- [section2]
- foo = http://192.168.1.1:8080
- EOS
- )
- end
- it "should modify an existing setting" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'section2', :setting => 'foo', :value => 'yippee'))
- provider = described_class.new(resource)
- provider.exists?.should be true
- provider.value.should eq('http://192.168.1.1:8080')
- provider.value=('yippee')
- validate_file(<<-EOS
- [section2]
- foo = yippee
- EOS
- )
- end
- it "should add a new setting" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'section2', :setting => 'bar', :value => 'baz'))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- [section2]
- foo = http://192.168.1.1:8080
- bar = baz
- EOS
- )
- end
- end
- context "when overriding the separator" do
- let(:orig_content) {
- <<-EOS
- [section2]
- foo=bar
- EOS
- }
- it "should modify an existing setting" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'section2',
- :setting => 'foo',
- :value => 'yippee',
- :key_val_separator => '='))
- provider = described_class.new(resource)
- provider.exists?.should be true
- provider.value.should eq('bar')
- provider.value=('yippee')
- validate_file(<<-EOS
- [section2]
- foo=yippee
- EOS
- )
- end
- end
- context "when overriding the separator to something other than =" do
- let(:orig_content) {
- <<-EOS
- [section2]
- foo: bar
- EOS
- }
- it "should modify an existing setting" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'section2',
- :setting => 'foo',
- :value => 'yippee',
- :key_val_separator => ': '))
- provider = described_class.new(resource)
- provider.exists?.should be true
- provider.value.should eq('bar')
- provider.value=('yippee')
- validate_file(<<-EOS
- [section2]
- foo: yippee
- EOS
- )
- end
- it "should add a new setting" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'section2',
- :setting => 'bar',
- :value => 'baz',
- :key_val_separator => ': '))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- [section2]
- foo: bar
- bar: baz
- EOS
- )
- end
- end
- context "when overriding the separator to a space" do
- let(:orig_content) {
- <<-EOS
- [section2]
- foo bar
- EOS
- }
- it "should modify an existing setting" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'section2',
- :setting => 'foo',
- :value => 'yippee',
- :key_val_separator => ' '))
- provider = described_class.new(resource)
- provider.exists?.should be true
- provider.value.should eq('bar')
- provider.value=('yippee')
- validate_file(<<-EOS
- [section2]
- foo yippee
- EOS
- )
- end
- it "should add a new setting" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'section2',
- :setting => 'bar',
- :value => 'baz',
- :key_val_separator => ' '))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- [section2]
- foo bar
- bar baz
- EOS
- )
- end
- end
- context "when ensuring that a setting is absent" do
- let(:orig_content) {
- <<-EOS
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section3]
- # com = ment
- uncom = ment
- [section4]
- uncom = ment
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- -nonstandard-
- shoes = purple
- EOS
- }
- it "should remove a setting that exists" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'section1', :setting => 'foo', :ensure => 'absent'))
- provider = described_class.new(resource)
- provider.exists?.should be true
- provider.destroy
- validate_file(<<-EOS
- [section1]
- ; This is also a comment
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section3]
- # com = ment
- uncom = ment
- [section4]
- uncom = ment
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- -nonstandard-
- shoes = purple
- EOS
- )
- end
- it "should remove a setting with pre/suffix that exists" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'nonstandard', :setting => 'shoes', :ensure => 'absent',
- :section_prefix => '-', :section_suffix => '-' ))
- provider = described_class.new(resource)
- provider.exists?.should be true
- provider.destroy
- validate_file(<<-EOS
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section3]
- # com = ment
- uncom = ment
- [section4]
- uncom = ment
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- EOS
- )
- end
- it "should do nothing for a setting that does not exist" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'section:sub', :setting => 'foo', :ensure => 'absent'))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.destroy
- validate_file(<<-EOS
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section3]
- # com = ment
- uncom = ment
- [section4]
- uncom = ment
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- -nonstandard-
- shoes = purple
- EOS
- )
- end
- it "should do nothing for a setting with pre/suffix that does not exist" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'nonstandard', :setting => 'foo', :ensure => 'absent',
- :section_prefix => '-', :section_suffix => '-' ))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.destroy
- validate_file(<<-EOS
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section3]
- # com = ment
- uncom = ment
- [section4]
- uncom = ment
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- -nonstandard-
- shoes = purple
- EOS
- )
- end
- it "does not remove a section when the last uncommented setting is removed if there are comments" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'section3',
- :setting => 'uncom',
- :ensure => 'absent',
- ))
- provider = described_class.new(resource)
- provider.exists?.should be true
- provider.destroy
- validate_file(<<-EOS
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section3]
- # com = ment
- [section4]
- uncom = ment
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- -nonstandard-
- shoes = purple
- EOS
- )
- end
- it "removes the section when removing the last line in the section" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'section4',
- :setting => 'uncom',
- :ensure => 'absent',
- ))
- provider = described_class.new(resource)
- provider.exists?.should be true
- provider.destroy
- validate_file(<<-EOS
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section3]
- # com = ment
- uncom = ment
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- -nonstandard-
- shoes = purple
- EOS
- )
- end
- end
- context "when dealing with indentation in sections" do
- let(:orig_content) {
- <<-EOS
- # This is a comment
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section:sub]
- subby=bar
- #another comment
- fleezy = flam
- ; yet another comment
- EOS
- }
- it "should add a missing setting at the correct indentation when the header is aligned" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'section1', :setting => 'yahoo', :value => 'yippee'))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- # This is a comment
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- yahoo = yippee
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section:sub]
- subby=bar
- #another comment
- fleezy = flam
- ; yet another comment
- EOS
- )
- end
- it "should update an existing setting at the correct indentation when the header is aligned" do
- resource = Puppet::Type::Ini_setting.new(
- common_params.merge(:section => 'section1', :setting => 'bar', :value => 'barvalue2'))
- provider = described_class.new(resource)
- provider.exists?.should be true
- provider.create
- validate_file(<<-EOS
- # This is a comment
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue2
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section:sub]
- subby=bar
- #another comment
- fleezy = flam
- ; yet another comment
- EOS
- )
- end
- it "should add a missing setting at the correct indentation when the header is not aligned" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'section2', :setting => 'yahoo', :value => 'yippee'))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- # This is a comment
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- yahoo = yippee
- [section:sub]
- subby=bar
- #another comment
- fleezy = flam
- ; yet another comment
- EOS
- )
- end
- it "should update an existing setting at the correct indentation when the header is not aligned" do
- resource = Puppet::Type::Ini_setting.new(
- common_params.merge(:section => 'section2', :setting => 'baz', :value => 'bazvalue2'))
- provider = described_class.new(resource)
- provider.exists?.should be true
- provider.create
- validate_file(<<-EOS
- # This is a comment
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue2
- url = http://192.168.1.1:8080
- [section:sub]
- subby=bar
- #another comment
- fleezy = flam
- ; yet another comment
- EOS
- )
- end
- it "should add a missing setting at the min indentation when the section is not aligned" do
- resource = Puppet::Type::Ini_setting.new(
- common_params.merge(:section => 'section:sub', :setting => 'yahoo', :value => 'yippee'))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- # This is a comment
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section:sub]
- subby=bar
- #another comment
- fleezy = flam
- ; yet another comment
- yahoo = yippee
- EOS
- )
- end
- it "should update an existing setting at the previous indentation when the section is not aligned" do
- resource = Puppet::Type::Ini_setting.new(
- common_params.merge(:section => 'section:sub', :setting => 'fleezy', :value => 'flam2'))
- provider = described_class.new(resource)
- provider.exists?.should be true
- provider.create
- validate_file(<<-EOS
- # This is a comment
- [section1]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section2]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section:sub]
- subby=bar
- #another comment
- fleezy = flam2
- ; yet another comment
- EOS
- )
- end
- end
- context "when dealing settings that have a commented version present" do
- let(:orig_content) {
- <<-EOS
- [section1]
- # foo=foovalue
- bar=barvalue
- foo = foovalue2
- [section2]
- # foo = foovalue
- ;bar=barvalue
- blah = blah
- #baz=
- EOS
- }
- it "should add a new setting below a commented version of that setting" do
- resource = Puppet::Type::Ini_setting.new(
- common_params.merge(:section => 'section2', :setting => 'foo', :value => 'foo3'))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- [section1]
- # foo=foovalue
- bar=barvalue
- foo = foovalue2
- [section2]
- # foo = foovalue
- foo = foo3
- ;bar=barvalue
- blah = blah
- #baz=
- EOS
- )
- end
- it "should update an existing setting in place, even if there is a commented version of that setting" do
- resource = Puppet::Type::Ini_setting.new(
- common_params.merge(:section => 'section1', :setting => 'foo', :value => 'foo3'))
- provider = described_class.new(resource)
- provider.exists?.should be true
- provider.create
- validate_file(<<-EOS
- [section1]
- # foo=foovalue
- bar=barvalue
- foo = foo3
- [section2]
- # foo = foovalue
- ;bar=barvalue
- blah = blah
- #baz=
- EOS
- )
- end
- it "should add a new setting below a commented version of that setting, respecting semicolons as comments" do
- resource = Puppet::Type::Ini_setting.new(
- common_params.merge(:section => 'section2', :setting => 'bar', :value => 'bar3'))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- [section1]
- # foo=foovalue
- bar=barvalue
- foo = foovalue2
- [section2]
- # foo = foovalue
- ;bar=barvalue
- bar=bar3
- blah = blah
- #baz=
- EOS
- )
- end
- it "should add a new setting below an empty commented version of that setting" do
- resource = Puppet::Type::Ini_setting.new(
- common_params.merge(:section => 'section2', :setting => 'baz', :value => 'bazvalue'))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- [section1]
- # foo=foovalue
- bar=barvalue
- foo = foovalue2
- [section2]
- # foo = foovalue
- ;bar=barvalue
- blah = blah
- #baz=
- baz=bazvalue
- EOS
- )
- end
- context 'when a section only contains comments' do
- let(:orig_content) {
- <<-EOS
- [section1]
- # foo=foovalue
- # bar=bar2
- EOS
- }
- it 'should be able to add a new setting when a section contains only comments' do
- resource = Puppet::Type::Ini_setting.new(
- common_params.merge(:section => 'section1', :setting => 'foo', :value => 'foovalue2')
- )
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- [section1]
- # foo=foovalue
- foo=foovalue2
- # bar=bar2
- EOS
- )
- end
- it 'should be able to add a new setting when it matches a commented out line other than the first one' do
- resource = Puppet::Type::Ini_setting.new(
- common_params.merge(:section => 'section1', :setting => 'bar', :value => 'barvalue2')
- )
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- [section1]
- # foo=foovalue
- # bar=bar2
- bar=barvalue2
- EOS
- )
- end
- end
- context "when sections have spaces and dashes" do
- let(:orig_content) {
- <<-EOS
- # This is a comment
- [section - one]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section - two]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- EOS
- }
- it "should add a missing setting to the correct section" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'section - two', :setting => 'yahoo', :value => 'yippee'))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- # This is a comment
- [section - one]
- ; This is also a comment
- foo=foovalue
- bar = barvalue
- master = true
- [section - two]
- foo= foovalue2
- baz=bazvalue
- url = http://192.168.1.1:8080
- yahoo = yippee
- [section:sub]
- subby=bar
- #another comment
- ; yet another comment
- EOS
- )
- end
- end
- end
- context "when sections have spaces and quotations" do
- let(:orig_content) do
- <<-EOS
- [branch "master"]
- remote = origin
- merge = refs/heads/master
- [alias]
- to-deploy = log --merges --grep='pull request' --format='%s (%cN)' origin/production..origin/master
- [branch "production"]
- remote = origin
- merge = refs/heads/production
- EOS
- end
- it "should add a missing setting to the correct section" do
- resource = Puppet::Type::Ini_setting.new(common_params.merge(
- :section => 'alias',
- :setting => 'foo',
- :value => 'bar'
- ))
- provider = described_class.new(resource)
- provider.exists?.should be false
- provider.create
- validate_file(<<-EOS
- [branch "master"]
- remote = origin
- merge = refs/heads/master
- [alias]
- to-deploy = log --merges --grep='pull request' --format='%s (%cN)' origin/production..origin/master
- foo = bar
- [branch "production"]
- remote = origin
- merge = refs/heads/production
- EOS
- )
- end
- end
- end
|