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

Unified Diff: chrome/browser/chromeos/drive/drive_file_system_unittest.cc

Issue 13149003: drive: Use "/drive/root" namespace and fix Files app and tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 7 years, 9 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/chromeos/drive/drive_file_system_unittest.cc
diff --git a/chrome/browser/chromeos/drive/drive_file_system_unittest.cc b/chrome/browser/chromeos/drive/drive_file_system_unittest.cc
index 8fee4a801259b0499a9d3f656747fc5535ccf3a9..d2398760601194eccad024b4294d16d681160927 100644
--- a/chrome/browser/chromeos/drive/drive_file_system_unittest.cc
+++ b/chrome/browser/chromeos/drive/drive_file_system_unittest.cc
@@ -308,9 +308,9 @@ class DriveFileSystemTest : public testing::Test {
};
// Saves a file representing a filesystem with directories:
- // drive, drive/Dir1, drive/Dir1/SubDir2
+ // drive/root, drive/root/Dir1, drive/root/Dir1/SubDir2
// and files
- // drive/File1, drive/Dir1/File2, drive/Dir1/SubDir2/File3.
+ // drive/root/File1, drive/root/Dir1/File2, drive/root/Dir1/SubDir2/File3.
// If |use_up_to_date_timestamp| is true, sets the changestamp to 654321,
// equal to that of "account_metadata.json" test data, indicating the cache is
// holding the latest file system info.
@@ -340,7 +340,12 @@ class DriveFileSystemTest : public testing::Test {
if (error != DRIVE_FILE_OK)
return false;
- // drive/File1
+ // drive/root is already prepared by DriveResourceMetadata.
+ // TODO(haruki): Create drive/root here when we start creating it in
+ // ChangeListLoader.
+ base::FilePath file_path;
+
+ // drive/root/File1
DriveEntryProto file1;
file1.set_title("File1");
file1.set_resource_id("resource_id:File1");
@@ -349,7 +354,6 @@ class DriveFileSystemTest : public testing::Test {
file1.mutable_file_specific_info()->set_file_md5("md5");
file1.mutable_file_info()->set_is_directory(false);
file1.mutable_file_info()->set_size(1048576);
- base::FilePath file_path;
resource_metadata->AddEntry(
file1,
google_apis::test_util::CreateCopyResultCallback(&error, &file_path));
@@ -357,7 +361,7 @@ class DriveFileSystemTest : public testing::Test {
if (error != DRIVE_FILE_OK)
return false;
- // drive/Dir1
+ // drive/root/Dir1
DriveEntryProto dir1;
dir1.set_title("Dir1");
dir1.set_resource_id("resource_id:Dir1");
@@ -371,7 +375,7 @@ class DriveFileSystemTest : public testing::Test {
if (error != DRIVE_FILE_OK)
return false;
- // drive/Dir1/File2
+ // drive/root/Dir1/File2
DriveEntryProto file2;
file2.set_title("File2");
file2.set_resource_id("resource_id:File2");
@@ -387,7 +391,7 @@ class DriveFileSystemTest : public testing::Test {
if (error != DRIVE_FILE_OK)
return false;
- // drive/Dir1/SubDir2
+ // drive/root/Dir1/SubDir2
DriveEntryProto dir2;
dir2.set_title("SubDir2");
dir2.set_resource_id("resource_id:SubDir2");
@@ -401,7 +405,7 @@ class DriveFileSystemTest : public testing::Test {
if (error != DRIVE_FILE_OK)
return false;
- // drive/Dir1/SubDir2/File3
+ // drive/root/Dir1/SubDir2/File3
DriveEntryProto file3;
file3.set_title("File3");
file3.set_resource_id("resource_id:File3");
@@ -489,7 +493,7 @@ void AsyncInitializationCallback(
TEST_F(DriveFileSystemTest, DuplicatedAsyncInitialization) {
// The root directory will be loaded that triggers the event.
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root"))))).Times(1);
int counter = 0;
ReadDirectoryWithSettingCallback callback = base::Bind(
@@ -499,9 +503,9 @@ TEST_F(DriveFileSystemTest, DuplicatedAsyncInitialization) {
&message_loop_);
file_system_->ReadDirectoryByPath(
- base::FilePath(FILE_PATH_LITERAL("drive")), callback);
+ base::FilePath(FILE_PATH_LITERAL("drive/root")), callback);
file_system_->ReadDirectoryByPath(
- base::FilePath(FILE_PATH_LITERAL("drive")), callback);
+ base::FilePath(FILE_PATH_LITERAL("drive/root")), callback);
message_loop_.Run(); // Wait to get our result
EXPECT_EQ(2, counter);
@@ -521,20 +525,34 @@ TEST_F(DriveFileSystemTest, DuplicatedAsyncInitialization) {
<< ": " << fake_drive_service_->resource_list_load_count();
}
-TEST_F(DriveFileSystemTest, GetRootEntry) {
- const base::FilePath kFilePath = base::FilePath(FILE_PATH_LITERAL("drive"));
+TEST_F(DriveFileSystemTest, GetGrandRootEntry) {
+ const base::FilePath kFilePath =
+ base::FilePath(FILE_PATH_LITERAL("drive"));
+ scoped_ptr<DriveEntryProto> entry = GetEntryInfoByPathSync(kFilePath);
+ ASSERT_TRUE(entry.get());
+ EXPECT_EQ(util::kDriveGrandRootSpecialResourceId, entry->resource_id());
+
+ // Getting the grand root entry should not cause the resource load to happen.
+ EXPECT_EQ(0, fake_drive_service_->about_resource_load_count());
+ EXPECT_EQ(0, fake_drive_service_->resource_list_load_count());
+}
+
+TEST_F(DriveFileSystemTest, GetMyDriveRootEntry) {
satorux1 2013/04/05 01:55:30 Shouldn't we have a test for drive/other too?
Haruki Sato 2013/04/05 04:35:25 Done.
+ const base::FilePath kFilePath =
+ base::FilePath(FILE_PATH_LITERAL("drive/root"));
scoped_ptr<DriveEntryProto> entry = GetEntryInfoByPathSync(kFilePath);
ASSERT_TRUE(entry.get());
EXPECT_EQ(fake_drive_service_->GetRootResourceId(), entry->resource_id());
- // Getting the root entry should not cause the resource load to happen.
+ // Getting the "My Drive" root entry should not cause the resource load to
+ // happen.
EXPECT_EQ(0, fake_drive_service_->about_resource_load_count());
EXPECT_EQ(0, fake_drive_service_->resource_list_load_count());
}
TEST_F(DriveFileSystemTest, GetNonRootEntry) {
const base::FilePath kFilePath =
- base::FilePath(FILE_PATH_LITERAL("drive/whatever.txt"));
+ base::FilePath(FILE_PATH_LITERAL("drive/root/whatever.txt"));
scoped_ptr<DriveEntryProto> entry = GetEntryInfoByPathSync(kFilePath);
// The entry should not exist as the resource metadata only contains the
// root entry now.
@@ -548,7 +566,8 @@ TEST_F(DriveFileSystemTest, GetNonRootEntry) {
TEST_F(DriveFileSystemTest, SearchRootDirectory) {
ASSERT_TRUE(LoadRootFeedDocument());
- const base::FilePath kFilePath = base::FilePath(FILE_PATH_LITERAL("drive"));
+ const base::FilePath kFilePath =
+ base::FilePath(FILE_PATH_LITERAL("drive/root"));
scoped_ptr<DriveEntryProto> entry = GetEntryInfoByPathSync(kFilePath);
ASSERT_TRUE(entry.get());
EXPECT_EQ(fake_drive_service_->GetRootResourceId(), entry->resource_id());
@@ -562,7 +581,7 @@ TEST_F(DriveFileSystemTest, SearchExistingFile) {
ASSERT_TRUE(LoadRootFeedDocument());
const base::FilePath kFilePath = base::FilePath(
- FILE_PATH_LITERAL("drive/File 1.txt"));
+ FILE_PATH_LITERAL("drive/root/File 1.txt"));
scoped_ptr<DriveEntryProto> entry = GetEntryInfoByPathSync(kFilePath);
ASSERT_TRUE(entry.get());
EXPECT_EQ("file:2_file_resource_id", entry->resource_id());
@@ -572,7 +591,7 @@ TEST_F(DriveFileSystemTest, SearchExistingDocument) {
ASSERT_TRUE(LoadRootFeedDocument());
const base::FilePath kFilePath = base::FilePath(
- FILE_PATH_LITERAL("drive/Document 1.gdoc"));
+ FILE_PATH_LITERAL("drive/root/Document 1.gdoc"));
scoped_ptr<DriveEntryProto> entry = GetEntryInfoByPathSync(kFilePath);
ASSERT_TRUE(entry.get());
EXPECT_EQ("document:5_document_resource_id", entry->resource_id());
@@ -582,7 +601,7 @@ TEST_F(DriveFileSystemTest, SearchNonExistingFile) {
ASSERT_TRUE(LoadRootFeedDocument());
const base::FilePath kFilePath = base::FilePath(
- FILE_PATH_LITERAL("drive/nonexisting.file"));
+ FILE_PATH_LITERAL("drive/root/nonexisting.file"));
scoped_ptr<DriveEntryProto> entry = GetEntryInfoByPathSync(kFilePath);
ASSERT_FALSE(entry.get());
}
@@ -591,18 +610,18 @@ TEST_F(DriveFileSystemTest, SearchEncodedFileNames) {
ASSERT_TRUE(LoadRootFeedDocument());
const base::FilePath kFilePath1 = base::FilePath(
- FILE_PATH_LITERAL("drive/Slash / in file 1.txt"));
+ FILE_PATH_LITERAL("drive/root/Slash / in file 1.txt"));
scoped_ptr<DriveEntryProto> entry = GetEntryInfoByPathSync(kFilePath1);
ASSERT_FALSE(entry.get());
const base::FilePath kFilePath2 = base::FilePath::FromUTF8Unsafe(
- "drive/Slash \xE2\x88\x95 in file 1.txt");
+ "drive/root/Slash \xE2\x88\x95 in file 1.txt");
entry = GetEntryInfoByPathSync(kFilePath2);
ASSERT_TRUE(entry.get());
EXPECT_EQ("file:slash_file_resource_id", entry->resource_id());
const base::FilePath kFilePath3 = base::FilePath::FromUTF8Unsafe(
- "drive/Slash \xE2\x88\x95 in directory/Slash SubDir File.txt");
+ "drive/root/Slash \xE2\x88\x95 in directory/Slash SubDir File.txt");
entry = GetEntryInfoByPathSync(kFilePath3);
ASSERT_TRUE(entry.get());
EXPECT_EQ("file:slash_subdir_file", entry->resource_id());
@@ -612,18 +631,18 @@ TEST_F(DriveFileSystemTest, SearchEncodedFileNamesLoadingRoot) {
ASSERT_TRUE(LoadRootFeedDocument());
const base::FilePath kFilePath1 = base::FilePath(
- FILE_PATH_LITERAL("drive/Slash / in file 1.txt"));
+ FILE_PATH_LITERAL("drive/root/Slash / in file 1.txt"));
scoped_ptr<DriveEntryProto> entry = GetEntryInfoByPathSync(kFilePath1);
ASSERT_FALSE(entry.get());
const base::FilePath kFilePath2 = base::FilePath::FromUTF8Unsafe(
- "drive/Slash \xE2\x88\x95 in file 1.txt");
+ "drive/root/Slash \xE2\x88\x95 in file 1.txt");
entry = GetEntryInfoByPathSync(kFilePath2);
ASSERT_TRUE(entry.get());
EXPECT_EQ("file:slash_file_resource_id", entry->resource_id());
const base::FilePath kFilePath3 = base::FilePath::FromUTF8Unsafe(
- "drive/Slash \xE2\x88\x95 in directory/Slash SubDir File.txt");
+ "drive/root/Slash \xE2\x88\x95 in directory/Slash SubDir File.txt");
entry = GetEntryInfoByPathSync(kFilePath3);
ASSERT_TRUE(entry.get());
EXPECT_EQ("file:slash_subdir_file", entry->resource_id());
@@ -633,13 +652,13 @@ TEST_F(DriveFileSystemTest, SearchDuplicateNames) {
ASSERT_TRUE(LoadRootFeedDocument());
const base::FilePath kFilePath1 = base::FilePath(
- FILE_PATH_LITERAL("drive/Duplicate Name.txt"));
+ FILE_PATH_LITERAL("drive/root/Duplicate Name.txt"));
scoped_ptr<DriveEntryProto> entry = GetEntryInfoByPathSync(kFilePath1);
ASSERT_TRUE(entry.get());
const std::string resource_id1 = entry->resource_id();
const base::FilePath kFilePath2 = base::FilePath(
- FILE_PATH_LITERAL("drive/Duplicate Name (2).txt"));
+ FILE_PATH_LITERAL("drive/root/Duplicate Name (2).txt"));
entry = GetEntryInfoByPathSync(kFilePath2);
ASSERT_TRUE(entry.get());
const std::string resource_id2 = entry->resource_id();
@@ -658,7 +677,7 @@ TEST_F(DriveFileSystemTest, SearchExistingDirectory) {
ASSERT_TRUE(LoadRootFeedDocument());
const base::FilePath kFilePath = base::FilePath(
- FILE_PATH_LITERAL("drive/Directory 1"));
+ FILE_PATH_LITERAL("drive/root/Directory 1"));
scoped_ptr<DriveEntryProto> entry = GetEntryInfoByPathSync(kFilePath);
ASSERT_TRUE(entry.get());
ASSERT_EQ("folder:1_folder_resource_id", entry->resource_id());
@@ -672,7 +691,7 @@ TEST_F(DriveFileSystemTest, SearchInSubdir) {
ASSERT_TRUE(LoadRootFeedDocument());
const base::FilePath kFilePath = base::FilePath(
- FILE_PATH_LITERAL("drive/Directory 1/SubDirectory File 1.txt"));
+ FILE_PATH_LITERAL("drive/root/Directory 1/SubDirectory File 1.txt"));
scoped_ptr<DriveEntryProto> entry = GetEntryInfoByPathSync(kFilePath);
ASSERT_TRUE(entry.get());
ASSERT_EQ("file:subdirectory_file_1_id", entry->resource_id());
@@ -683,7 +702,7 @@ TEST_F(DriveFileSystemTest, SearchInSubSubdir) {
ASSERT_TRUE(LoadRootFeedDocument());
const base::FilePath kFilePath = base::FilePath(
- FILE_PATH_LITERAL("drive/Directory 1/Sub Directory Folder/"
+ FILE_PATH_LITERAL("drive/root/Directory 1/Sub Directory Folder/"
"Sub Sub Directory Folder"));
scoped_ptr<DriveEntryProto> entry = GetEntryInfoByPathSync(kFilePath);
ASSERT_TRUE(entry.get());
@@ -691,13 +710,12 @@ TEST_F(DriveFileSystemTest, SearchInSubSubdir) {
}
TEST_F(DriveFileSystemTest, ReadDirectoryByPath_Root) {
- // The root directory will be loaded that triggers the event.
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root"))))).Times(1);
// ReadDirectoryByPath() should kick off the resource list loading.
scoped_ptr<DriveEntryProtoVector> entries(
- ReadDirectoryByPathSync(base::FilePath::FromUTF8Unsafe("drive")));
+ ReadDirectoryByPathSync(base::FilePath::FromUTF8Unsafe("drive/root")));
// The root directory should be read correctly.
ASSERT_TRUE(entries.get());
EXPECT_EQ(8U, entries->size());
@@ -707,7 +725,7 @@ TEST_F(DriveFileSystemTest, ReadDirectoryByPath_NonRootDirectory) {
// ReadDirectoryByPath() should kick off the resource list loading.
scoped_ptr<DriveEntryProtoVector> entries(
ReadDirectoryByPathSync(
- base::FilePath::FromUTF8Unsafe("drive/Directory 1")));
+ base::FilePath::FromUTF8Unsafe("drive/root/Directory 1")));
// The non root directory should also be read correctly.
// There was a bug (crbug.com/181487), which broke this behavior.
// Make sure this is fixed.
@@ -719,27 +737,26 @@ TEST_F(DriveFileSystemTest, FilePathTests) {
ASSERT_TRUE(LoadRootFeedDocument());
EXPECT_TRUE(
- EntryExists(base::FilePath(FILE_PATH_LITERAL("drive/File 1.txt"))));
+ EntryExists(base::FilePath(FILE_PATH_LITERAL("drive/root/File 1.txt"))));
EXPECT_TRUE(
- EntryExists(base::FilePath(FILE_PATH_LITERAL("drive/Directory 1"))));
- EXPECT_TRUE(EntryExists(
- base::FilePath(
- FILE_PATH_LITERAL("drive/Directory 1/SubDirectory File 1.txt"))));
+ EntryExists(base::FilePath(FILE_PATH_LITERAL("drive/root/Directory 1"))));
+ EXPECT_TRUE(EntryExists(base::FilePath(
+ FILE_PATH_LITERAL("drive/root/Directory 1/SubDirectory File 1.txt"))));
}
TEST_F(DriveFileSystemTest, ChangeFeed_AddAndDeleteFileInRoot) {
ASSERT_TRUE(LoadRootFeedDocument());
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive"))))).Times(2);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root"))))).Times(2);
ASSERT_TRUE(LoadChangeFeed("chromeos/gdata/delta_file_added_in_root.json"));
- EXPECT_TRUE(
- EntryExists(base::FilePath(FILE_PATH_LITERAL("drive/Added file.gdoc"))));
+ EXPECT_TRUE(EntryExists(
+ base::FilePath(FILE_PATH_LITERAL("drive/root/Added file.gdoc"))));
ASSERT_TRUE(LoadChangeFeed("chromeos/gdata/delta_file_deleted_in_root.json"));
- EXPECT_FALSE(
- EntryExists(base::FilePath(FILE_PATH_LITERAL("drive/Added file.gdoc"))));
+ EXPECT_FALSE(EntryExists(
+ base::FilePath(FILE_PATH_LITERAL("drive/root/Added file.gdoc"))));
}
@@ -747,44 +764,48 @@ TEST_F(DriveFileSystemTest, ChangeFeed_AddAndDeleteFileFromExistingDirectory) {
ASSERT_TRUE(LoadRootFeedDocument());
EXPECT_TRUE(
- EntryExists(base::FilePath(FILE_PATH_LITERAL("drive/Directory 1"))));
+ EntryExists(base::FilePath(FILE_PATH_LITERAL("drive/root/Directory 1"))));
// Add file to an existing directory.
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root"))))).Times(1);
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive/Directory 1"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root/Directory 1")))))
+ .Times(1);
ASSERT_TRUE(
LoadChangeFeed("chromeos/gdata/delta_file_added_in_directory.json"));
EXPECT_TRUE(EntryExists(base::FilePath(
- FILE_PATH_LITERAL("drive/Directory 1/Added file.gdoc"))));
+ FILE_PATH_LITERAL("drive/root/Directory 1/Added file.gdoc"))));
// Remove that file from the directory.
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive/Directory 1"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root/Directory 1")))))
+ .Times(1);
ASSERT_TRUE(
LoadChangeFeed("chromeos/gdata/delta_file_deleted_in_directory.json"));
EXPECT_TRUE(
- EntryExists(base::FilePath(FILE_PATH_LITERAL("drive/Directory 1"))));
+ EntryExists(base::FilePath(FILE_PATH_LITERAL("drive/root/Directory 1"))));
EXPECT_FALSE(EntryExists(base::FilePath(
- FILE_PATH_LITERAL("drive/Directory 1/Added file.gdoc"))));
+ FILE_PATH_LITERAL("drive/root/Directory 1/Added file.gdoc"))));
}
TEST_F(DriveFileSystemTest, ChangeFeed_AddFileToNewDirectory) {
ASSERT_TRUE(LoadRootFeedDocument());
// Add file to a new directory.
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root"))))).Times(1);
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive/New Directory"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root/New Directory")))))
+ .Times(1);
ASSERT_TRUE(
LoadChangeFeed("chromeos/gdata/delta_file_added_in_new_directory.json"));
EXPECT_TRUE(
- EntryExists(base::FilePath(FILE_PATH_LITERAL("drive/New Directory"))));
+ EntryExists(base::FilePath(
+ FILE_PATH_LITERAL("drive/root/New Directory"))));
EXPECT_TRUE(EntryExists(base::FilePath(
- FILE_PATH_LITERAL("drive/New Directory/File in new dir.gdoc"))));
+ FILE_PATH_LITERAL("drive/root/New Directory/File in new dir.gdoc"))));
}
TEST_F(DriveFileSystemTest, ChangeFeed_AddFileToNewButDeletedDirectory) {
@@ -802,41 +823,45 @@ TEST_F(DriveFileSystemTest, ChangeFeed_DirectoryMovedFromRootToDirectory) {
ASSERT_TRUE(LoadRootFeedDocument());
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 2"))));
+ "drive/root/Directory 2"))));
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 1"))));
+ "drive/root/Directory 1"))));
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 1/SubDirectory File 1.txt"))));
+ "drive/root/Directory 1/SubDirectory File 1.txt"))));
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 1/Sub Directory Folder"))));
+ "drive/root/Directory 1/Sub Directory Folder"))));
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 1/Sub Directory Folder/Sub Sub Directory Folder"))));
+ "drive/root/Directory 1/Sub Directory Folder/"
+ "Sub Sub Directory Folder"))));
- // This will move "Directory 1" from "drive/" to "drive/Directory 2/".
- EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive"))))).Times(1);
+ // This will move "Directory 1" from "drive/root/" to
+ // "drive/root/Directory 2/".
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive/Directory 1"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root"))))).Times(1);
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive/Directory 2"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root/Directory 1")))))
+ .Times(1);
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive/Directory 2/Directory 1")))))
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root/Directory 2")))))
.Times(1);
+ EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
+ Eq(base::FilePath(
+ FILE_PATH_LITERAL("drive/root/Directory 2/Directory 1"))))).Times(1);
ASSERT_TRUE(LoadChangeFeed(
"chromeos/gdata/delta_dir_moved_from_root_to_directory.json"));
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 2"))));
+ "drive/root/Directory 2"))));
EXPECT_FALSE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 1"))));
+ "drive/root/Directory 1"))));
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 2/Directory 1"))));
+ "drive/root/Directory 2/Directory 1"))));
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 2/Directory 1/SubDirectory File 1.txt"))));
+ "drive/root/Directory 2/Directory 1/SubDirectory File 1.txt"))));
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 2/Directory 1/Sub Directory Folder"))));
+ "drive/root/Directory 2/Directory 1/Sub Directory Folder"))));
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 2/Directory 1/Sub Directory Folder/"
+ "drive/root/Directory 2/Directory 1/Sub Directory Folder/"
"Sub Sub Directory Folder"))));
}
@@ -844,54 +869,58 @@ TEST_F(DriveFileSystemTest, ChangeFeed_FileMovedFromDirectoryToRoot) {
ASSERT_TRUE(LoadRootFeedDocument());
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 1"))));
+ "drive/root/Directory 1"))));
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 1/Sub Directory Folder"))));
+ "drive/root/Directory 1/Sub Directory Folder"))));
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 1/Sub Directory Folder/Sub Sub Directory Folder"))));
+ "drive/root/Directory 1/Sub Directory Folder/"
+ "Sub Sub Directory Folder"))));
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 1/SubDirectory File 1.txt"))));
+ "drive/root/Directory 1/SubDirectory File 1.txt"))));
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root"))))).Times(1);
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive/Directory 1"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root/Directory 1")))))
+ .Times(1);
ASSERT_TRUE(LoadChangeFeed(
"chromeos/gdata/delta_file_moved_from_directory_to_root.json"));
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 1"))));
+ "drive/root/Directory 1"))));
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 1/Sub Directory Folder"))));
+ "drive/root/Directory 1/Sub Directory Folder"))));
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 1/Sub Directory Folder/Sub Sub Directory Folder"))));
+ "drive/root/Directory 1/Sub Directory Folder/"
+ "Sub Sub Directory Folder"))));
EXPECT_FALSE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 1/SubDirectory File 1.txt"))));
+ "drive/root/Directory 1/SubDirectory File 1.txt"))));
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/SubDirectory File 1.txt"))));
+ "drive/root/SubDirectory File 1.txt"))));
}
TEST_F(DriveFileSystemTest, ChangeFeed_FileRenamedInDirectory) {
ASSERT_TRUE(LoadRootFeedDocument());
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 1"))));
+ "drive/root/Directory 1"))));
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 1/SubDirectory File 1.txt"))));
+ "drive/root/Directory 1/SubDirectory File 1.txt"))));
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root"))))).Times(1);
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive/Directory 1"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root/Directory 1")))))
+ .Times(1);
ASSERT_TRUE(LoadChangeFeed(
"chromeos/gdata/delta_file_renamed_in_directory.json"));
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 1"))));
+ "drive/root/Directory 1"))));
EXPECT_FALSE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 1/SubDirectory File 1.txt"))));
+ "drive/root/Directory 1/SubDirectory File 1.txt"))));
EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL(
- "drive/Directory 1/New SubDirectory File 1.txt"))));
+ "drive/root/Directory 1/New SubDirectory File 1.txt"))));
}
TEST_F(DriveFileSystemTest, CachedFeedLoading) {
@@ -899,14 +928,16 @@ TEST_F(DriveFileSystemTest, CachedFeedLoading) {
// Tests that cached data can be loaded even if the server is not reachable.
fake_drive_service_->set_offline(true);
- EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL("drive/File1"))));
- EXPECT_TRUE(EntryExists(base::FilePath(FILE_PATH_LITERAL("drive/Dir1"))));
- EXPECT_TRUE(
- EntryExists(base::FilePath(FILE_PATH_LITERAL("drive/Dir1/File2"))));
+ EXPECT_TRUE(EntryExists(base::FilePath(
+ FILE_PATH_LITERAL("drive/root/File1"))));
+ EXPECT_TRUE(EntryExists(base::FilePath(
+ FILE_PATH_LITERAL("drive/root/Dir1"))));
EXPECT_TRUE(
- EntryExists(base::FilePath(FILE_PATH_LITERAL("drive/Dir1/SubDir2"))));
+ EntryExists(base::FilePath(FILE_PATH_LITERAL("drive/root/Dir1/File2"))));
+ EXPECT_TRUE(EntryExists(base::FilePath(
+ FILE_PATH_LITERAL("drive/root/Dir1/SubDir2"))));
EXPECT_TRUE(EntryExists(
- base::FilePath(FILE_PATH_LITERAL("drive/Dir1/SubDir2/File3"))));
+ base::FilePath(FILE_PATH_LITERAL("drive/root/Dir1/SubDir2/File3"))));
}
TEST_F(DriveFileSystemTest, CachedFeedLoadingThenServerFeedLoading) {
@@ -963,7 +994,7 @@ TEST_F(DriveFileSystemTest, TransferFileFromLocalToRemote_RegularFile) {
// We'll add a file to the Drive root directory.
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root"))))).Times(1);
// Prepare a local file.
base::ScopedTempDir temp_dir;
@@ -975,7 +1006,7 @@ TEST_F(DriveFileSystemTest, TransferFileFromLocalToRemote_RegularFile) {
// Confirm that the remote file does not exist.
const base::FilePath remote_dest_file_path(
- FILE_PATH_LITERAL("drive/remote.txt"));
+ FILE_PATH_LITERAL("drive/root/remote.txt"));
EXPECT_FALSE(EntryExists(remote_dest_file_path));
// Transfer the local file to Drive.
@@ -1011,14 +1042,15 @@ TEST_F(DriveFileSystemTest, TransferFileFromLocalToRemote_HostedDocument) {
// Confirm that the remote file does not exist.
const base::FilePath remote_dest_file_path(
- FILE_PATH_LITERAL("drive/Directory 1/Document 1.gdoc"));
+ FILE_PATH_LITERAL("drive/root/Directory 1/Document 1.gdoc"));
EXPECT_FALSE(EntryExists(remote_dest_file_path));
// We'll add a file to the Drive root and then move to "Directory 1".
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root"))))).Times(1);
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive/Directory 1"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root/Directory 1")))))
+ .Times(1);
// We'll copy a hosted document using CopyHostedDocument.
// ".gdoc" suffix should be stripped when copying.
@@ -1049,14 +1081,15 @@ TEST_F(DriveFileSystemTest, TransferFileFromRemoteToLocal_RegularFile) {
// The transfered file is cached and the change of "offline available"
// attribute is notified.
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root"))))).Times(1);
base::ScopedTempDir temp_dir;
ASSERT_TRUE(temp_dir.CreateUniqueTempDir());
base::FilePath local_dest_file_path =
temp_dir.path().AppendASCII("local_copy.txt");
- base::FilePath remote_src_file_path(FILE_PATH_LITERAL("drive/File 1.txt"));
+ base::FilePath remote_src_file_path(
+ FILE_PATH_LITERAL("drive/root/File 1.txt"));
scoped_ptr<DriveEntryProto> file = GetEntryInfoByPathSync(
remote_src_file_path);
const int64 file_size = file->file_info().size();
@@ -1102,7 +1135,7 @@ TEST_F(DriveFileSystemTest, TransferFileFromRemoteToLocal_HostedDocument) {
base::FilePath local_dest_file_path =
temp_dir.path().AppendASCII("local_copy.txt");
base::FilePath remote_src_file_path(
- FILE_PATH_LITERAL("drive/Document 1.gdoc"));
+ FILE_PATH_LITERAL("drive/root/Document 1.gdoc"));
DriveFileError error = DRIVE_FILE_ERROR_FAILED;
file_system_->TransferFileFromRemoteToLocal(
remote_src_file_path,
@@ -1119,8 +1152,8 @@ TEST_F(DriveFileSystemTest, TransferFileFromRemoteToLocal_HostedDocument) {
}
TEST_F(DriveFileSystemTest, CopyNotExistingFile) {
- base::FilePath src_file_path(FILE_PATH_LITERAL("drive/Dummy file.txt"));
- base::FilePath dest_file_path(FILE_PATH_LITERAL("drive/Test.log"));
+ base::FilePath src_file_path(FILE_PATH_LITERAL("drive/root/Dummy file.txt"));
+ base::FilePath dest_file_path(FILE_PATH_LITERAL("drive/root/Test.log"));
ASSERT_TRUE(LoadRootFeedDocument());
@@ -1139,9 +1172,9 @@ TEST_F(DriveFileSystemTest, CopyNotExistingFile) {
}
TEST_F(DriveFileSystemTest, CopyFileToNonExistingDirectory) {
- base::FilePath src_file_path(FILE_PATH_LITERAL("drive/File 1.txt"));
- base::FilePath dest_parent_path(FILE_PATH_LITERAL("drive/Dummy"));
- base::FilePath dest_file_path(FILE_PATH_LITERAL("drive/Dummy/Test.log"));
+ base::FilePath src_file_path(FILE_PATH_LITERAL("drive/root/File 1.txt"));
+ base::FilePath dest_parent_path(FILE_PATH_LITERAL("drive/root/Dummy"));
+ base::FilePath dest_file_path(FILE_PATH_LITERAL("drive/root/Dummy/Test.log"));
ASSERT_TRUE(LoadRootFeedDocument());
@@ -1171,11 +1204,11 @@ TEST_F(DriveFileSystemTest, CopyFileToNonExistingDirectory) {
// Test the case where the parent of |dest_file_path| is an existing file,
// not a directory.
TEST_F(DriveFileSystemTest, CopyFileToInvalidPath) {
- base::FilePath src_file_path(FILE_PATH_LITERAL("drive/Document 1.gdoc"));
+ base::FilePath src_file_path(FILE_PATH_LITERAL("drive/root/Document 1.gdoc"));
base::FilePath dest_parent_path(
- FILE_PATH_LITERAL("drive/Duplicate Name.txt"));
+ FILE_PATH_LITERAL("drive/root/Duplicate Name.txt"));
base::FilePath dest_file_path(FILE_PATH_LITERAL(
- "drive/Duplicate Name.txt/Document 1.gdoc"));
+ "drive/root/Duplicate Name.txt/Document 1.gdoc"));
ASSERT_TRUE(LoadRootFeedDocument());
@@ -1209,10 +1242,11 @@ TEST_F(DriveFileSystemTest, CopyFileToInvalidPath) {
TEST_F(DriveFileSystemTest, RenameFile) {
const base::FilePath src_file_path(
- FILE_PATH_LITERAL("drive/Directory 1/SubDirectory File 1.txt"));
- const base::FilePath src_parent_path(FILE_PATH_LITERAL("drive/Directory 1"));
+ FILE_PATH_LITERAL("drive/root/Directory 1/SubDirectory File 1.txt"));
+ const base::FilePath src_parent_path(
+ FILE_PATH_LITERAL("drive/root/Directory 1"));
const base::FilePath dest_file_path(
- FILE_PATH_LITERAL("drive/Directory 1/Test.log"));
+ FILE_PATH_LITERAL("drive/root/Directory 1/Test.log"));
ASSERT_TRUE(LoadRootFeedDocument());
@@ -1224,7 +1258,8 @@ TEST_F(DriveFileSystemTest, RenameFile) {
src_entry_proto->resource_id();
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive/Directory 1"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root/Directory 1")))))
+ .Times(1);
DriveFileError error = DRIVE_FILE_ERROR_FAILED;
file_system_->Move(
@@ -1240,10 +1275,10 @@ TEST_F(DriveFileSystemTest, RenameFile) {
}
TEST_F(DriveFileSystemTest, MoveFileFromRootToSubDirectory) {
- base::FilePath src_file_path(FILE_PATH_LITERAL("drive/File 1.txt"));
- base::FilePath dest_parent_path(FILE_PATH_LITERAL("drive/Directory 1"));
+ base::FilePath src_file_path(FILE_PATH_LITERAL("drive/root/File 1.txt"));
+ base::FilePath dest_parent_path(FILE_PATH_LITERAL("drive/root/Directory 1"));
base::FilePath dest_file_path(
- FILE_PATH_LITERAL("drive/Directory 1/Test.log"));
+ FILE_PATH_LITERAL("drive/root/Directory 1/Test.log"));
ASSERT_TRUE(LoadRootFeedDocument());
@@ -1264,9 +1299,10 @@ TEST_F(DriveFileSystemTest, MoveFileFromRootToSubDirectory) {
// Expect notification for both source and destination directories.
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root"))))).Times(1);
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive/Directory 1"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root/Directory 1")))))
+ .Times(1);
DriveFileError error = DRIVE_FILE_ERROR_FAILED;
file_system_->Move(
@@ -1282,10 +1318,10 @@ TEST_F(DriveFileSystemTest, MoveFileFromRootToSubDirectory) {
}
TEST_F(DriveFileSystemTest, MoveFileFromSubDirectoryToRoot) {
- base::FilePath src_parent_path(FILE_PATH_LITERAL("drive/Directory 1"));
+ base::FilePath src_parent_path(FILE_PATH_LITERAL("drive/root/Directory 1"));
base::FilePath src_file_path(
- FILE_PATH_LITERAL("drive/Directory 1/SubDirectory File 1.txt"));
- base::FilePath dest_file_path(FILE_PATH_LITERAL("drive/Test.log"));
+ FILE_PATH_LITERAL("drive/root/Directory 1/SubDirectory File 1.txt"));
+ base::FilePath dest_file_path(FILE_PATH_LITERAL("drive/root/Test.log"));
ASSERT_TRUE(LoadRootFeedDocument());
@@ -1306,9 +1342,10 @@ TEST_F(DriveFileSystemTest, MoveFileFromSubDirectoryToRoot) {
// Expect notification for both source and destination directories.
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root"))))).Times(1);
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive/Directory 1"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root/Directory 1")))))
+ .Times(1);
DriveFileError error = DRIVE_FILE_ERROR_FAILED;
file_system_->Move(
@@ -1324,18 +1361,18 @@ TEST_F(DriveFileSystemTest, MoveFileFromSubDirectoryToRoot) {
}
TEST_F(DriveFileSystemTest, MoveFileBetweenSubDirectories) {
- base::FilePath src_parent_path(FILE_PATH_LITERAL("drive/Directory 1"));
+ base::FilePath src_parent_path(FILE_PATH_LITERAL("drive/root/Directory 1"));
base::FilePath src_file_path(
- FILE_PATH_LITERAL("drive/Directory 1/SubDirectory File 1.txt"));
- base::FilePath dest_parent_path(FILE_PATH_LITERAL("drive/New Folder 1"));
+ FILE_PATH_LITERAL("drive/root/Directory 1/SubDirectory File 1.txt"));
+ base::FilePath dest_parent_path(FILE_PATH_LITERAL("drive/root/New Folder 1"));
base::FilePath dest_file_path(
- FILE_PATH_LITERAL("drive/New Folder 1/Test.log"));
- base::FilePath interim_file_path(FILE_PATH_LITERAL("drive/Test.log"));
+ FILE_PATH_LITERAL("drive/root/New Folder 1/Test.log"));
+ base::FilePath interim_file_path(FILE_PATH_LITERAL("drive/root/Test.log"));
ASSERT_TRUE(LoadRootFeedDocument());
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root"))))).Times(1);
AddDirectoryFromFile(dest_parent_path,
"chromeos/gdata/directory_entry_atom.json");
@@ -1366,9 +1403,11 @@ TEST_F(DriveFileSystemTest, MoveFileBetweenSubDirectories) {
// Expect notification for both source and destination directories.
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive/Directory 1"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root/Directory 1")))))
+ .Times(1);
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive/New Folder 1"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root/New Folder 1")))))
+ .Times(1);
DriveFileError error = DRIVE_FILE_ERROR_FAILED;
file_system_->Move(
@@ -1387,8 +1426,8 @@ TEST_F(DriveFileSystemTest, MoveFileBetweenSubDirectories) {
}
TEST_F(DriveFileSystemTest, MoveNotExistingFile) {
- base::FilePath src_file_path(FILE_PATH_LITERAL("drive/Dummy file.txt"));
- base::FilePath dest_file_path(FILE_PATH_LITERAL("drive/Test.log"));
+ base::FilePath src_file_path(FILE_PATH_LITERAL("drive/root/Dummy file.txt"));
+ base::FilePath dest_file_path(FILE_PATH_LITERAL("drive/root/Test.log"));
ASSERT_TRUE(LoadRootFeedDocument());
@@ -1407,9 +1446,9 @@ TEST_F(DriveFileSystemTest, MoveNotExistingFile) {
}
TEST_F(DriveFileSystemTest, MoveFileToNonExistingDirectory) {
- base::FilePath src_file_path(FILE_PATH_LITERAL("drive/File 1.txt"));
- base::FilePath dest_parent_path(FILE_PATH_LITERAL("drive/Dummy"));
- base::FilePath dest_file_path(FILE_PATH_LITERAL("drive/Dummy/Test.log"));
+ base::FilePath src_file_path(FILE_PATH_LITERAL("drive/root/File 1.txt"));
+ base::FilePath dest_parent_path(FILE_PATH_LITERAL("drive/root/Dummy"));
+ base::FilePath dest_file_path(FILE_PATH_LITERAL("drive/root/Dummy/Test.log"));
ASSERT_TRUE(LoadRootFeedDocument());
@@ -1438,11 +1477,11 @@ TEST_F(DriveFileSystemTest, MoveFileToNonExistingDirectory) {
// Test the case where the parent of |dest_file_path| is a existing file,
// not a directory.
TEST_F(DriveFileSystemTest, MoveFileToInvalidPath) {
- base::FilePath src_file_path(FILE_PATH_LITERAL("drive/File 1.txt"));
+ base::FilePath src_file_path(FILE_PATH_LITERAL("drive/root/File 1.txt"));
base::FilePath dest_parent_path(
- FILE_PATH_LITERAL("drive/Duplicate Name.txt"));
+ FILE_PATH_LITERAL("drive/root/Duplicate Name.txt"));
base::FilePath dest_file_path(FILE_PATH_LITERAL(
- "drive/Duplicate Name.txt/Test.log"));
+ "drive/root/Duplicate Name.txt/Test.log"));
ASSERT_TRUE(LoadRootFeedDocument());
@@ -1475,11 +1514,12 @@ TEST_F(DriveFileSystemTest, MoveFileToInvalidPath) {
TEST_F(DriveFileSystemTest, RemoveEntries) {
ASSERT_TRUE(LoadRootFeedDocument());
- base::FilePath nonexisting_file(FILE_PATH_LITERAL("drive/Dummy file.txt"));
- base::FilePath file_in_root(FILE_PATH_LITERAL("drive/File 1.txt"));
- base::FilePath dir_in_root(FILE_PATH_LITERAL("drive/Directory 1"));
+ base::FilePath nonexisting_file(
+ FILE_PATH_LITERAL("drive/root/Dummy file.txt"));
+ base::FilePath file_in_root(FILE_PATH_LITERAL("drive/root/File 1.txt"));
+ base::FilePath dir_in_root(FILE_PATH_LITERAL("drive/root/Directory 1"));
base::FilePath file_in_subdir(
- FILE_PATH_LITERAL("drive/Directory 1/SubDirectory File 1.txt"));
+ FILE_PATH_LITERAL("drive/root/Directory 1/SubDirectory File 1.txt"));
ASSERT_TRUE(EntryExists(file_in_root));
scoped_ptr<DriveEntryProto> file_in_root_proto = GetEntryInfoByPathSync(
@@ -1499,7 +1539,7 @@ TEST_F(DriveFileSystemTest, RemoveEntries) {
// Once for file in root and once for file...
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive"))))).Times(2);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root"))))).Times(2);
// Remove first file in root.
EXPECT_TRUE(RemoveEntry(file_in_root));
@@ -1520,7 +1560,7 @@ TEST_F(DriveFileSystemTest, RemoveEntries) {
EXPECT_FALSE(RemoveEntry(nonexisting_file));
// Try removing root file element.
- EXPECT_FALSE(RemoveEntry(base::FilePath(FILE_PATH_LITERAL("drive"))));
+ EXPECT_FALSE(RemoveEntry(base::FilePath(FILE_PATH_LITERAL("drive/root"))));
// Need this to ensure OnDirectoryChanged() is run.
google_apis::test_util::RunBlockingPoolTask();
@@ -1530,20 +1570,21 @@ TEST_F(DriveFileSystemTest, CreateDirectory) {
ASSERT_TRUE(LoadRootFeedDocument());
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root"))))).Times(1);
// Create directory in root.
- base::FilePath dir_path(FILE_PATH_LITERAL("drive/New Folder 1"));
+ base::FilePath dir_path(FILE_PATH_LITERAL("drive/root/New Folder 1"));
EXPECT_FALSE(EntryExists(dir_path));
AddDirectoryFromFile(dir_path, "chromeos/gdata/directory_entry_atom.json");
EXPECT_TRUE(EntryExists(dir_path));
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive/New Folder 1"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root/New Folder 1")))))
+ .Times(1);
// Create directory in a sub directory.
base::FilePath subdir_path(
- FILE_PATH_LITERAL("drive/New Folder 1/New Folder 2"));
+ FILE_PATH_LITERAL("drive/root/New Folder 1/New Folder 2"));
EXPECT_FALSE(EntryExists(subdir_path));
AddDirectoryFromFile(subdir_path,
"chromeos/gdata/directory_entry_atom2.json");
@@ -1554,11 +1595,11 @@ TEST_F(DriveFileSystemTest, CreateDirectory) {
TEST_F(DriveFileSystemTest, CreateDirectoryWithService) {
ASSERT_TRUE(LoadRootFeedDocument());
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root"))))).Times(1);
DriveFileError error = DRIVE_FILE_ERROR_FAILED;
file_system_->CreateDirectory(
- base::FilePath(FILE_PATH_LITERAL("drive/Sample Directory Title")),
+ base::FilePath(FILE_PATH_LITERAL("drive/root/Sample Directory Title")),
false, // is_exclusive
true, // is_recursive
google_apis::test_util::CreateCopyResultCallback(&error));
@@ -1573,9 +1614,9 @@ TEST_F(DriveFileSystemTest, GetFileByPath_FromGData_EnoughSpace) {
// The transfered file is cached and the change of "offline available"
// attribute is notified.
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root"))))).Times(1);
- base::FilePath file_in_root(FILE_PATH_LITERAL("drive/File 1.txt"));
+ base::FilePath file_in_root(FILE_PATH_LITERAL("drive/root/File 1.txt"));
scoped_ptr<DriveEntryProto> entry_proto(GetEntryInfoByPathSync(file_in_root));
const int64 file_size = entry_proto->file_info().size();
@@ -1599,7 +1640,7 @@ TEST_F(DriveFileSystemTest, GetFileByPath_FromGData_EnoughSpace) {
TEST_F(DriveFileSystemTest, GetFileByPath_FromGData_NoSpaceAtAll) {
ASSERT_TRUE(LoadRootFeedDocument());
- base::FilePath file_in_root(FILE_PATH_LITERAL("drive/File 1.txt"));
+ base::FilePath file_in_root(FILE_PATH_LITERAL("drive/root/File 1.txt"));
scoped_ptr<DriveEntryProto> entry_proto(GetEntryInfoByPathSync(file_in_root));
// Pretend we have no space at all.
@@ -1623,9 +1664,9 @@ TEST_F(DriveFileSystemTest, GetFileByPath_FromGData_NoEnoughSpaceButCanFreeUp) {
// The transfered file is cached and the change of "offline available"
// attribute is notified.
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root"))))).Times(1);
- base::FilePath file_in_root(FILE_PATH_LITERAL("drive/File 1.txt"));
+ base::FilePath file_in_root(FILE_PATH_LITERAL("drive/root/File 1.txt"));
scoped_ptr<DriveEntryProto> entry_proto(GetEntryInfoByPathSync(file_in_root));
const int64 file_size = entry_proto->file_info().size();
@@ -1675,7 +1716,7 @@ TEST_F(DriveFileSystemTest, GetFileByPath_FromGData_NoEnoughSpaceButCanFreeUp) {
TEST_F(DriveFileSystemTest, GetFileByPath_FromGData_EnoughSpaceButBecomeFull) {
ASSERT_TRUE(LoadRootFeedDocument());
- base::FilePath file_in_root(FILE_PATH_LITERAL("drive/File 1.txt"));
+ base::FilePath file_in_root(FILE_PATH_LITERAL("drive/root/File 1.txt"));
scoped_ptr<DriveEntryProto> entry_proto(GetEntryInfoByPathSync(file_in_root));
const int64 file_size = entry_proto->file_info().size();
@@ -1705,7 +1746,7 @@ TEST_F(DriveFileSystemTest, GetFileByPath_FromCache) {
ASSERT_TRUE(LoadRootFeedDocument());
- base::FilePath file_in_root(FILE_PATH_LITERAL("drive/File 1.txt"));
+ base::FilePath file_in_root(FILE_PATH_LITERAL("drive/root/File 1.txt"));
scoped_ptr<DriveEntryProto> entry_proto(GetEntryInfoByPathSync(file_in_root));
// Store something as cached version of this file.
@@ -1734,7 +1775,7 @@ TEST_F(DriveFileSystemTest, GetFileByPath_FromCache) {
TEST_F(DriveFileSystemTest, GetFileByPath_HostedDocument) {
ASSERT_TRUE(LoadRootFeedDocument());
- base::FilePath file_in_root(FILE_PATH_LITERAL("drive/Document 1.gdoc"));
+ base::FilePath file_in_root(FILE_PATH_LITERAL("drive/root/Document 1.gdoc"));
scoped_ptr<DriveEntryProto> src_entry_proto =
GetEntryInfoByPathSync(file_in_root);
ASSERT_TRUE(src_entry_proto.get());
@@ -1761,11 +1802,11 @@ TEST_F(DriveFileSystemTest, GetFileByResourceId) {
// The transfered file is cached and the change of "offline available"
// attribute is notified.
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root"))))).Times(1);
ASSERT_TRUE(LoadRootFeedDocument());
- base::FilePath file_in_root(FILE_PATH_LITERAL("drive/File 1.txt"));
+ base::FilePath file_in_root(FILE_PATH_LITERAL("drive/root/File 1.txt"));
scoped_ptr<DriveEntryProto> entry_proto(GetEntryInfoByPathSync(file_in_root));
DriveFileError error = DRIVE_FILE_OK;
@@ -1785,7 +1826,7 @@ TEST_F(DriveFileSystemTest, GetFileByResourceId) {
}
TEST_F(DriveFileSystemTest, CancelGetFile) {
- base::FilePath cancel_file_path(FILE_PATH_LITERAL("drive/File 1.txt"));
+ base::FilePath cancel_file_path(FILE_PATH_LITERAL("drive/root/File 1.txt"));
file_system_->CancelGetFile(cancel_file_path);
EXPECT_EQ(cancel_file_path, fake_drive_service_->last_cancelled_file());
}
@@ -1795,7 +1836,7 @@ TEST_F(DriveFileSystemTest, GetFileByResourceId_FromCache) {
ASSERT_TRUE(LoadRootFeedDocument());
- base::FilePath file_in_root(FILE_PATH_LITERAL("drive/File 1.txt"));
+ base::FilePath file_in_root(FILE_PATH_LITERAL("drive/root/File 1.txt"));
scoped_ptr<DriveEntryProto> entry_proto(GetEntryInfoByPathSync(file_in_root));
// Store something as cached version of this file.
@@ -1834,7 +1875,7 @@ TEST_F(DriveFileSystemTest, UpdateFileByResourceId_PersistentFile) {
ASSERT_TRUE(LoadRootFeedDocument());
// This is a file defined in root_feed.json.
- const base::FilePath kFilePath(FILE_PATH_LITERAL("drive/File 1.txt"));
+ const base::FilePath kFilePath(FILE_PATH_LITERAL("drive/root/File 1.txt"));
const std::string kResourceId("file:2_file_resource_id");
const std::string kMd5("3b4382ebefec6e743578c76bbd0575ce");
@@ -1878,7 +1919,7 @@ TEST_F(DriveFileSystemTest, UpdateFileByResourceId_PersistentFile) {
// Check the number of files in the root directory. We'll compare the
// number after updating a file.
scoped_ptr<DriveEntryProtoVector> root_directory_entries(
- ReadDirectoryByPathSync(base::FilePath::FromUTF8Unsafe("drive")));
+ ReadDirectoryByPathSync(base::FilePath::FromUTF8Unsafe("drive/root")));
ASSERT_TRUE(root_directory_entries.get());
const int num_files_in_root = CountFiles(*root_directory_entries);
@@ -1901,7 +1942,8 @@ TEST_F(DriveFileSystemTest, UpdateFileByResourceId_NonexistentFile) {
ASSERT_TRUE(LoadRootFeedDocument());
// This is nonexistent in root_feed.json.
- const base::FilePath kFilePath(FILE_PATH_LITERAL("drive/Nonexistent.txt"));
+ const base::FilePath kFilePath(
+ FILE_PATH_LITERAL("drive/root/Nonexistent.txt"));
const std::string kResourceId("file:nonexistent_resource_id");
const std::string kMd5("nonexistent_md5");
@@ -1920,12 +1962,12 @@ TEST_F(DriveFileSystemTest, ContentSearch) {
ASSERT_TRUE(LoadRootFeedDocument());
const SearchResultPair kExpectedResults[] = {
- { "drive/Directory 1/Sub Directory Folder/Sub Sub Directory Folder",
+ { "drive/root/Directory 1/Sub Directory Folder/Sub Sub Directory Folder",
true },
- { "drive/Directory 1/Sub Directory Folder", true },
- { "drive/Directory 1/SubDirectory File 1.txt", false },
- { "drive/Directory 1", true },
- { "drive/Directory 2", true },
+ { "drive/root/Directory 1/Sub Directory Folder", true },
+ { "drive/root/Directory 1/SubDirectory File 1.txt", false },
+ { "drive/root/Directory 1", true },
+ { "drive/root/Directory 2", true },
};
SearchCallback callback = base::Bind(&DriveSearchCallback,
@@ -1955,7 +1997,7 @@ TEST_F(DriveFileSystemTest, ContentSearchWithNewEntry) {
// shouldn't be included in the search result even though it matches
// "Directory 1".
const SearchResultPair kExpectedResults[] = {
- { "drive/Directory 1", true }
+ { "drive/root/Directory 1", true }
};
// At the same time, unknown entry should trigger delta feed request.
@@ -2020,9 +2062,9 @@ TEST_F(DriveFileSystemTest, OpenAndCloseFile) {
// The transfered file is cached and the change of "offline available"
// attribute is notified.
EXPECT_CALL(*mock_directory_observer_, OnDirectoryChanged(
- Eq(base::FilePath(FILE_PATH_LITERAL("drive"))))).Times(1);
+ Eq(base::FilePath(FILE_PATH_LITERAL("drive/root"))))).Times(1);
- const base::FilePath kFileInRoot(FILE_PATH_LITERAL("drive/File 1.txt"));
+ const base::FilePath kFileInRoot(FILE_PATH_LITERAL("drive/root/File 1.txt"));
scoped_ptr<DriveEntryProto> entry_proto(GetEntryInfoByPathSync(kFileInRoot));
const int64 file_size = entry_proto->file_info().size();
const std::string& file_resource_id =
@@ -2037,7 +2079,7 @@ TEST_F(DriveFileSystemTest, OpenAndCloseFile) {
fake_free_disk_space_getter_->set_fake_free_disk_space(
file_size + kMinFreeSpace);
- // Open kFileInRoot ("drive/File 1.txt").
+ // Open kFileInRoot ("drive/root/File 1.txt").
DriveFileError error = DRIVE_FILE_ERROR_FAILED;
base::FilePath file_path;
file_system_->OpenFile(
@@ -2080,7 +2122,7 @@ TEST_F(DriveFileSystemTest, OpenAndCloseFile) {
EXPECT_EQ(DRIVE_FILE_OK, error);
EXPECT_EQ(cache_file_path, opened_file_path);
- // Close kFileInRoot ("drive/File 1.txt").
+ // Close kFileInRoot ("drive/root/File 1.txt").
file_system_->CloseFile(
kFileInRoot,
google_apis::test_util::CreateCopyResultCallback(&error));

Powered by Google App Engine
This is Rietveld 408576698