diff options
author | Helen Campbell <helen@puppetlabs.com> | 2016-08-23 15:02:39 +0100 |
---|---|---|
committer | Helen Campbell <helen@puppetlabs.com> | 2016-09-05 17:43:56 +0100 |
commit | 6c6c6d8e3448e3072d590a0782237486e46bc88d (patch) | |
tree | b65308e8b83806063cbd7f20507c57599f0efa8c /spec/functions/is_string_spec.rb | |
parent | 0acff935f8b12938f0a21769a8959a8ba983e60d (diff) |
Deprecation function to be mutable in all cases
Diffstat (limited to 'spec/functions/is_string_spec.rb')
-rwxr-xr-x | spec/functions/is_string_spec.rb | 24 |
1 files changed, 19 insertions, 5 deletions
diff --git a/spec/functions/is_string_spec.rb b/spec/functions/is_string_spec.rb index 4f59b63..460d6e0 100755 --- a/spec/functions/is_string_spec.rb +++ b/spec/functions/is_string_spec.rb @@ -1,12 +1,8 @@ require 'spec_helper' describe 'is_string' do + it { is_expected.not_to eq(nil) } - # Checking for deprecation warning - it 'should display a single deprecation' do - scope.expects(:warning).with(includes('This method is deprecated')) - is_expected.to run.with_params('ha') - end it { is_expected.to run.with_params().and_raise_error(Puppet::ParseError, /wrong number of arguments/i) } it { pending("Current implementation ignores parameters after the first.") @@ -30,4 +26,22 @@ describe 'is_string' do it { is_expected.to run.with_params(false).and_return(false) } it { is_expected.to run.with_params('one').and_return(true) } it { is_expected.to run.with_params('0001234').and_return(true) } + + context 'Checking for deprecation warning' do + after(:context) do + ENV.delete('STDLIB_LOG_DEPRECATIONS') + end + # Checking for deprecation warning, which should only be provoked when the env variable for it is set. + it 'should display a single deprecation' do + ENV['STDLIB_LOG_DEPRECATIONS'] = "true" + scope.expects(:warning).with(includes('This method is deprecated')) + is_expected.to run.with_params('sponge').and_return(true) + end + it 'should display no warning for deprecation' do + ENV['STDLIB_LOG_DEPRECATIONS'] = "false" + scope.expects(:warning).with(includes('This method is deprecated')).never + is_expected.to run.with_params('bob').and_return(true) + end + end + end |