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

Unified Diff: chrome/browser/sync_file_system/drive_metadata_store_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_store_unittest.cc
diff --git a/chrome/browser/sync_file_system/drive_metadata_store_unittest.cc b/chrome/browser/sync_file_system/drive_metadata_store_unittest.cc
index acb3b989280c918456e4ba01c9f70b8148e59ed4..a01890c5d8b6a1c1a085bbc2762b0eb8266852cd 100644
--- a/chrome/browser/sync_file_system/drive_metadata_store_unittest.cc
+++ b/chrome/browser/sync_file_system/drive_metadata_store_unittest.cc
@@ -588,82 +588,6 @@ TEST_F(DriveMetadataStoreTest, GetResourceIdForOrigin) {
VerifyReverseMap();
}
-TEST_F(DriveMetadataStoreTest, MigrationFromV0) {
- 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 kChangeStampKey[] = "CHANGE_STAMP";
- const char kSyncRootDirectoryKey[] = "SYNC_ROOT_DIR";
- const char kDriveMetadataKeyPrefix[] = "METADATA: ";
- const char kDriveBatchSyncOriginKeyPrefix[] = "BSYNC_ORIGIN: ";
- const char kDriveIncrementalSyncOriginKeyPrefix[] = "ISYNC_ORIGIN: ";
-
- const char kV0ServiceName[] = "drive";
- ASSERT_TRUE(RegisterSyncableFileSystem(kV0ServiceName));
- leveldb::Options options;
- options.create_if_missing = true;
- leveldb::DB* db_ptr = NULL;
- std::string db_dir = fileapi::FilePathToString(
- base_dir().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());
-
- leveldb::WriteOptions write_options;
- db->Put(write_options, kChangeStampKey, "1");
- db->Put(write_options, kSyncRootDirectoryKey, kSyncRootResourceId);
-
- 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);
-
- fileapi::FileSystemURL url = CreateSyncableFileSystemURL(
- kOrigin1, kV0ServiceName, kFile);
- std::string serialized_url;
- SerializeSyncableFileSystemURL(url, &serialized_url);
- std::string metadata_string;
- drive_metadata.SerializeToString(&metadata_string);
-
- db->Put(write_options,
- kDriveMetadataKeyPrefix + serialized_url, metadata_string);
- db->Put(write_options,
- kDriveBatchSyncOriginKeyPrefix + kOrigin1.spec(), kResourceId1);
- db->Put(write_options,
- kDriveIncrementalSyncOriginKeyPrefix + kOrigin2.spec(),
- kResourceId2);
- EXPECT_TRUE(RevokeSyncableFileSystem(kV0ServiceName));
- MarkAsCreated();
- }
-
- InitializeDatabase();
-
- EXPECT_EQ(1, metadata_store()->GetLargestChangeStamp());
- EXPECT_EQ(kSyncRootResourceId, metadata_store()->sync_root_directory());
- EXPECT_EQ(kResourceId2, metadata_store()->GetResourceIdForOrigin(kOrigin2));
-
- DriveMetadata metadata;
- EXPECT_EQ(SYNC_STATUS_OK,
- metadata_store()->ReadEntry(
- CreateSyncableFileSystemURL(kOrigin1, kServiceName, kFile),
- &metadata));
- EXPECT_EQ(kFileResourceId, metadata.resource_id());
- EXPECT_EQ(kFileMD5, metadata.md5_checksum());
- EXPECT_FALSE(metadata.conflicted());
- EXPECT_FALSE(metadata.to_be_fetched());
-
- VerifyReverseMap();
-}
-
TEST_F(DriveMetadataStoreTest, DeprecateBatchSyncOrigins) {
// Make sure that previously saved batch sync origins were deleted from the DB
// as they are no longer used.

Powered by Google App Engine
This is Rietveld 408576698