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

Unified Diff: chrome/browser/search_engines/search_host_to_urls_map_unittest.cc

Issue 382663004: Componentize tests under search_engines (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Include search_engines even if os=="ios" Created 6 years, 5 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: chrome/browser/search_engines/search_host_to_urls_map_unittest.cc
diff --git a/chrome/browser/search_engines/search_host_to_urls_map_unittest.cc b/chrome/browser/search_engines/search_host_to_urls_map_unittest.cc
deleted file mode 100644
index 967166a443087d26e40f6698134472362b1d12b2..0000000000000000000000000000000000000000
--- a/chrome/browser/search_engines/search_host_to_urls_map_unittest.cc
+++ /dev/null
@@ -1,107 +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 "base/basictypes.h"
-#include "base/memory/scoped_ptr.h"
-#include "components/search_engines/search_host_to_urls_map.h"
-#include "components/search_engines/search_terms_data.h"
-#include "components/search_engines/template_url.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-typedef SearchHostToURLsMap::TemplateURLSet TemplateURLSet;
-
-// Basic functionality for the SearchHostToURLsMap tests.
-class SearchHostToURLsMapTest : public testing::Test {
- public:
- SearchHostToURLsMapTest() {}
-
- virtual void SetUp();
-
- protected:
- scoped_ptr<SearchHostToURLsMap> provider_map_;
- scoped_ptr<TemplateURL> t_urls_[2];
- std::string host_;
-
- DISALLOW_COPY_AND_ASSIGN(SearchHostToURLsMapTest);
-};
-
-void SearchHostToURLsMapTest::SetUp() {
- // Add some entries to the search host map.
- host_ = "www.unittest.com";
- TemplateURLData data;
- data.SetURL("http://" + host_ + "/path1");
- t_urls_[0].reset(new TemplateURL(data));
- data.SetURL("http://" + host_ + "/path2");
- t_urls_[1].reset(new TemplateURL(data));
- std::vector<TemplateURL*> template_urls;
- template_urls.push_back(t_urls_[0].get());
- template_urls.push_back(t_urls_[1].get());
-
- provider_map_.reset(new SearchHostToURLsMap);
- provider_map_->Init(template_urls, SearchTermsData());
-}
-
-TEST_F(SearchHostToURLsMapTest, Add) {
- std::string new_host = "example.com";
- TemplateURLData data;
- data.SetURL("http://" + new_host + "/");
- TemplateURL new_t_url(data);
- provider_map_->Add(&new_t_url, SearchTermsData());
-
- ASSERT_EQ(&new_t_url, provider_map_->GetTemplateURLForHost(new_host));
-}
-
-TEST_F(SearchHostToURLsMapTest, Remove) {
- provider_map_->Remove(t_urls_[0].get());
-
- const TemplateURL* found_url = provider_map_->GetTemplateURLForHost(host_);
- ASSERT_EQ(t_urls_[1].get(), found_url);
-
- const TemplateURLSet* urls = provider_map_->GetURLsForHost(host_);
- ASSERT_TRUE(urls != NULL);
-
- int url_count = 0;
- for (TemplateURLSet::const_iterator i(urls->begin()); i != urls->end(); ++i) {
- url_count++;
- ASSERT_EQ(t_urls_[1].get(), *i);
- }
- ASSERT_EQ(1, url_count);
-}
-
-TEST_F(SearchHostToURLsMapTest, GetTemplateURLForKnownHost) {
- const TemplateURL* found_url = provider_map_->GetTemplateURLForHost(host_);
- ASSERT_TRUE(found_url == t_urls_[0].get() || found_url == t_urls_[1].get());
-}
-
-TEST_F(SearchHostToURLsMapTest, GetTemplateURLForUnknownHost) {
- const TemplateURL* found_url =
- provider_map_->GetTemplateURLForHost("a" + host_);
- ASSERT_TRUE(found_url == NULL);
-}
-
-TEST_F(SearchHostToURLsMapTest, GetURLsForKnownHost) {
- const TemplateURLSet* urls = provider_map_->GetURLsForHost(host_);
- ASSERT_TRUE(urls != NULL);
-
- bool found_urls[arraysize(t_urls_)] = { false };
-
- for (TemplateURLSet::const_iterator i(urls->begin()); i != urls->end(); ++i) {
- const TemplateURL* url = *i;
- for (size_t i = 0; i < arraysize(found_urls); ++i) {
- if (url == t_urls_[i].get()) {
- found_urls[i] = true;
- break;
- }
- }
- }
-
- for (size_t i = 0; i < arraysize(found_urls); ++i)
- ASSERT_TRUE(found_urls[i]);
-}
-
-TEST_F(SearchHostToURLsMapTest, GetURLsForUnknownHost) {
- const SearchHostToURLsMap::TemplateURLSet* urls =
- provider_map_->GetURLsForHost("a" + host_);
- ASSERT_TRUE(urls == NULL);
-}

Powered by Google App Engine
This is Rietveld 408576698