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

Unified Diff: chrome/browser/page_cycler/page_cycler_browsertest.cc

Issue 10491010: Reland r140188: CommandLine Page Cycler (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 7 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/browser/page_cycler/page_cycler.cc ('k') | chrome/browser/page_cycler/page_cycler_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/page_cycler/page_cycler_browsertest.cc
diff --git a/chrome/browser/page_cycler/page_cycler_browsertest.cc b/chrome/browser/page_cycler/page_cycler_browsertest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..960f8b0a851b324152f6e11592f9fbb4a8446aa7
--- /dev/null
+++ b/chrome/browser/page_cycler/page_cycler_browsertest.cc
@@ -0,0 +1,381 @@
+// 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/command_line.h"
+#include "base/file_path.h"
+#include "base/file_util.h"
+#include "base/path_service.h"
+#include "base/string_split.h"
+#include "base/string_util.h"
+#include "base/threading/sequenced_worker_pool.h"
+#include "chrome/browser/browser_process.h"
+#include "chrome/browser/page_cycler/page_cycler.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/profiles/profile_manager.h"
+#include "chrome/browser/ui/browser_list.h"
+#include "chrome/common/chrome_notification_types.h"
+#include "chrome/common/chrome_paths.h"
+#include "chrome/common/chrome_switches.h"
+#include "chrome/test/base/in_process_browser_test.h"
+#include "chrome/test/base/testing_profile.h"
+#include "chrome/test/base/ui_test_utils.h"
+#include "content/public/browser/notification_observer.h"
+#include "content/public/browser/notification_registrar.h"
+#include "content/public/browser/notification_service.h"
+#include "content/public/common/content_switches.h"
+#include "content/public/common/url_constants.h"
+#include "googleurl/src/gurl.h"
+
+// Basic PageCyclerBrowserTest structure; used in testing most of PageCycler's
+// functionality.
+class PageCyclerBrowserTest : public content::NotificationObserver,
+ public InProcessBrowserTest {
+ public:
+ PageCyclerBrowserTest() : page_cycler_(NULL) {
+ }
+
+ virtual ~PageCyclerBrowserTest() {
+ }
+
+ // Initialize file paths within a temporary directory; this should be
+ // empty and nonexistent.
+ virtual void InitFilePaths(FilePath temp_path) {
+ temp_path_ = temp_path;
+ urls_file_ = temp_path.AppendASCII("urls_file");
+ errors_file_ = temp_path.AppendASCII("errors");
+ stats_file_ = temp_path.AppendASCII("stats");
+
+ ASSERT_FALSE(file_util::PathExists(urls_file_));
+ ASSERT_FALSE(file_util::PathExists(errors_file_));
+ ASSERT_FALSE(file_util::PathExists(stats_file_));
+ }
+
+ // Initialize a PageCycler using either the base fields, or using provided
+ // ones.
+ void InitPageCycler() {
+ page_cycler_ = new PageCycler(browser(), urls_file());
+ page_cycler_->set_errors_file(errors_file());
+ page_cycler_->set_stats_file(stats_file());
+ }
+
+ void InitPageCycler(FilePath urls_file,
+ FilePath errors_file,
+ FilePath stats_file) {
+ page_cycler_ = new PageCycler(browser(), urls_file);
+ page_cycler_->set_errors_file(errors_file);
+ page_cycler_->set_stats_file(stats_file);
+ }
+
+ void RegisterForNotifications() {
+ registrar_.Add(this, chrome::NOTIFICATION_BROWSER_CLOSED,
+ content::NotificationService::AllSources());
+ }
+
+ // Get a collection of basic urls which are stored in the test directory.
+ // NOTE: |test_server| must be started first!
+ std::vector<GURL> GetURLs() {
+ std::vector<GURL> urls;
+ urls.push_back(test_server()->GetURL("files/page_cycler/basic_html.html"));
+ urls.push_back(test_server()->GetURL("files/page_cycler/basic_js.html"));
+ urls.push_back(test_server()->GetURL("files/page_cycler/basic_css.html"));
+ return urls;
+ }
+
+ // Read the errors file, and generate a vector of error strings.
+ std::vector<std::string> GetErrorsFromFile() {
+ std::string error_file_contents;
+ CHECK(file_util::ReadFileToString(errors_file_,
+ &error_file_contents));
+ if (error_file_contents[error_file_contents.size() - 1] == '\n')
+ error_file_contents.resize(error_file_contents.size() - 1);
+
+ std::vector<std::string> errors;
+ base::SplitString(error_file_contents, '\n', &errors);
+
+ return errors;
+ }
+
+ // Convert a vector of GURLs into a newline-separated string, ready to be
+ // written to the urls file for PageCycler to use.
+ std::string GetStringFromURLs(std::vector<GURL> urls) {
+ std::string urls_string;
+ for (std::vector<GURL>::const_iterator iter = urls.begin();
+ iter != urls.end(); ++iter)
+ urls_string.append(iter->spec() + "\n");
+ return urls_string;
+ }
+
+ // content::NotificationObserver.
+ virtual void Observe(int type,
+ const content::NotificationSource& source,
+ const content::NotificationDetails& details) {
+ switch (type) {
+ case chrome::NOTIFICATION_BROWSER_CLOSED:
+ MessageLoop::current()->PostTask(FROM_HERE, MessageLoop::QuitClosure());
+ break;
+ default:
+ NOTREACHED();
+ break;
+ }
+ }
+
+ FilePath urls_file() { return urls_file_; }
+ FilePath errors_file() { return errors_file_; }
+ FilePath stats_file() { return stats_file_; }
+ PageCycler* page_cycler() { return page_cycler_; }
+
+ protected:
+ FilePath temp_path_;
+ FilePath urls_file_;
+ FilePath errors_file_;
+ FilePath stats_file_;
+ PageCycler* page_cycler_;
+ content::NotificationRegistrar registrar_;
+};
+
+// Structure used for testing PageCycler's ability to playback a series of
+// URLs given a cache directory.
+class PageCyclerCachedBrowserTest : public PageCyclerBrowserTest {
+ public:
+ // For a cached test, we use the provided user data directory from the test
+ // directory.
+ virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
+ InProcessBrowserTest::SetUpCommandLine(command_line);
+
+ FilePath test_dir;
+ ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &test_dir));
+ test_dir = test_dir.AppendASCII("page_cycler");
+
+ FilePath source_data_dir = test_dir.AppendASCII("cached_data_dir");
+ CHECK(file_util::PathExists(source_data_dir));
+
+ CHECK(user_data_dir_.CreateUniqueTempDir());
+
+ FilePath dest_data_dir =
+ user_data_dir_.path().AppendASCII("cached_data_dir");
+ CHECK(!file_util::PathExists(dest_data_dir));
+
+ CHECK(file_util::CopyDirectory(source_data_dir,
+ user_data_dir_.path(),
+ true)); // recursive.
+ CHECK(file_util::PathExists(dest_data_dir));
+
+ command_line->AppendSwitchPath(switches::kUserDataDir,
+ dest_data_dir);
+ command_line->AppendSwitch(switches::kPlaybackMode);
+ }
+
+ // Initialize the file paths to use the UserDataDir's urls file, instead
+ // of one to be written.
+ virtual void InitFilePaths(FilePath temp_path) OVERRIDE {
+ urls_file_ = user_data_dir_.path().AppendASCII("cached_data_dir")
+ .AppendASCII("urls");
+ errors_file_ = temp_path.AppendASCII("errors");
+ stats_file_ = temp_path.AppendASCII("stats");
+
+ ASSERT_TRUE(file_util::PathExists(urls_file_));
+ ASSERT_FALSE(file_util::PathExists(errors_file_));
+ ASSERT_FALSE(file_util::PathExists(stats_file_));
+ }
+
+ private:
+ // The directory storing the copy of the UserDataDir.
+ ScopedTempDir user_data_dir_;
+};
+
+// Sanity check; iterate through a series of URLs and make sure there are no
+// errors.
+IN_PROC_BROWSER_TEST_F(PageCyclerBrowserTest, BasicTest) {
+ const size_t kNumIterations = 3;
+ ScopedTempDir temp;
+ ASSERT_TRUE(temp.CreateUniqueTempDir());
+
+ RegisterForNotifications();
+ InitFilePaths(temp.path());
+
+ ASSERT_TRUE(test_server()->Start());
+
+ std::string urls_string = GetStringFromURLs(GetURLs());;
+
+ ASSERT_TRUE(file_util::WriteFile(urls_file(), urls_string.c_str(),
+ urls_string.size()));
+
+ InitPageCycler();
+ page_cycler()->Run(kNumIterations);
+
+ ui_test_utils::RunMessageLoop();
+ ASSERT_FALSE(file_util::PathExists(errors_file()));
+ ASSERT_TRUE(file_util::PathExists(stats_file()));
+}
+
+// Test to make sure that PageCycler will recognize unvisitable URLs, and will
+// handle them appropriately.
+IN_PROC_BROWSER_TEST_F(PageCyclerBrowserTest, UnvisitableURL) {
+ const size_t kNumIterations = 3;
+ const char kFakeURL[] = "http://www.pleasenoonehavethisurlanytimeinthenext"
+ "century.com/gibberish";
+ ScopedTempDir temp;
+ ASSERT_TRUE(temp.CreateUniqueTempDir());
+
+ RegisterForNotifications();
+ InitFilePaths(temp.path());
+
+ ASSERT_TRUE(test_server()->Start());
+
+ std::vector<GURL> urls = GetURLs();
+ urls.push_back(GURL(kFakeURL));
+ std::string urls_string = GetStringFromURLs(urls);
+
+ ASSERT_TRUE(file_util::WriteFile(urls_file(), urls_string.c_str(),
+ urls_string.size()));
+
+ InitPageCycler();
+ page_cycler()->Run(kNumIterations);
+
+ ui_test_utils::RunMessageLoop();
+ ASSERT_TRUE(file_util::PathExists(errors_file()));
+ ASSERT_TRUE(file_util::PathExists(stats_file()));
+
+ std::vector<std::string> errors = GetErrorsFromFile();
+
+ size_t num_errors = errors.size();
+ ASSERT_EQ(kNumIterations, num_errors);
+
+ // Check that each error message contains the fake URL (i.e., that it wasn't
+ // from a valid URL, and that the fake URL was caught each time).
+ for (std::vector<std::string>::const_iterator iter = errors.begin();
+ iter != errors.end(); ++iter) {
+ ASSERT_NE(iter->find(kFakeURL), std::string::npos);
+ }
+}
+
+// Test that PageCycler will remove an invalid URL prior to running.
+IN_PROC_BROWSER_TEST_F(PageCyclerBrowserTest, InvalidURL) {
+ const size_t kNumIterations = 1;
+ const char kBadURL[] = "notarealurl";
+
+ ScopedTempDir temp;
+ ASSERT_TRUE(temp.CreateUniqueTempDir());
+
+ RegisterForNotifications();
+ InitFilePaths(temp.path());
+
+ ASSERT_TRUE(test_server()->Start());
+
+ std::string urls_string = GetStringFromURLs(GetURLs());
+ urls_string.append(kBadURL).append("\n");
+
+ ASSERT_TRUE(file_util::WriteFile(urls_file(), urls_string.c_str(),
+ urls_string.size()));
+
+ InitPageCycler();
+ page_cycler()->Run(kNumIterations);
+
+ ui_test_utils::RunMessageLoop();
+ ASSERT_TRUE(file_util::PathExists(errors_file()));
+ ASSERT_TRUE(file_util::PathExists(stats_file()));
+
+ std::vector<std::string> errors = GetErrorsFromFile();
+ ASSERT_EQ(1u, errors.size());
+
+ std::string expected_error = "Omitting invalid URL: ";
+ expected_error.append(kBadURL).append(".");
+
+ ASSERT_FALSE(errors[0].compare(expected_error));
+}
+
+// Test that PageCycler will remove a Chrome Error URL prior to running.
+IN_PROC_BROWSER_TEST_F(PageCyclerBrowserTest, ChromeErrorURL) {
+ const size_t kNumIterations = 1;
+ ScopedTempDir temp;
+ ASSERT_TRUE(temp.CreateUniqueTempDir());
+
+ RegisterForNotifications();
+ InitFilePaths(temp.path());
+
+ ASSERT_TRUE(test_server()->Start());
+
+ std::vector<GURL> urls = GetURLs();
+ urls.push_back(GURL(content::kUnreachableWebDataURL));
+ std::string urls_string = GetStringFromURLs(urls);
+
+ ASSERT_TRUE(file_util::WriteFile(urls_file(), urls_string.c_str(),
+ urls_string.size()));
+
+ InitPageCycler();
+ page_cycler()->Run(kNumIterations);
+
+ ui_test_utils::RunMessageLoop();
+ ASSERT_TRUE(file_util::PathExists(errors_file()));
+ ASSERT_TRUE(file_util::PathExists(stats_file()));
+
+ std::vector<std::string> errors = GetErrorsFromFile();
+ ASSERT_EQ(1u, errors.size());
+
+ std::string expected_error = "Chrome error pages are not allowed as urls. "
+ "Omitting url: ";
+ expected_error.append(content::kUnreachableWebDataURL).append(".");
+
+ ASSERT_FALSE(errors[0].compare(expected_error));
+}
+
+// Test that PageCycler will visit all the urls from a cache directory
+// successfully while in playback mode.
+IN_PROC_BROWSER_TEST_F(PageCyclerCachedBrowserTest, PlaybackMode) {
+ const size_t kNumIterations = 1;
+ ScopedTempDir temp;
+ ASSERT_TRUE(temp.CreateUniqueTempDir());
+
+ RegisterForNotifications();
+ InitFilePaths(temp.path());
+
+ InitPageCycler();
+
+ page_cycler()->Run(kNumIterations);
+
+ ui_test_utils::RunMessageLoop();
+ ASSERT_TRUE(file_util::PathExists(stats_file()));
+ ASSERT_FALSE(file_util::PathExists(errors_file()));
+}
+
+// Test that PageCycler will have a cache miss if a URL is missing from the
+// cache directory while in playback mode.
+IN_PROC_BROWSER_TEST_F(PageCyclerCachedBrowserTest, URLNotInCache) {
+ const size_t kNumIterations = 1;
+ const char kCacheMissURL[] = "http://www.images.google.com/";
+
+ ScopedTempDir temp;
+ ASSERT_TRUE(temp.CreateUniqueTempDir());
+
+ RegisterForNotifications();
+ InitFilePaths(temp.path());
+
+ std::string urls_string;
+ ASSERT_TRUE(file_util::ReadFileToString(urls_file(),
+ &urls_string));
+
+ urls_string.append("\n").append(kCacheMissURL);
+ FilePath new_urls_file = temp.path().AppendASCII("urls");
+ ASSERT_FALSE(file_util::PathExists(new_urls_file));
+
+ ASSERT_TRUE(file_util::WriteFile(new_urls_file, urls_string.c_str(),
+ urls_string.size()));
+
+ InitPageCycler(new_urls_file, errors_file(), stats_file());
+ page_cycler()->Run(kNumIterations);
+
+ ui_test_utils::RunMessageLoop();
+ ASSERT_TRUE(file_util::PathExists(errors_file()));
+ ASSERT_TRUE(file_util::PathExists(stats_file()));
+
+ std::vector<std::string> errors = GetErrorsFromFile();
+ ASSERT_EQ(1u, errors.size());
+
+ std::string expected_error;
+ expected_error.append("Failed to load the page at: ")
+ .append(kCacheMissURL)
+ .append(": The requested entry was not found in the cache.");
+
+ ASSERT_FALSE(errors[0].compare(expected_error));
+}
« no previous file with comments | « chrome/browser/page_cycler/page_cycler.cc ('k') | chrome/browser/page_cycler/page_cycler_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698