Lets use puppetlabs_spec_helper

Signed-off-by: Paul Belanger <paul.belanger@polybeacon.com>

diff --git a/.fixtures.yml b/.fixtures.yml
new file mode 100644 (file)
index 0000000..2d6fee0
--- /dev/null
+++ b/.fixtures.yml
@@ -0,0 +1,3 @@
+fixtures:
+  symlinks:
+    'concat': '#{source_dir}'

diff --git a/Rakefile b/Rakefile
index 764aebd26dfd994ea26ae6280b77fc8f28108861..14f1c24622acbaba717085fbc8d1d20786543522 100644 (file)
--- a/Rakefile
+++ b/Rakefile
@@ -1,13 +1,2 @@
-require 'rake'
-require 'rspec/core/rake_task'
-
-task :default => [:spec]
-
-desc "Run all module spec tests (Requires rspec-puppet gem)"
-RSpec::Core::RakeTask.new(:spec)
-
-desc "Build package"
-task :build do
-  system("puppet-module build")
-end
-
+require 'rubygems'
+require 'puppetlabs_spec_helper/rake_tasks'

diff --git a/spec/defines/init_spec.rb b/spec/defines/init_spec.rb
index d968a26c8254ce791374a5760fb26739f1bcb8a4..34fb24bfcf6bcc9504fcf9769910ee45a6017738 100644 (file)
--- a/spec/defines/init_spec.rb
+++ b/spec/defines/init_spec.rb
@@ -3,18 +3,55 @@ require 'spec_helper'
 describe 'concat' do
   basedir = '/var/lib/puppet/concat'
   let(:title) { '/etc/foo.bar' }
-  let(:facts) { { :concat_basedir => '/var/lib/puppet/concat' } }
+  let(:facts) { { 
+    :concat_basedir => '/var/lib/puppet/concat',
+    :id             => 'root',
+  } }
   let :pre_condition do
     'include concat::setup'
   end
-  it { should contain_file("#{basedir}/_etc_foo.bar").with('ensure' => 'directory') }
-  it { should contain_file("#{basedir}/_etc_foo.bar/fragments").with('ensure' => 'directory') }
 
-  it { should contain_file("#{basedir}/_etc_foo.bar/fragments.concat").with('ensure' => 'present') }
-  it { should contain_file("/etc/foo.bar").with('ensure' => 'present') }
-  it { should contain_exec("concat_/etc/foo.bar").with_command(
-                                        "#{basedir}/bin/concatfragments.sh "+
-                                        "-o #{basedir}/_etc_foo.bar/fragments.concat.out "+
-                                        "-d #{basedir}/_etc_foo.bar   ")
-  }
+  directories = [
+    "#{basedir}/_etc_foo.bar",
+    "#{basedir}/_etc_foo.bar/fragments",
+  ]
+
+  directories.each do |dirs|
+    it do
+      should contain_file(dirs).with({
+        'ensure'  => 'directory',
+        'backup'  => 'puppet',
+        'group'   => 0,
+        'mode'    => '0644',
+        'owner'   => 'root',
+      })
+    end
+  end
+
+  files = [
+    "/etc/foo.bar",
+    "#{basedir}/_etc_foo.bar/fragments.concat",
+  ]
+
+  files.each do |file|
+    it do
+      should contain_file(file).with({
+        'ensure'  => 'present',
+        'backup'  => 'puppet',
+        'group'   => 0,
+        'mode'    => '0644',
+        'owner'   => 'root',
+      })
+    end
+  end
+
+  it do
+    should contain_exec("concat_/etc/foo.bar").with_command(
+      "#{basedir}/bin/concatfragments.sh " +
+      "-o #{basedir}/_etc_foo.bar/fragments.concat.out " +
+      "-d #{basedir}/_etc_foo.bar   "
+    )
+  end
 end
+
+# vim:sw=2:ts=2:expandtab:textwidth=79

diff --git a/spec/fixtures/manifests/site.pp b/spec/fixtures/manifests/site.pp
new file mode 100644 (file)
index 0000000..e69de29

diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
index e6e9309bc97ecdf9d67769211a00db3334fa749f..2c6f56649aeb15c37a49bf07898570e8ab51cf97 100644 (file)
--- a/spec/spec_helper.rb
+++ b/spec/spec_helper.rb
@@ -1,9 +1 @@
-require 'puppet'
-require 'rspec'
-require 'rspec-puppet'
-
-RSpec.configure do |c|
-  c.module_path = File.expand_path(File.join(File.dirname(__FILE__), 'fixtures/modules/'))
-  # Using an empty site.pp file to avoid: https://github.com/rodjek/rspec-puppet/issues/15
-  c.manifest_dir = File.expand_path(File.join(File.dirname(__FILE__), 'fixtures/manifests'))
-end
+require 'puppetlabs_spec_helper/module_spec_helper'