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

Unified Diff: chrome/browser/sync_file_system/drive/metadata_db_migration_util_unittest.cc

Issue 15951007: SyncFS: Factor out DriveMetadataDB migration functions (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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
Index: chrome/browser/sync_file_system/drive/metadata_db_migration_util_unittest.cc
diff --git a/chrome/browser/sync_file_system/drive/metadata_db_migration_util_unittest.cc b/chrome/browser/sync_file_system/drive/metadata_db_migration_util_unittest.cc
index e37361cd6e81d3c515df1e7cf1b41143e774d79e..15d9429d51a2ebda1e7c6dc0a4d0ba72a55be09b 100644
--- a/chrome/browser/sync_file_system/drive/metadata_db_migration_util_unittest.cc
+++ b/chrome/browser/sync_file_system/drive/metadata_db_migration_util_unittest.cc
@@ -46,6 +46,114 @@ TEST(DriveMetadataDBMigrationUtilTest, RemoveWapiIdPrefix) {
EXPECT_EQ("foo:xxx", RemoveWapiIdPrefix("foo:xxx"));
}
+TEST(DriveMetadataDBMigrationUtilTest, MigrationFromV0) {
+ const char kDatabaseVersionKey[] = "VERSION";
+ const char kChangeStampKey[] = "CHANGE_STAMP";
+ const char kSyncRootDirectoryKey[] = "SYNC_ROOT_DIR";
+ const char kDriveMetadataKeyPrefix[] = "METADATA: ";
+ const char kDriveBatchSyncOriginKeyPrefix[] = "BSYNC_ORIGIN: ";
+ const char kDriveIncrementalSyncOriginKeyPrefix[] = "ISYNC_ORIGIN: ";
+
+ const GURL kOrigin1("chrome-extension://example1");
+ const GURL kOrigin2("chrome-extension://example2");
+ const std::string kSyncRootResourceId("folder:sync_root_resource_id");
+ const std::string kResourceId1("folder:hoge");
+ const std::string kResourceId2("folder:fuga");
+ const std::string kFileResourceId("file:piyo");
+ const base::FilePath kFile(FPL("foo bar"));
+ const std::string kFileMD5("file_md5");
+
+ const char kV0ServiceName[] = "drive";
+ ASSERT_TRUE(RegisterSyncableFileSystem(kV0ServiceName));
+
+ base::ScopedTempDir base_dir;
+ ASSERT_TRUE(base_dir.CreateUniqueTempDir());
+
+ leveldb::Options options;
+ options.create_if_missing = true;
+ leveldb::DB* db_ptr = NULL;
+ std::string db_dir = fileapi::FilePathToString(
+ base_dir.path().Append(DriveMetadataStore::kDatabaseName));
+ leveldb::Status status = leveldb::DB::Open(options, db_dir, &db_ptr);
+
+ scoped_ptr<leveldb::DB> db(db_ptr);
+ ASSERT_TRUE(status.ok());
+
+ // Setup the database with the scheme version 0.
+ leveldb::WriteBatch batch;
+ batch.Put(kChangeStampKey, "1");
+ batch.Put(kSyncRootDirectoryKey, kSyncRootResourceId);
+
+ fileapi::FileSystemURL url =
+ CreateSyncableFileSystemURL(kOrigin1, kV0ServiceName, kFile);
+
+ // Setup drive metadata.
+ DriveMetadata drive_metadata;
+ drive_metadata.set_resource_id(kFileResourceId);
+ drive_metadata.set_md5_checksum(kFileMD5);
+ drive_metadata.set_conflicted(false);
+ drive_metadata.set_to_be_fetched(false);
+
+ std::string serialized_url;
+ SerializeSyncableFileSystemURL(url, &serialized_url);
+ std::string metadata_string;
+ drive_metadata.SerializeToString(&metadata_string);
+ batch.Put(kDriveMetadataKeyPrefix + serialized_url, metadata_string);
+
+ // Setup batch sync origin and incremental sync origin.
+ batch.Put(kDriveBatchSyncOriginKeyPrefix + kOrigin1.spec(), kResourceId1);
+ batch.Put(kDriveIncrementalSyncOriginKeyPrefix + kOrigin2.spec(),
+ kResourceId2);
+
+ status = db->Write(leveldb::WriteOptions(), &batch);
+ EXPECT_EQ(SYNC_STATUS_OK, LevelDBStatusToSyncStatusCode(status));
+ EXPECT_TRUE(RevokeSyncableFileSystem(kV0ServiceName));
+
+ // Migrate the database.
+ drive::MigrateDatabaseFromV0ToV1(db.get());
+
+ scoped_ptr<leveldb::Iterator> itr(db->NewIterator(leveldb::ReadOptions()));
+
+ // Verify DB scheme version.
+ int64 database_version = 0;
+ itr->Seek(kDatabaseVersionKey);
+ EXPECT_TRUE(itr->Valid());
+ EXPECT_TRUE(base::StringToInt64(itr->value().ToString(), &database_version));
+ EXPECT_EQ(1, database_version);
+
+ // Verify the largest changestamp.
+ int64 changestamp = 0;
+ itr->Seek(kChangeStampKey);
+ EXPECT_TRUE(itr->Valid());
+ EXPECT_TRUE(base::StringToInt64(itr->value().ToString(), &changestamp));
+ EXPECT_EQ(1, changestamp);
+
+ // Verify the sync root directory.
+ itr->Seek(kSyncRootDirectoryKey);
+ EXPECT_TRUE(itr->Valid());
+ EXPECT_EQ(kSyncRootResourceId, itr->value().ToString());
+
+ // Verify the metadata.
+ itr->Seek(kDriveMetadataKeyPrefix);
+ EXPECT_TRUE(itr->Valid());
+ DriveMetadata metadata;
+ EXPECT_TRUE(metadata.ParseFromString(itr->value().ToString()));
+ EXPECT_EQ(kFileResourceId, metadata.resource_id());
+ EXPECT_EQ(kFileMD5, metadata.md5_checksum());
+ EXPECT_FALSE(metadata.conflicted());
+ EXPECT_FALSE(metadata.to_be_fetched());
+
+ // Verify the batch sync origin.
+ itr->Seek(kDriveBatchSyncOriginKeyPrefix);
+ EXPECT_TRUE(itr->Valid());
+ EXPECT_EQ(kResourceId1, itr->value().ToString());
+
+ // Verify the incremental sync origin.
+ itr->Seek(kDriveIncrementalSyncOriginKeyPrefix);
+ EXPECT_TRUE(itr->Valid());
+ EXPECT_EQ(kResourceId2, itr->value().ToString());
+}
+
TEST(DriveMetadataDBMigrationUtilTest, MigrationFromV1) {
const char kDatabaseVersionKey[] = "VERSION";
const char kChangeStampKey[] = "CHANGE_STAMP";
@@ -124,6 +232,13 @@ TEST(DriveMetadataDBMigrationUtilTest, MigrationFromV1) {
EXPECT_TRUE(base::StringToInt64(itr->value().ToString(), &database_version));
EXPECT_EQ(2, database_version);
+ // Verify the largest changestamp.
+ int64 changestamp = 0;
+ itr->Seek(kChangeStampKey);
+ EXPECT_TRUE(itr->Valid());
+ EXPECT_TRUE(base::StringToInt64(itr->value().ToString(), &changestamp));
+ EXPECT_EQ(1, changestamp);
+
// Verify the sync root directory.
itr->Seek(kSyncRootDirectoryKey);
EXPECT_TRUE(itr->Valid());

Powered by Google App Engine
This is Rietveld 408576698