From 7258924d59592745795ec1c12eafb20a30625826 Mon Sep 17 00:00:00 2001 From: "shess@chromium.org" Date: Wed, 11 Feb 2009 01:29:39 +0000 Subject: Unittest gardening on MacOS X: - base/gfx/rect_unittest.cc had a typo - base/waitable_event_watcher_unittest.cc enabled - chrome/common/pref_member_unittest.cc enabled - chrome/common/json_value_serializer_unittest.cc had a Windows-only bit which currently works fine on Mac. - These worked once an anonymous namespace was cleared up: chrome/browser/history/query_parser_unittest.cc chrome/browser/history/url_database_unittest.cc chrome/common/json_value_serializer_unittest.cc Some required minor editting to remove anonymous namespaces for test classes. Review URL: http://codereview.chromium.org/20241 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@9539 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/history/query_parser_unittest.cc | 4 ---- chrome/browser/history/url_database_unittest.cc | 4 ++-- 2 files changed, 2 insertions(+), 6 deletions(-) (limited to 'chrome/browser/history') diff --git a/chrome/browser/history/query_parser_unittest.cc b/chrome/browser/history/query_parser_unittest.cc index 25d4f4e..6d425e91 100644 --- a/chrome/browser/history/query_parser_unittest.cc +++ b/chrome/browser/history/query_parser_unittest.cc @@ -7,8 +7,6 @@ #include "chrome/common/scoped_vector.h" #include "testing/gtest/include/gtest/gtest.h" -namespace { - class QueryParserTest : public testing::Test { public: struct TestData { @@ -22,8 +20,6 @@ class QueryParserTest : public testing::Test { QueryParser query_parser_; }; -}; // namespace - // Test helper: Convert a user query string to a SQLite query string. std::wstring QueryParserTest::QueryToString(const std::wstring& query) { std::wstring sqlite_query; diff --git a/chrome/browser/history/url_database_unittest.cc b/chrome/browser/history/url_database_unittest.cc index 9520fa7..a33829a 100644 --- a/chrome/browser/history/url_database_unittest.cc +++ b/chrome/browser/history/url_database_unittest.cc @@ -28,6 +28,8 @@ bool IsURLRowEqual(const URLRow& a, a.last_visit() - b.last_visit() <= TimeDelta::FromSeconds(1) && a.hidden() == b.hidden(); } + +} // namespace class URLDatabaseTest : public testing::Test, public URLDatabase { @@ -70,8 +72,6 @@ class URLDatabaseTest : public testing::Test, SqliteStatementCache* statement_cache_; }; -} // namespace - // Test add and query for the URL table in the HistoryDatabase TEST_F(URLDatabaseTest, AddURL) { // first, add two URLs -- cgit v1.1