Add common test cases for cinder_config resource

Change-Id: I19fa59b387079db43e5268acc7a8934ea3120bb3

spec/unit/type/cinder_api_paste_ini_spec.rb

@@ -1,21 +1,11 @@
-require 'spec_helper'
+require 'puppet'
-# this hack is required for now to ensure that the path is set up correctly
-# to retrieve the parent provider
-$LOAD_PATH.push(
-  File.join(
-    File.dirname(__FILE__),
-    '..',
-    '..',
-    'fixtures',
-    'modules',
-    'inifile',
-    'lib')
-)
 require 'puppet/type/cinder_api_paste_ini'
+
 describe 'Puppet::Type.type(:cinder_api_paste_ini)' do
   before :each do
     @cinder_api_paste_ini = Puppet::Type.type(:cinder_api_paste_ini).new(:name => 'DEFAULT/foo', :value => 'bar')
   end
+
   it 'should accept a valid value' do
     @cinder_api_paste_ini[:value] = 'bar'
     expect(@cinder_api_paste_ini[:value]).to eq('bar')

spec/unit/type/cinder_config_spec.rb

@@ -6,6 +6,51 @@ describe 'Puppet::Type.type(:cinder_config)' do
     @cinder_config = Puppet::Type.type(:cinder_config).new(:name => 'DEFAULT/foo', :value => 'bar')
   end
 
+  it 'should require a name' do
+    expect {
+      Puppet::Type.type(:cinder_config).new({})
+    }.to raise_error(Puppet::Error, 'Title or name must be provided')
+  end
+
+  it 'should not expect a name with whitespace' do
+    expect {
+      Puppet::Type.type(:cinder_config).new(:name => 'f oo')
+    }.to raise_error(Puppet::Error, /Parameter name failed/)
+  end
+
+  it 'should fail when there is no section' do
+    expect {
+      Puppet::Type.type(:cinder_config).new(:name => 'foo')
+    }.to raise_error(Puppet::Error, /Parameter name failed/)
+  end
+
+  it 'should not require a value when ensure is absent' do
+    Puppet::Type.type(:cinder_config).new(:name => 'DEFAULT/foo', :ensure => :absent)
+  end
+
+  it 'should accept a valid value' do
+    @cinder_config[:value] = 'bar'
+    expect(@cinder_config[:value]).to eq('bar')
+  end
+
+  it 'should not accept a value with whitespace' do
+    @cinder_config[:value] = 'b ar'
+    expect(@cinder_config[:value]).to eq('b ar')
+  end
+
+  it 'should accept valid ensure values' do
+    @cinder_config[:ensure] = :present
+    expect(@cinder_config[:ensure]).to eq(:present)
+    @cinder_config[:ensure] = :absent
+    expect(@cinder_config[:ensure]).to eq(:absent)
+  end
+
+  it 'should not accept invalid ensure values' do
+    expect {
+      @cinder_config[:ensure] = :latest
+    }.to raise_error(Puppet::Error, /Invalid value/)
+  end
+
   it 'should autorequire the package that install the file' do
     catalog = Puppet::Resource::Catalog.new
     anchor = Puppet::Type.type(:anchor).new(:name => 'cinder::install::end')