Permalink
Browse files

Use http_proxy to avoid warnings.

Stop restoring already empty http proxy ENV variables

Finished #199
  • Loading branch information...
1 parent 8aea059 commit 54fc13b16dc8ddbcbaf8e20bfd7542fb4dc48d4e @drbrain drbrain committed Nov 27, 2012
Showing with 4 additions and 30 deletions.
  1. +4 −30 test/rubygems/test_gem_remote_fetcher.rb
@@ -642,47 +642,21 @@ def conn.request(req)
end
def test_observe_no_proxy_env_single_host
- orig_env_HTTP_PROXY = ENV['HTTP_PROXY']
-
- orig_env_NO_PROXY = ENV["NO_PROXY"]
- orig_env_no_proxy = ENV["no_proxy"]
-
use_ui @ui do
- ENV["HTTP_PROXY"] = @proxy_uri
- ENV["NO_PROXY"] = "#{URI::parse(@server_uri).host}"
+ ENV["http_proxy"] = @proxy_uri
+ ENV["no_proxy"] = URI::parse(@server_uri).host
fetcher = Gem::RemoteFetcher.new nil
assert_data_from_server fetcher.fetch_path(@server_uri)
end
-
- ensure
- orig_env_HTTP_PROXY.nil? ? ENV.delete('HTTP_PROXY') :
- ENV['HTTP_PROXY'] = orig_env_HTTP_PROXY
- orig_env_NO_PROXY.nil? ? ENV.delete('NO_PROXY') :
- ENV['NO_PROXY'] = orig_env_NO_PROXY
- orig_env_no_proxy.nil? ? ENV.delete('no_proxy') :
- ENV['no_proxy'] = orig_env_no_proxy
end
def test_observe_no_proxy_env_list
- orig_env_HTTP_PROXY = ENV['HTTP_PROXY']
-
- orig_env_NO_PROXY = ENV["NO_PROXY"]
- orig_env_no_proxy = ENV["no_proxy"]
-
use_ui @ui do
- ENV["HTTP_PROXY"] = @proxy_uri
- ENV["NO_PROXY"] = "fakeurl.com, #{URI::parse(@server_uri).host}"
+ ENV["http_proxy"] = @proxy_uri
+ ENV["no_proxy"] = "fakeurl.com, #{URI::parse(@server_uri).host}"
fetcher = Gem::RemoteFetcher.new nil
assert_data_from_server fetcher.fetch_path(@server_uri)
end
-
- ensure
- orig_env_HTTP_PROXY.nil? ? ENV.delete('HTTP_PROXY') :
- ENV['HTTP_PROXY'] = orig_env_HTTP_PROXY
- orig_env_NO_PROXY.nil? ? ENV.delete('NO_PROXY') :
- ENV['NO_PROXY'] = orig_env_NO_PROXY
- orig_env_no_proxy.nil? ? ENV.delete('no_proxy') :
- ENV['no_proxy'] = orig_env_no_proxy
end
def test_request

0 comments on commit 54fc13b

Please sign in to comment.