From 7d0b5709c9bd3c5dfce30abf58e5637258ac54c4 Mon Sep 17 00:00:00 2001 From: Paul Belanger Date: Sat, 23 Jun 2012 17:13:08 -0400 Subject: Lets use puppetlabs_spec_helper Signed-off-by: Paul Belanger --- .fixtures.yml | 3 +++ Rakefile | 15 ++--------- spec/defines/init_spec.rb | 57 +++++++++++++++++++++++++++++++++-------- spec/fixtures/manifests/site.pp | 0 spec/spec_helper.rb | 10 +------- 5 files changed, 53 insertions(+), 32 deletions(-) create mode 100644 .fixtures.yml create mode 100644 spec/fixtures/manifests/site.pp diff --git a/.fixtures.yml b/.fixtures.yml new file mode 100644 index 0000000..2d6fee0 --- /dev/null +++ b/.fixtures.yml @@ -0,0 +1,3 @@ +fixtures: + symlinks: + 'concat': '#{source_dir}' diff --git a/Rakefile b/Rakefile index 764aebd..14f1c24 100644 --- 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 d968a26..34fb24b 100644 --- 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 index 0000000..e69de29 diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb index e6e9309..2c6f566 100644 --- 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' -- cgit v1.2.3