diff options
author | David Schmitt <david.schmitt@puppet.com> | 2016-09-06 09:55:34 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2016-09-06 09:55:34 +0100 |
commit | 0262410b696f8d3644fb0dbd08db8b4794810eed (patch) | |
tree | f0388a89e8d6892d982d9935f8f8ff8314b1af0f /spec/functions/is_float_spec.rb | |
parent | 56c9587e1124bebd88017e8dd82c1d6e584c7d97 (diff) | |
parent | 6c6c6d8e3448e3072d590a0782237486e46bc88d (diff) |
Merge pull request #641 from HelenCampbell/deprecationUpdate
Deprecation function to be mutable in all cases
Diffstat (limited to 'spec/functions/is_float_spec.rb')
-rwxr-xr-x | spec/functions/is_float_spec.rb | 25 |
1 files changed, 19 insertions, 6 deletions
diff --git a/spec/functions/is_float_spec.rb b/spec/functions/is_float_spec.rb index cb7e148..44bdc48 100755 --- a/spec/functions/is_float_spec.rb +++ b/spec/functions/is_float_spec.rb @@ -1,13 +1,8 @@ require 'spec_helper' describe 'is_float' 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(3) - end + it { is_expected.not_to eq(nil) } it { is_expected.to run.with_params().and_raise_error(Puppet::ParseError, /wrong number of arguments/i) } it { is_expected.to run.with_params(0.1, 0.2).and_raise_error(Puppet::ParseError, /wrong number of arguments/i) } @@ -26,4 +21,22 @@ describe 'is_float' do it { is_expected.to run.with_params(1.0).and_return(true) } it { is_expected.to run.with_params(1).and_return(false) } end + + 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(2.2).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(1.0).and_return(true) + end + end + end |