Browse Source

Additional tests - multiple ini settings creation

Helen Campbell 7 years ago
parent
commit
fff45971c8

+ 1 - 0
.fixtures.yml

@@ -1,3 +1,4 @@
 fixtures:
   symlinks:
     inifile: "#{source_dir}"
+    create_multiple_ini_settings: "#{source_dir}/spec/fixtures/create_multiple_ini_settings"

+ 2 - 1
.gitignore

@@ -2,7 +2,8 @@
 pkg/
 Gemfile.lock
 vendor/
-spec/fixtures/
+spec/fixtures/manifests/
+spec/fixtures/modules/
 .vagrant/
 .bundle/
 coverage/

+ 2 - 0
README.markdown

@@ -449,6 +449,8 @@ Default value: '{}'.
 
 This module has been tested on [all PE-supported platforms](https://forge.puppetlabs.com/supported#compat-matrix), and no issues have been identified. Additionally, it is tested (but not supported) on Windows 7, Mac OS X 10.9, and Solaris 12.
 
+Due to (PUP-4709) the create_ini_settings function will cause errors when attempting to create multiple ini_settings in one go when using Puppet 4.0.x or 4.1.x. If needed, the temporary fix for this can be found here: https://github.com/puppetlabs/puppetlabs-inifile/pull/196.
+
 ##Development
 
 Puppet Labs modules on the Puppet Forge are open projects, and community contributions are essential for keeping them great. We can't access the huge number of platforms and myriad of hardware, software, and deployment configurations that Puppet is intended to serve.

+ 5 - 0
spec/classes/create_multiple_ini_settings_spec.rb

@@ -0,0 +1,5 @@
+require 'spec_helper'
+
+describe 'create_multiple_ini_settings' do
+  it { is_expected.to compile }
+end

+ 15 - 0
spec/fixtures/create_multiple_ini_settings/manifests/init.pp

@@ -0,0 +1,15 @@
+class create_multiple_ini_settings {
+
+$defaults = { 'path' => '/tmp/foo.ini' }
+$example = {
+  'section1' => {
+    'setting1'  => 'value1',
+    'settings2' => {
+      'ensure' => 'absent'
+    }
+  }
+}
+create_ini_settings($example, $defaults)
+
+}
+