diff options
author | Dan Bode <dan@puppetlabs.com> | 2012-08-15 15:56:57 -0700 |
---|---|---|
committer | Dan Bode <dan@puppetlabs.com> | 2012-08-15 15:56:57 -0700 |
commit | b42c82af19f64ed4398863092971c7ba81298a02 (patch) | |
tree | b3566a13a2c8fd1930fa826f49f4ba5935de2ed1 /spec/functions | |
parent | 1e0983362464e8f2832239b09cdbc9175a51e6ec (diff) | |
download | puppet-stdlib-b42c82af19f64ed4398863092971c7ba81298a02.tar.gz puppet-stdlib-b42c82af19f64ed4398863092971c7ba81298a02.tar.bz2 |
Revert "Revert "Merge pull request #86 from bodepd/ensure_resource""
This reverts commit 1e0983362464e8f2832239b09cdbc9175a51e6ec.
Diffstat (limited to 'spec/functions')
-rw-r--r-- | spec/functions/defined_with_params_spec.rb | 37 | ||||
-rw-r--r-- | spec/functions/ensure_resource_spec.rb | 40 |
2 files changed, 77 insertions, 0 deletions
diff --git a/spec/functions/defined_with_params_spec.rb b/spec/functions/defined_with_params_spec.rb new file mode 100644 index 0000000..28dbab3 --- /dev/null +++ b/spec/functions/defined_with_params_spec.rb @@ -0,0 +1,37 @@ +#! /usr/bin/env ruby -S rspec +require 'spec_helper' + +require 'rspec-puppet' +describe 'defined_with_params' do + describe 'when a resource is not specified' do + it { should run.with_params().and_raise_error(ArgumentError) } + end + describe 'when compared against a resource with no attributes' do + let :pre_condition do + 'user { "dan": }' + end + it do + should run.with_params('User[dan]', {}).and_return(true) + should run.with_params('User[bob]', {}).and_return(false) + should run.with_params('User[dan]', {'foo' => 'bar'}).and_return(false) + end + end + + describe 'when compared against a resource with attributes' do + let :pre_condition do + 'user { "dan": ensure => present, shell => "/bin/csh", managehome => false}' + end + it do + should run.with_params('User[dan]', {}).and_return(true) + should run.with_params('User[dan]', '').and_return(true) + should run.with_params('User[dan]', {'ensure' => 'present'} + ).and_return(true) + should run.with_params('User[dan]', + {'ensure' => 'present', 'managehome' => false} + ).and_return(true) + should run.with_params('User[dan]', + {'ensure' => 'absent', 'managehome' => false} + ).and_return(false) + end + end +end diff --git a/spec/functions/ensure_resource_spec.rb b/spec/functions/ensure_resource_spec.rb new file mode 100644 index 0000000..611666e --- /dev/null +++ b/spec/functions/ensure_resource_spec.rb @@ -0,0 +1,40 @@ +#! /usr/bin/env ruby -S rspec +require 'spec_helper' + +require 'rspec-puppet' +describe 'ensure_resource' do + describe 'when a type or title is not specified' do + it do + should run.with_params().and_raise_error(ArgumentError) + should run.with_params(['type']).and_raise_error(ArgumentError) + end + end + describe 'when compared against a resource with no attributes' do + let :pre_condition do + 'user { "dan": }' + end + it do + should run.with_params('user', 'dan', {}) + compiler.catalog.resource('User[dan]').to_s.should == 'User[dan]' + end + end + + describe 'when compared against a resource with attributes' do + let :pre_condition do + 'user { "dan": ensure => present, shell => "/bin/csh", managehome => false}' + end + it do + # these first three should not fail + should run.with_params('User', 'dan', {}) + should run.with_params('User', 'dan', '') + should run.with_params('User', 'dan', {'ensure' => 'present'}) + should run.with_params('User', 'dan', + {'ensure' => 'present', 'managehome' => false} + ) + # test that this fails + should run.with_params('User', 'dan', + {'ensure' => 'absent', 'managehome' => false} + ).and_raise_error(Puppet::Error) + end + end +end |