Migrate old Shortcuts DB data to conform to new type values

chrome/browser/history/shortcuts_database_unittest.cc

 // 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/files/scoped_temp_dir.h"
 #include "base/format_macros.h"
 #include "base/path_service.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/time/time.h"
@@ skipping 281 matching lines @@
     // The other three columns have default values.
     EXPECT_EQ(content::PAGE_TRANSITION_TYPED,
               static_cast<content::PageTransition>(statement.ColumnInt(2)));
     EXPECT_EQ(AutocompleteMatchType::HISTORY_TITLE,
               static_cast<AutocompleteMatch::Type>(statement.ColumnInt(3)));
     EXPECT_TRUE(statement.ColumnString(4).empty());
   }
   EXPECT_TRUE(statement.Succeeded());
 }
 
+TEST(ShortcutsDatabaseMigrationTest, MigrateV2ToV3) {

[Peter Kasting, 2014/02/12 00:27:50]

Nit: Hmm, I see we're using "v2" and "v3" here because of how we used version
numbers in the previous test, but it's really confusing that they don't match
the meta table numbers.

We should probably rename the above test something like
"MigrateTableAddFillIntoEdit" and rename its .sql file to use
"no_fill_into_edit" in place of "v1"; then we can call this test "MigrateV0ToV1"
and name its .sql file similarly.

The other option would be to start the meta table with version 3, but that would
also be confusing since there never would have been a version 1 or 2 and we
wouldn't be able to look for those versions to check for databases that needed
the older migration step.

[Anuj, 2014/02/12 23:54:40]

I think we are caught in a no good-names situation. Made the changes you
suggested.

+  // Open the v1 test file and use it to create a test database in a temp dir.
+  base::FilePath sql_path;
+  ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &sql_path));
+  sql_path = sql_path.AppendASCII("History").AppendASCII("Shortcuts.v2.sql");
+  base::ScopedTempDir temp_dir;
+  ASSERT_TRUE(temp_dir.CreateUniqueTempDir());
+  base::FilePath db_path(temp_dir.path().AppendASCII("TestShortcuts.db"));
+  ASSERT_TRUE(sql::test::CreateDatabaseFromSQL(db_path, sql_path));
+
+  // Create a ShortcutsDatabase from the test database, which will migrate the
+  // test database to the current version.
+  {
+    scoped_refptr<ShortcutsDatabase> db(new ShortcutsDatabase(db_path));
+    db->Init();
+  }
+
+  // Check the values in each of the new columns.

[Peter Kasting, 2014/02/12 00:27:50]

Nit: Maybe "Check that all the old type values got converted to new values"?

[Anuj, 2014/02/12 23:54:40]

Done.

+  sql::Connection connection;
+  ASSERT_TRUE(connection.Open(db_path));
+  sql::Statement statement(connection.GetUniqueStatement(
+      "SELECT count(1) FROM omni_box_shortcuts WHERE type in (9, 10, 11, 12)"));
+  ASSERT_TRUE(statement.is_valid());
+  while (statement.Step()) {

[Peter Kasting, 2014/02/12 00:27:50]

Nit: {} not necessary

[Anuj, 2014/02/12 23:54:40]

Done.

+    EXPECT_EQ(0, statement.ColumnInt(0));
+  }
+  EXPECT_TRUE(statement.Succeeded());
+}
+
 }  // namespace history