summaryrefslogtreecommitdiffstats
path: root/net/proxy/proxy_config_service_win_unittest.cc
diff options
context:
space:
mode:
authorsdoyon@chromium.org <sdoyon@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-04-20 16:50:56 +0000
committersdoyon@chromium.org <sdoyon@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-04-20 16:50:56 +0000
commit861c6c6d56add9157e04aa2ac43bec8e33bf1cc1 (patch)
tree835dd6466912cd073e3ac16177252fe0c8865c93 /net/proxy/proxy_config_service_win_unittest.cc
parentbc1529bbafc42d96d9ac0839bcb0ba1ab9328445 (diff)
downloadchromium_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.cc43
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);
}
}