SyncFS: Convert WAPI ResourceID to DriveAPI FileID

chrome/browser/sync_file_system/drive/metadata_db_migration_util_unittest.cc

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
new file mode 100644
index 0000000000000000000000000000000000000000..8ad732afa252cb0a51d769999259d30504520fb9
--- /dev/null
+++ b/chrome/browser/sync_file_system/drive/metadata_db_migration_util_unittest.cc
@@ -0,0 +1,158 @@
+// Copyright 2013 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/sync_file_system/drive/metadata_db_migration_util.h"
+
+#include "base/files/scoped_temp_dir.h"
+#include "base/string_util.h"
+#include "base/strings/string_number_conversions.h"
+#include "chrome/browser/sync_file_system/drive_metadata_store.h"
+#include "googleurl/src/gurl.h"
+#include "testing/gtest/include/gtest/gtest.h"
+#include "third_party/leveldatabase/src/include/leveldb/db.h"
+#include "third_party/leveldatabase/src/include/leveldb/write_batch.h"
+#include "webkit/common/fileapi/file_system_util.h"
+#include "webkit/fileapi/syncable/syncable_file_system_util.h"
+
+#define FPL FILE_PATH_LITERAL
+
+namespace sync_file_system {
+namespace drive {
+
+TEST(DriveMetadataDBMigrationUtilTest, AddWapiFilePrefix) {
+  EXPECT_EQ("file:xxx", AddWapiFilePrefix("xxx"));
+  EXPECT_EQ("file:yyy", AddWapiFilePrefix("file:yyy"));
+  EXPECT_EQ("folder:zzz", AddWapiFilePrefix("folder:zzz"));
+}
+
+TEST(DriveMetadataDBMigrationUtilTest, AddWapiFolderPrefix) {
+  EXPECT_EQ("folder:xxx", AddWapiFolderPrefix("xxx"));
+  EXPECT_EQ("file:yyy", AddWapiFolderPrefix("file:yyy"));
+  EXPECT_EQ("folder:zzz", AddWapiFolderPrefix("folder:zzz"));
+}
+
+TEST(DriveMetadataDBMigrationUtilTest, AddWapiIdPrefix) {
+  DriveMetadata_ResourceType type_file =
+      DriveMetadata_ResourceType_RESOURCE_TYPE_FILE;
+  DriveMetadata_ResourceType type_folder =
+      DriveMetadata_ResourceType_RESOURCE_TYPE_FOLDER;
+
+  EXPECT_EQ("file:xxx", AddWapiIdPrefix("xxx", type_file));
+  EXPECT_EQ("folder:xxx", AddWapiIdPrefix("xxx", type_folder));
+  EXPECT_EQ("file:yyy", AddWapiIdPrefix("file:yyy", type_file));
+  EXPECT_EQ("file:yyy", AddWapiIdPrefix("file:yyy", type_folder));
+  EXPECT_EQ("folder:zzz", AddWapiIdPrefix("folder:zzz", type_file));
+  EXPECT_EQ("folder:zzz", AddWapiIdPrefix("folder:zzz", type_folder));
+}
+
+TEST(DriveMetadataDBMigrationUtilTest, RemoveWapiIdPrefix) {
+  EXPECT_EQ("xxx", RemoveWapiIdPrefix("xxx"));
+  EXPECT_EQ("yyy", RemoveWapiIdPrefix("file:yyy"));
+  EXPECT_EQ("zzz", RemoveWapiIdPrefix("folder:zzz"));

[kinuko, 2013/05/28 15:09:20]

nit: should we also test something like "foo:xxx" -> "foo:xxx" (while it
wouldn't have much meaning in a real situation)

[nhiroki, 2013/05/29 04:10:18]

Done.

+}
+
+TEST(DriveMetadataDBMigrationUtilTest, MigrationFromV1) {
+  const char kDatabaseVersionKey[] = "VERSION";
+  const char kChangeStampKey[] = "CHANGE_STAMP";
+  const char kSyncRootDirectoryKey[] = "SYNC_ROOT_DIR";
+  const char kDriveMetadataKeyPrefix[] = "METADATA: ";
+  const char kMetadataKeySeparator = ' ';
+  const char kDriveIncrementalSyncOriginKeyPrefix[] = "ISYNC_ORIGIN: ";
+  const char kDriveDisabledOriginKeyPrefix[] = "DISABLED_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 kV1ServiceName[] = "drive";
+  ASSERT_TRUE(RegisterSyncableFileSystem(kV1ServiceName));
+
+  base::ScopedTempDir base_dir;
+  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 1.
+  leveldb::WriteBatch batch;
+  batch.Put(kDatabaseVersionKey, "1");
+  batch.Put(kChangeStampKey, "1");
+  batch.Put(kSyncRootDirectoryKey, kSyncRootResourceId);
+
+  fileapi::FileSystemURL url =
+      CreateSyncableFileSystemURL(kOrigin1, kV1ServiceName, 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 metadata_string;
+  drive_metadata.SerializeToString(&metadata_string);
+  std::string metadata_key = kDriveMetadataKeyPrefix + kOrigin1.spec() +
+                             kMetadataKeySeparator + url.path().AsUTF8Unsafe();
+  batch.Put(metadata_key, metadata_string);
+
+  // Setup incremental sync origin and disabled origin.
+  batch.Put(kDriveIncrementalSyncOriginKeyPrefix + kOrigin1.spec(),
+            kResourceId1);
+  batch.Put(kDriveDisabledOriginKeyPrefix + kOrigin2.spec(),
+            kResourceId2);
+
+  status = db->Write(leveldb::WriteOptions(), &batch);
+  EXPECT_EQ(SYNC_STATUS_OK, LevelDBStatusToSyncStatusCode(status));
+  EXPECT_TRUE(RevokeSyncableFileSystem(kV1ServiceName));
+
+  // Migrate the database.
+  drive::MigrateDatabaseFromV1ToV2(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(2, database_version);
+
+  // Verify the sync root directory.
+  itr->Seek(kSyncRootDirectoryKey);
+  EXPECT_TRUE(itr->Valid());
+  EXPECT_EQ(RemoveWapiIdPrefix(kSyncRootResourceId), itr->value().ToString());
+
+  // Verify metadata.
+  itr->Seek(kDriveMetadataKeyPrefix);
+  EXPECT_TRUE(itr->Valid());
+  DriveMetadata metadata;
+  EXPECT_TRUE(metadata.ParseFromString(itr->value().ToString()));
+  EXPECT_EQ(RemoveWapiIdPrefix(kFileResourceId), metadata.resource_id());
+  EXPECT_EQ(kFileMD5, metadata.md5_checksum());
+  EXPECT_FALSE(metadata.conflicted());
+  EXPECT_FALSE(metadata.to_be_fetched());
+
+  // Verify incremental sync origins.
+  itr->Seek(kDriveIncrementalSyncOriginKeyPrefix);
+  EXPECT_TRUE(itr->Valid());
+  EXPECT_EQ(RemoveWapiIdPrefix(kResourceId1), itr->value().ToString());
+
+  // Verify incremental sync origins.

[kinuko, 2013/05/28 15:09:20]

incremental -> disabled

[nhiroki, 2013/05/29 04:10:18]

Done.

+  itr->Seek(kDriveDisabledOriginKeyPrefix);
+  EXPECT_TRUE(itr->Valid());
+  EXPECT_EQ(RemoveWapiIdPrefix(kResourceId2), itr->value().ToString());
+}
+
+}  // namespace drive
+}  // namespace sync_file_system