diff options
author | Jay Mundrawala <jdmundrawala@gmail.com> | 2014-10-13 05:36:41 -0700 |
---|---|---|
committer | Jay Mundrawala <jdmundrawala@gmail.com> | 2014-10-13 05:49:33 -0700 |
commit | 2387bdb975053f7b817d0fbcbae0f05711ed4d2e (patch) | |
tree | f34f4f6e718cab00f4a2cae03a8655be2b4105a3 | |
parent | 6576eb994c1e886e9cef22840938e643ddb47bd7 (diff) | |
download | chef-2387bdb975053f7b817d0fbcbae0f05711ed4d2e.tar.gz |
Merge pull request #2208 from opscode/jdmundrawala/env-path-spec-fixjdmundrawala/11-stable-env-path-spec-fix
restore ENV['PATH'] in env_spec after test is complete
-rwxr-xr-x | spec/functional/resource/env_spec.rb | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/spec/functional/resource/env_spec.rb b/spec/functional/resource/env_spec.rb index 24fe5e1dff..cf77fef703 100755 --- a/spec/functional/resource/env_spec.rb +++ b/spec/functional/resource/env_spec.rb @@ -126,7 +126,8 @@ describe Chef::Resource::Env, :windows_only do context 'when using PATH' do let(:random_name) { Time.now.to_i } let(:env_val) { "#{env_value_expandable}_#{random_name}"} - let(:path_before) { test_resource.provider_for_action(test_resource.action).env_value('PATH') } + let!(:path_before) { test_resource.provider_for_action(test_resource.action).env_value('PATH') || '' } + let!(:env_path_before) { ENV['PATH'] } it 'should expand PATH' do path_before.should_not include(env_val) @@ -142,9 +143,7 @@ describe Chef::Resource::Env, :windows_only do test_resource.key_name('PATH') test_resource.value(path_before) test_resource.run_action(:create) - if test_resource.provider_for_action(test_resource.action).env_value('PATH') != path_before - raise 'Failed to cleanup after ourselves' - end + ENV['PATH'] = env_path_before end end |