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

Unified Diff: chrome/browser/history/android/android_urls_database_unittest.cc

Issue 10067030: Upgrade the history database to version 23 (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Sync and update test data Created 8 years, 8 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/history/android/android_urls_database.cc ('k') | chrome/browser/history/history_database.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/history/android/android_urls_database_unittest.cc
diff --git a/chrome/browser/history/android/android_urls_database_unittest.cc b/chrome/browser/history/android/android_urls_database_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..8650c5520a438f492eb476af5ba5a0ca35c1bbf6
--- /dev/null
+++ b/chrome/browser/history/android/android_urls_database_unittest.cc
@@ -0,0 +1,67 @@
+// 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 "chrome/browser/history/android/android_urls_database.h"
+
+#include "base/file_path.h"
+#include "base/file_util.h"
+#include "base/path_service.h"
+#include "base/scoped_temp_dir.h"
+#include "chrome/browser/history/history_database.h"
+#include "chrome/browser/history/history_unittest_base.h"
+#include "chrome/common/chrome_constants.h"
+#include "chrome/common/chrome_paths.h"
+#include "chrome/test/base/testing_profile.h"
+
+namespace history {
+
+class AndroidURLsMigrationTest : public HistoryUnitTestBase {
+ public:
+ AndroidURLsMigrationTest() {
+ }
+ ~AndroidURLsMigrationTest() {
+ }
+
+ protected:
+ virtual void SetUp() {
+ profile_.reset(new TestingProfile);
+
+ FilePath data_path;
+ ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &data_path));
+ data_path = data_path.AppendASCII("History");
+
+ history_db_name_ = profile_->GetPath().Append(chrome::kHistoryFilename);
+ // Set up history as they would be before migration.
+ ASSERT_NO_FATAL_FAILURE(
+ ExecuteSQLScript(data_path.AppendASCII("history.21.sql"),
+ history_db_name_));
+ }
+
+ protected:
+ FilePath history_db_name_;
+ scoped_ptr<TestingProfile> profile_;
+};
+
+TEST_F(AndroidURLsMigrationTest, MigrateToVersion22) {
+ HistoryDatabase db;
+ ASSERT_EQ(sql::INIT_OK, db.Init(history_db_name_, profile_->GetPath()));
+ // Migration has done.
+ // The column of previous table shouldn't exist.
+ EXPECT_FALSE(db.GetDB().DoesColumnExist("android_urls", "bookmark"));
+ sql::Statement statement(db.GetDB().GetUniqueStatement(
+ "SELECT id, url_id, raw_url FROM android_urls ORDER BY id ASC"));
+ ASSERT_TRUE(statement.Step());
+ EXPECT_EQ(1, statement.ColumnInt64(0));
+ EXPECT_EQ("http://google.com/", statement.ColumnString(2));
+ EXPECT_EQ(1, statement.ColumnInt64(1));
+
+ ASSERT_TRUE(statement.Step());
+ EXPECT_EQ(4, statement.ColumnInt64(0));
+ EXPECT_EQ("www.google.com/", statement.ColumnString(2));
+ EXPECT_EQ(3, statement.ColumnInt64(1));
+
+ EXPECT_FALSE(statement.Step());
+}
+
+} // namespace history
« no previous file with comments | « chrome/browser/history/android/android_urls_database.cc ('k') | chrome/browser/history/history_database.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698