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

Unified Diff: components/drive/resource_metadata_storage_unittest.cc

Issue 2346873004: Added 'starred' to EntryProperty in FileManagerPrivateAPI. (Closed)
Patch Set: Added and modified comments. Created 4 years, 3 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
« no previous file with comments | « components/drive/resource_metadata_storage.cc ('k') | components/drive/service/drive_api_service.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/drive/resource_metadata_storage_unittest.cc
diff --git a/components/drive/resource_metadata_storage_unittest.cc b/components/drive/resource_metadata_storage_unittest.cc
index 5f4a0fa089cc59e84d4a830b86346bd8b858afec..e4d15acefee1f0b61f63a6359d0d7e8f2f096cf7 100644
--- a/components/drive/resource_metadata_storage_unittest.cc
+++ b/components/drive/resource_metadata_storage_unittest.cc
@@ -43,6 +43,21 @@ class ResourceMetadataStorageTest : public testing::Test {
EXPECT_EQ(FILE_ERROR_OK, storage_->PutHeader(header));
}
+ // Overwrites |storage_|'s starred_property_initialized.
+ void SetStarredPropertyInitialized(bool value) {
+ ResourceMetadataHeader header;
+ ASSERT_EQ(FILE_ERROR_OK, storage_->GetHeader(&header));
+ header.set_starred_property_initialized(value);
+ EXPECT_EQ(FILE_ERROR_OK, storage_->PutHeader(header));
+ }
+
+ // Returns |storage_|'s starred_property_initialized.
+ bool GetStarredPropertyInitialized() {
+ ResourceMetadataHeader header;
+ EXPECT_EQ(FILE_ERROR_OK, storage_->GetHeader(&header));
+ return header.starred_property_initialized();
+ }
+
bool CheckValidity() {
return storage_->CheckValidity();
}
@@ -634,5 +649,28 @@ TEST_F(ResourceMetadataStorageTest, CheckValidity) {
EXPECT_TRUE(CheckValidity());
}
+TEST_F(ResourceMetadataStorageTest, ChangeStarredPropertyInitialized) {
+ // Suppose 'Starred' property has not loaded.
+ bool starred_property_initialized = false;
+ SetStarredPropertyInitialized(starred_property_initialized);
+
+ const int64_t kLargestChangestamp = 1234567890;
+ EXPECT_EQ(FILE_ERROR_OK,
+ storage_->SetLargestChangestamp(kLargestChangestamp));
+
+ // Close DB and reopen.
+ storage_.reset(new ResourceMetadataStorage(
+ temp_dir_.GetPath(), base::ThreadTaskRunnerHandle::Get().get()));
+ ASSERT_TRUE(storage_->Initialize());
+
+ starred_property_initialized = GetStarredPropertyInitialized();
+ EXPECT_TRUE(starred_property_initialized);
+
+ int64_t largest_changestamp = 0;
+ EXPECT_EQ(FILE_ERROR_OK,
+ storage_->GetLargestChangestamp(&largest_changestamp));
+ EXPECT_EQ(0, largest_changestamp);
+}
+
} // namespace internal
} // namespace drive
« no previous file with comments | « components/drive/resource_metadata_storage.cc ('k') | components/drive/service/drive_api_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698