Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(584)

Unified Diff: components/data_reduction_proxy/core/browser/data_reduction_proxy_configurator_unittest.cc

Issue 880963006: Create a new DataReductionProxyConfig class (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/data_reduction_proxy/core/browser/data_reduction_proxy_configurator_unittest.cc
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_configurator_unittest.cc b/components/data_reduction_proxy/core/browser/data_reduction_proxy_configurator_unittest.cc
index 9b431a955956ede1cded07683c476d49029eb3f8..eaa4b63a065c5978ca1977f2586eb295ccdc360d 100644
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_configurator_unittest.cc
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_configurator_unittest.cc
@@ -17,7 +17,7 @@
namespace data_reduction_proxy {
-class DataReductionProxyConfigTest : public testing::Test {
+class DataReductionProxyConfiguratorTest : public testing::Test {
public:
void SetUp() override {
task_runner_ = new base::TestSimpleTaskRunner();
@@ -54,7 +54,7 @@ class DataReductionProxyConfigTest : public testing::Test {
data_reduction_proxy_event_store_;
};
-TEST_F(DataReductionProxyConfigTest, TestUnrestricted) {
+TEST_F(DataReductionProxyConfiguratorTest, TestUnrestricted) {
config_->Enable(false,
false,
"https://www.foo.com:443/",
@@ -66,7 +66,7 @@ TEST_F(DataReductionProxyConfigTest, TestUnrestricted) {
"", "");
}
-TEST_F(DataReductionProxyConfigTest, TestUnrestrictedSSL) {
+TEST_F(DataReductionProxyConfiguratorTest, TestUnrestrictedSSL) {
config_->Enable(false,
false,
"https://www.foo.com:443/",
@@ -79,7 +79,7 @@ TEST_F(DataReductionProxyConfigTest, TestUnrestrictedSSL) {
"");
}
-TEST_F(DataReductionProxyConfigTest, TestUnrestrictedWithBypassRule) {
+TEST_F(DataReductionProxyConfiguratorTest, TestUnrestrictedWithBypassRule) {
config_->AddHostPatternToBypass("<local>");
config_->AddHostPatternToBypass("*.goo.com");
config_->Enable(false,
@@ -93,13 +93,13 @@ TEST_F(DataReductionProxyConfigTest, TestUnrestrictedWithBypassRule) {
"<local>;*.goo.com;");
}
-TEST_F(DataReductionProxyConfigTest, TestUnrestrictedWithoutFallback) {
+TEST_F(DataReductionProxyConfiguratorTest, TestUnrestrictedWithoutFallback) {
config_->Enable(false, false, "https://www.foo.com:443/", "", "");
CheckProxyConfig(net::ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME,
"HTTPS www.foo.com:443;DIRECT", "", "");
}
-TEST_F(DataReductionProxyConfigTest, TestRestricted) {
+TEST_F(DataReductionProxyConfiguratorTest, TestRestricted) {
config_->Enable(true,
false,
"https://www.foo.com:443/",
@@ -109,7 +109,7 @@ TEST_F(DataReductionProxyConfigTest, TestRestricted) {
"PROXY www.bar.com:80;DIRECT", "", "");
}
-TEST_F(DataReductionProxyConfigTest, TestFallbackRestricted) {
+TEST_F(DataReductionProxyConfiguratorTest, TestFallbackRestricted) {
config_->Enable(false,
true,
"https://www.foo.com:443/",
@@ -119,7 +119,7 @@ TEST_F(DataReductionProxyConfigTest, TestFallbackRestricted) {
"HTTPS www.foo.com:443;DIRECT", "", "");
}
-TEST_F(DataReductionProxyConfigTest, TestDisable) {
+TEST_F(DataReductionProxyConfiguratorTest, TestDisable) {
data_reduction_proxy::DataReductionProxyParams params(
data_reduction_proxy::DataReductionProxyParams::
kAllowAllProxyConfigurations);
@@ -132,7 +132,7 @@ TEST_F(DataReductionProxyConfigTest, TestDisable) {
CheckProxyConfig(net::ProxyConfig::ProxyRules::TYPE_NO_RULES, "", "", "");
}
-TEST_F(DataReductionProxyConfigTest, TestBypassList) {
+TEST_F(DataReductionProxyConfiguratorTest, TestBypassList) {
config_->AddHostPatternToBypass("http://www.google.com");
config_->AddHostPatternToBypass("fefe:13::abc/33");
config_->AddURLPatternToBypass("foo.org/images/*");

Powered by Google App Engine
This is Rietveld 408576698