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

Unified Diff: chrome/common/content_settings_pattern_parser_unittest.cc

Issue 541813002: Componentize chrome/common/content_settings files. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: GN fix Created 6 years, 3 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
« no previous file with comments | « chrome/common/content_settings.cc ('k') | chrome/common/content_settings_pattern_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/common/content_settings_pattern_parser_unittest.cc
diff --git a/chrome/common/content_settings_pattern_parser_unittest.cc b/chrome/common/content_settings_pattern_parser_unittest.cc
deleted file mode 100644
index b35b70007521cb07adbeae696bb1253d03e202c3..0000000000000000000000000000000000000000
--- a/chrome/common/content_settings_pattern_parser_unittest.cc
+++ /dev/null
@@ -1,230 +0,0 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "components/content_settings/core/common/content_settings_pattern.h"
-#include "components/content_settings/core/common/content_settings_pattern_parser.h"
-
-#include "testing/gmock/include/gmock/gmock.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-namespace {
-typedef ContentSettingsPattern::BuilderInterface BuilderInterface;
-} // namespace
-
-class MockBuilder : public ContentSettingsPattern::BuilderInterface {
- public:
- MOCK_METHOD0(WithSchemeWildcard, BuilderInterface*());
- MOCK_METHOD0(WithDomainWildcard, BuilderInterface*());
- MOCK_METHOD0(WithPortWildcard, BuilderInterface*());
- MOCK_METHOD1(WithScheme, BuilderInterface*(const std::string& scheme));
- MOCK_METHOD1(WithHost, BuilderInterface*(const std::string& host));
- MOCK_METHOD1(WithPort, BuilderInterface*(const std::string& port));
- MOCK_METHOD1(WithPath, BuilderInterface*(const std::string& path));
- MOCK_METHOD0(WithPathWildcard, BuilderInterface*());
- MOCK_METHOD0(Invalid, BuilderInterface*());
- MOCK_METHOD0(Build, ContentSettingsPattern());
-};
-
-TEST(ContentSettingsPatternParserTest, ParsePatterns) {
- // Test valid patterns
- ::testing::StrictMock<MockBuilder> builder;
-
- // WithPathWildcard() is not called for "*". (Need a strict Mock for this
- // case.)
- EXPECT_CALL(builder, WithSchemeWildcard()).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithDomainWildcard()).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithPortWildcard()).Times(1).WillOnce(
- ::testing::Return(&builder));
- content_settings::PatternParser::Parse("*", &builder);
- ::testing::Mock::VerifyAndClear(&builder);
-
- EXPECT_CALL(builder, WithScheme("http")).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithHost("www.youtube.com")).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithPort("8080")).Times(1).WillOnce(
- ::testing::Return(&builder));
- content_settings::PatternParser::Parse(
- "http://www.youtube.com:8080", &builder);
- ::testing::Mock::VerifyAndClear(&builder);
-
- EXPECT_CALL(builder, WithSchemeWildcard()).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithHost("www.gmail.com")).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithPort("80")).Times(1).WillOnce(
- ::testing::Return(&builder));
- content_settings::PatternParser::Parse("*://www.gmail.com:80", &builder);
- ::testing::Mock::VerifyAndClear(&builder);
-
- EXPECT_CALL(builder, WithScheme("http")).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithHost("www.gmail.com")).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithPortWildcard()).Times(1).WillOnce(
- ::testing::Return(&builder));
- content_settings::PatternParser::Parse("http://www.gmail.com:*", &builder);
- ::testing::Mock::VerifyAndClear(&builder);
-
- EXPECT_CALL(builder, WithScheme("http")).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithDomainWildcard()).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithHost("google.com")).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithPort("80")).Times(1).WillOnce(
- ::testing::Return(&builder));
- content_settings::PatternParser::Parse("http://[*.]google.com:80", &builder);
- ::testing::Mock::VerifyAndClear(&builder);
-
- EXPECT_CALL(builder, WithScheme("https")).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithHost("[::1]")).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithPort("8080")).Times(1).WillOnce(
- ::testing::Return(&builder));
- content_settings::PatternParser::Parse("https://[::1]:8080", &builder);
- ::testing::Mock::VerifyAndClear(&builder);
-
- EXPECT_CALL(builder, WithScheme("http")).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithHost("127.0.0.1")).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithPort("8080")).Times(1).WillOnce(
- ::testing::Return(&builder));
- content_settings::PatternParser::Parse("http://127.0.0.1:8080", &builder);
- ::testing::Mock::VerifyAndClear(&builder);
-
- // Test valid pattern short forms
- EXPECT_CALL(builder, WithSchemeWildcard()).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithHost("www.youtube.com")).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithPort("8080")).Times(1).WillOnce(
- ::testing::Return(&builder));
- content_settings::PatternParser::Parse("www.youtube.com:8080", &builder);
- ::testing::Mock::VerifyAndClear(&builder);
-
- EXPECT_CALL(builder, WithSchemeWildcard()).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithHost("www.youtube.com")).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithPortWildcard()).Times(1).WillOnce(
- ::testing::Return(&builder));
- content_settings::PatternParser::Parse("www.youtube.com", &builder);
- ::testing::Mock::VerifyAndClear(&builder);
-
- EXPECT_CALL(builder, WithSchemeWildcard()).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithDomainWildcard()).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithHost("youtube.com")).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithPortWildcard()).Times(1).WillOnce(
- ::testing::Return(&builder));
- content_settings::PatternParser::Parse("[*.]youtube.com", &builder);
- ::testing::Mock::VerifyAndClear(&builder);
-
- // Test invalid patterns
- EXPECT_CALL(builder, WithSchemeWildcard()).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, Invalid()).Times(1).WillOnce(
- ::testing::Return(&builder));
- content_settings::PatternParser::Parse("*youtube.com", &builder);
- ::testing::Mock::VerifyAndClear(&builder);
-
- EXPECT_CALL(builder, WithSchemeWildcard()).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, Invalid()).Times(1).WillOnce(
- ::testing::Return(&builder));
- content_settings::PatternParser::Parse("*.youtube.com", &builder);
- ::testing::Mock::VerifyAndClear(&builder);
-
- EXPECT_CALL(builder, WithSchemeWildcard()).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, Invalid()).Times(1).WillOnce(
- ::testing::Return(&builder));
- content_settings::PatternParser::Parse("www.youtube.com*", &builder);
- ::testing::Mock::VerifyAndClear(&builder);
-}
-
-TEST(ContentSettingsPatternParserTest, ParseFilePatterns) {
- ::testing::StrictMock<MockBuilder> builder;
-
- EXPECT_CALL(builder, WithScheme("file")).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithPath("/foo/bar/test.html")).Times(1).WillOnce(
- ::testing::Return(&builder));
- content_settings::PatternParser::Parse(
- "file:///foo/bar/test.html", &builder);
- ::testing::Mock::VerifyAndClear(&builder);
-
- EXPECT_CALL(builder, WithScheme("file")).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithDomainWildcard()).Times(1).WillOnce(
- ::testing::Return(&builder));
- content_settings::PatternParser::Parse(
- "file://*", &builder);
- ::testing::Mock::VerifyAndClear(&builder);
-
- EXPECT_CALL(builder, WithScheme("file")).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithDomainWildcard()).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithPath("/")).Times(1).WillOnce(
- ::testing::Return(&builder));
- content_settings::PatternParser::Parse(
- "file://*/", &builder);
- ::testing::Mock::VerifyAndClear(&builder);
-
- EXPECT_CALL(builder, WithScheme("file")).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithDomainWildcard()).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithPathWildcard()).Times(1).WillOnce(
- ::testing::Return(&builder));
- content_settings::PatternParser::Parse(
- "file://*/*", &builder);
- ::testing::Mock::VerifyAndClear(&builder);
-
- EXPECT_CALL(builder, WithScheme("file")).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, WithPathWildcard()).Times(1).WillOnce(
- ::testing::Return(&builder));
- content_settings::PatternParser::Parse(
- "file:///*", &builder);
- ::testing::Mock::VerifyAndClear(&builder);
-
- // Invalid file patterns.
- EXPECT_CALL(builder, WithScheme("file")).Times(1).WillOnce(
- ::testing::Return(&builder));
- EXPECT_CALL(builder, Invalid()).Times(1).WillOnce(
- ::testing::Return(&builder));
- content_settings::PatternParser::Parse(
- "file://**", &builder);
- ::testing::Mock::VerifyAndClear(&builder);
-}
-
-TEST(ContentSettingsPatternParserTest, SerializePatterns) {
- ContentSettingsPattern::PatternParts parts;
- parts.scheme = "http";
- parts.host = "www.youtube.com";
- parts.port = "8080";
- EXPECT_STREQ("http://www.youtube.com:8080",
- content_settings::PatternParser::ToString(parts).c_str());
-
- parts = ContentSettingsPattern::PatternParts();
- parts.scheme = "file";
- parts.path = "/foo/bar/test.html";
- EXPECT_STREQ("file:///foo/bar/test.html",
- content_settings::PatternParser::ToString(parts).c_str());
-
- parts = ContentSettingsPattern::PatternParts();
- parts.scheme = "file";
- parts.path = "";
- parts.is_path_wildcard = true;
- EXPECT_EQ("file:///*", content_settings::PatternParser::ToString(parts));
-}
« no previous file with comments | « chrome/common/content_settings.cc ('k') | chrome/common/content_settings_pattern_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698