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

Unified Diff: chrome/browser/chromeos/drive/sync/entry_revert_performer_unittest.cc

Issue 102133008: drive: Call OnDirectoryChanged from EntryRevertPerformer (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix test Created 7 years 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/sync/entry_revert_performer_unittest.cc
diff --git a/chrome/browser/chromeos/drive/sync/entry_revert_performer_unittest.cc b/chrome/browser/chromeos/drive/sync/entry_revert_performer_unittest.cc
index d60e742091f41d2f902162715926c3cda1f79905..313339d9c9bb663d6242fc8322756a399eccb95d 100644
--- a/chrome/browser/chromeos/drive/sync/entry_revert_performer_unittest.cc
+++ b/chrome/browser/chromeos/drive/sync/entry_revert_performer_unittest.cc
@@ -20,6 +20,7 @@ class EntryRevertPerformerTest : public file_system::OperationTestBase {
virtual void SetUp() OVERRIDE {
OperationTestBase::SetUp();
performer_.reset(new EntryRevertPerformer(blocking_task_runner(),
+ observer(),
scheduler(),
metadata()));
}
@@ -63,6 +64,9 @@ TEST_F(EntryRevertPerformerTest, RevertEntry) {
GetLocalResourceEntryById(src_entry.local_id(), &result_entry));
EXPECT_EQ(src_entry.title(), result_entry.title());
EXPECT_EQ(ResourceEntry::CLEAN, result_entry.metadata_edit_state());
+
+ EXPECT_EQ(1U, observer()->get_changed_paths().size());
+ EXPECT_TRUE(observer()->get_changed_paths().count(path.DirName()));
}
TEST_F(EntryRevertPerformerTest, RevertEntry_NotFoundOnServer) {
@@ -97,6 +101,10 @@ TEST_F(EntryRevertPerformerTest, RevertEntry_NotFoundOnServer) {
// Verify the entry was deleted locally.
EXPECT_EQ(FILE_ERROR_NOT_FOUND, GetLocalResourceEntryById(local_id, &entry));
+
+ EXPECT_EQ(1U, observer()->get_changed_paths().size());
+ EXPECT_TRUE(observer()->get_changed_paths().count(
+ util::GetDriveMyDriveRootPath()));
}
TEST_F(EntryRevertPerformerTest, RevertEntry_DeletedOnServer) {
@@ -126,6 +134,9 @@ TEST_F(EntryRevertPerformerTest, RevertEntry_DeletedOnServer) {
// Verify the entry was deleted locally.
EXPECT_EQ(FILE_ERROR_NOT_FOUND,
GetLocalResourceEntryById(entry.local_id(), &entry));
+
+ EXPECT_EQ(1U, observer()->get_changed_paths().size());
+ EXPECT_TRUE(observer()->get_changed_paths().count(path.DirName()));
}
} // namespace internal

Powered by Google App Engine
This is Rietveld 408576698