diff --git a/spec/classes/tripleo_certmonger_ovn_dbs.rb b/spec/classes/tripleo_certmonger_ovn_dbs_spec.rb similarity index 100% rename from spec/classes/tripleo_certmonger_ovn_dbs.rb rename to spec/classes/tripleo_certmonger_ovn_dbs_spec.rb diff --git a/spec/unit/provider/liquidio_config/ini_setting.rb b/spec/unit/provider/liquidio_config/ini_setting_spec.rb similarity index 69% rename from spec/unit/provider/liquidio_config/ini_setting.rb rename to spec/unit/provider/liquidio_config/ini_setting_spec.rb index b2985d120..34babbcd6 100644 --- a/spec/unit/provider/liquidio_config/ini_setting.rb +++ b/spec/unit/provider/liquidio_config/ini_setting_spec.rb @@ -1,10 +1,32 @@ -require 'puppet' +# +# these tests are a little concerning b/c they are hacking around the +# modulepath, so these tests will not catch issues that may eventually arise +# related to loading these plugins. +# I could not, for the life of me, figure out how to programmatically set the modulepath +$LOAD_PATH.push( + File.join( + File.dirname(__FILE__), + '..', + '..', + '..', + 'fixtures', + 'modules', + 'inifile', + 'lib') +) +$LOAD_PATH.push( + File.join( + File.dirname(__FILE__), + '..', + '..', + '..', + 'fixtures', + 'modules', + 'openstacklib', + 'lib') +) require 'spec_helper' -require 'puppet/provider/liquidio/ini_setting' - -provider_class = Puppet::Type.type(:liquidio_config). - provider(:ini_setting) - +provider_class = Puppet::Type.type(:liquidio_config).provider(:ini_setting) describe provider_class do it 'should default to the default setting when no other one is specified' do diff --git a/spec/unit/type/liquidio_config_sepc.rb b/spec/unit/type/liquidio_config_spec.rb similarity index 93% rename from spec/unit/type/liquidio_config_sepc.rb rename to spec/unit/type/liquidio_config_spec.rb index 90b52116c..90183e17d 100644 --- a/spec/unit/type/liquidio_config_sepc.rb +++ b/spec/unit/type/liquidio_config_spec.rb @@ -29,12 +29,12 @@ describe 'Puppet::Type.type(:liquidio_config)' do it 'should accept a valid value' do @liquidio_config[:value] = 'bar' - expect(@liquidio_config[:value]).to eq(['bar']) + expect(@liquidio_config[:value]).to eq('bar') end it 'should not accept a value with whitespace' do @liquidio_config[:value] = 'b ar' - expect(@liquidio_config[:value]).to eq(['b ar']) + expect(@liquidio_config[:value]).to eq('b ar') end it 'should accept valid ensure values' do