diff options
author | sdoyon@chromium.org <sdoyon@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-04-20 16:50:56 +0000 |
---|---|---|
committer | sdoyon@chromium.org <sdoyon@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-04-20 16:50:56 +0000 |
commit | 861c6c6d56add9157e04aa2ac43bec8e33bf1cc1 (patch) | |
tree | 835dd6466912cd073e3ac16177252fe0c8865c93 /net/proxy/proxy_config_service_win_unittest.cc | |
parent | bc1529bbafc42d96d9ac0839bcb0ba1ab9328445 (diff) | |
download | chromium_src-861c6c6d56add9157e04aa2ac43bec8e33bf1cc1.zip chromium_src-861c6c6d56add9157e04aa2ac43bec8e33bf1cc1.tar.gz chromium_src-861c6c6d56add9157e04aa2ac43bec8e33bf1cc1.tar.bz2 |
ProxyConfigService for Linux.
Establishes a ProxyConfig by reading settings from gconf or consulting
environment variables.
BUG=8143
Thanks to ermilov.maxim@gmail.com for his contribution: some ideas<
and code snippets from his patch were folded into this one.
(See http://codereview.chromium.org/49009)
Review URL: http://codereview.chromium.org/60009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14034 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/proxy/proxy_config_service_win_unittest.cc')
-rw-r--r-- | net/proxy/proxy_config_service_win_unittest.cc | 43 |
1 files changed, 4 insertions, 39 deletions
diff --git a/net/proxy/proxy_config_service_win_unittest.cc b/net/proxy/proxy_config_service_win_unittest.cc index f77ac16..c08b1a9 100644 --- a/net/proxy/proxy_config_service_win_unittest.cc +++ b/net/proxy/proxy_config_service_win_unittest.cc @@ -6,38 +6,11 @@ #include "net/base/net_errors.h" #include "net/proxy/proxy_config.h" +#include "net/proxy/proxy_config_service_common_unittest.h" #include "testing/gtest/include/gtest/gtest.h" namespace net { -static ProxyConfig::ProxyRules MakeProxyRules( - ProxyConfig::ProxyRules::Type type, - const char* single_proxy, - const char* proxy_for_http, - const char* proxy_for_https, - const char* proxy_for_ftp) { - ProxyConfig::ProxyRules rules; - rules.type = type; - rules.single_proxy = ProxyServer::FromURI(single_proxy); - rules.proxy_for_http = ProxyServer::FromURI(proxy_for_http); - rules.proxy_for_https = ProxyServer::FromURI(proxy_for_https); - rules.proxy_for_ftp = ProxyServer::FromURI(proxy_for_ftp); - return rules; -} - -static ProxyConfig::ProxyRules MakeSingleProxyRules(const char* single_proxy) { - return MakeProxyRules(ProxyConfig::ProxyRules::TYPE_SINGLE_PROXY, - single_proxy, "", "", ""); -} - -static ProxyConfig::ProxyRules MakeProxyPerSchemeRules( - const char* proxy_http, - const char* proxy_https, - const char* proxy_ftp) { - return MakeProxyRules(ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME, - "", proxy_http, proxy_https, proxy_ftp); -} - TEST(ProxyConfigServiceWinTest, SetFromIEConfig) { const struct { // Input. @@ -193,18 +166,10 @@ TEST(ProxyConfigServiceWinTest, SetFromIEConfig) { EXPECT_EQ(tests[i].auto_detect, config.auto_detect); EXPECT_EQ(tests[i].pac_url, config.pac_url); - - // Join the proxy bypass list using "\n" to make it into a single string. - std::string flattened_proxy_bypass; - typedef std::vector<std::string> BypassList; - for (BypassList::const_iterator it = config.proxy_bypass.begin(); - it != config.proxy_bypass.end(); ++it) { - flattened_proxy_bypass += *it + "\n"; - } - - EXPECT_EQ(tests[i].proxy_bypass_list, flattened_proxy_bypass); + EXPECT_EQ(tests[i].proxy_bypass_list, + FlattenProxyBypass(config.proxy_bypass)); EXPECT_EQ(tests[i].bypass_local_names, config.proxy_bypass_local_names); - EXPECT_TRUE(tests[i].proxy_rules == config.proxy_rules); + EXPECT_EQ(tests[i].proxy_rules, config.proxy_rules); } } |