From: Daniele Sluijters <daenney@users.noreply.github.com>
Date: Sat, 21 Feb 2015 23:22:59 +0000 (+0100)
Subject: Merge pull request #435 from mhaskel/setting_fix
X-Git-Tag: 2.0.0~38
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=7e7e3e2f50dd12db1f0c07a5c3df660c5d39c94d;hp=f88008146a11206901cb9f45a73153dedaa7a282;p=puppet-modules%2Fpuppetlabs-apt.git

Merge pull request #435 from mhaskel/setting_fix

Type is a reserved word in puppet 4
---

diff --git a/manifests/setting.pp b/manifests/setting.pp
index 99549e6..8d876fa 100644
--- a/manifests/setting.pp
+++ b/manifests/setting.pp
@@ -1,5 +1,5 @@
 define apt::setting (
-  $type,
+  $setting_type,
   $priority   = 50,
   $ensure     = file,
   $source     = undef,
@@ -17,7 +17,7 @@ define apt::setting (
     fail('apt::setting needs either of content or source')
   }
 
-  validate_re($type, ['conf', 'pref', 'list'])
+  validate_re($setting_type, ['conf', 'pref', 'list'])
   validate_re($ensure,  ['file', 'present', 'absent'])
 
   unless is_integer($priority) {
@@ -32,14 +32,14 @@ define apt::setting (
     validate_string($content)
   }
 
-  if $type == 'list' {
+  if $setting_type == 'list' {
     $_priority = ''
   } else {
     $_priority = $priority
   }
 
-  $_path = $::apt::config_files[$type]['path']
-  $_ext  = $::apt::config_files[$type]['ext']
+  $_path = $::apt::config_files[$setting_type]['path']
+  $_ext  = $::apt::config_files[$setting_type]['ext']
 
   file { "${_path}/${_priority}${title}${_ext}":
     ensure  => $ensure,
diff --git a/spec/defines/setting_spec.rb b/spec/defines/setting_spec.rb
index 7368ec3..3736dc9 100644
--- a/spec/defines/setting_spec.rb
+++ b/spec/defines/setting_spec.rb
@@ -5,39 +5,39 @@ describe 'apt::setting' do
   let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian' } }
   let(:title) { 'teddybear' }
 
-  let(:default_params) { { :type => 'conf', :content => 'di' } }
+  let(:default_params) { { :setting_type => 'conf', :content => 'di' } }
 
   describe 'when using the defaults' do
-    context 'without type' do
+    context 'without setting_type' do
       it do
-        expect { should compile }.to raise_error(Puppet::Error, /Must pass type /)
+        expect { should compile }.to raise_error(Puppet::Error, /Must pass setting_type /)
       end
     end
 
     context 'without source or content' do
-      let(:params) { { :type => 'conf' } }
+      let(:params) { { :setting_type => 'conf' } }
       it do
         expect { should compile }.to raise_error(Puppet::Error, /needs either of /)
       end
     end
 
-    context 'with type=conf' do
+    context 'with setting_type=conf' do
       let(:params) { default_params }
       it { should contain_file('/etc/apt/apt.conf.d/50teddybear') }
     end
 
-    context 'with type=pref' do
-      let(:params) { { :type => 'pref', :content => 'di' } }
+    context 'with setting_type=pref' do
+      let(:params) { { :setting_type => 'pref', :content => 'di' } }
       it { should contain_file('/etc/apt/preferences.d/50teddybear') }
     end
 
-    context 'with type=list' do
-      let(:params) { { :type => 'list', :content => 'di' } }
+    context 'with setting_type=list' do
+      let(:params) { { :setting_type => 'list', :content => 'di' } }
       it { should contain_file('/etc/apt/sources.list.d/teddybear.list') }
     end
 
     context 'with source' do
-      let(:params) { { :type => 'conf', :source => 'puppet:///la/die/dah' } }
+      let(:params) { { :setting_type => 'conf', :source => 'puppet:///la/die/dah' } }
       it {
         should contain_file('/etc/apt/apt.conf.d/50teddybear').with({
         :ensure => 'file',
@@ -68,8 +68,8 @@ describe 'apt::setting' do
       end
     end
 
-    context 'with type=ext' do
-      let(:params) { default_params.merge({ :type => 'ext' }) }
+    context 'with setting_type=ext' do
+      let(:params) { default_params.merge({ :setting_type => 'ext' }) }
       it do
         expect { should compile }.to raise_error(Puppet::Error, /"ext" does not /)
       end