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

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

Issue 55713002: drive: Use base_name recovered from trashed DB when DB corruption happens (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Use title Created 7 years, 1 month 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/file_cache_unittest.cc
diff --git a/chrome/browser/chromeos/drive/file_cache_unittest.cc b/chrome/browser/chromeos/drive/file_cache_unittest.cc
index 493fe0f2cf6e66507b34d1dd0c3a7c918fccc456..23a6149a396a05a83ae5e9ecf6b966f83e6eadd0 100644
--- a/chrome/browser/chromeos/drive/file_cache_unittest.cc
+++ b/chrome/browser/chromeos/drive/file_cache_unittest.cc
@@ -768,23 +768,37 @@ TEST_F(FileCacheTest, RecoverFilesFromCacheDirectory) {
ASSERT_TRUE(base::CopyFile(src_path, file_directory.AppendASCII("id_bar")));
ASSERT_TRUE(base::CopyFile(src_path, file_directory.AppendASCII("id_baz")));
- // Insert a dirty entry with "id_baz" to |recovered_cache_entries|.
+ // Insert a dirty entry with "id_baz" to |recovered_cache_info|.
// This should not prevent the file from being recovered.
- std::map<std::string, FileCacheEntry> recovered_cache_entries;
- recovered_cache_entries["id_baz"].set_is_dirty(true);
+ ResourceMetadataStorage::RecoveredCacheInfoMap recovered_cache_info;
+ recovered_cache_info["id_baz"].is_dirty = true;
+ recovered_cache_info["id_baz"].title = "baz.png";
// Recover files.
const base::FilePath dest_directory = temp_dir_.path().AppendASCII("dest");
EXPECT_TRUE(cache_->RecoverFilesFromCacheDirectory(dest_directory,
- recovered_cache_entries));
+ recovered_cache_info));
// Only two files should be recovered.
EXPECT_TRUE(base::PathExists(dest_directory));
- EXPECT_TRUE(base::ContentsEqual(src_path,
- dest_directory.Append("image00000001.png")));
- EXPECT_TRUE(base::ContentsEqual(src_path,
- dest_directory.Append("image00000002.png")));
- EXPECT_FALSE(base::PathExists(dest_directory.Append("image00000003.png")));
+ // base::FileEnumerator does not guarantee the order.
+ if (base::PathExists(dest_directory.AppendASCII("baz00000001.png"))) {
+ EXPECT_TRUE(base::ContentsEqual(
+ src_path,
+ dest_directory.AppendASCII("baz00000001.png")));
+ EXPECT_TRUE(base::ContentsEqual(
+ src_path,
+ dest_directory.AppendASCII("image00000002.png")));
+ } else {
+ EXPECT_TRUE(base::ContentsEqual(
+ src_path,
+ dest_directory.AppendASCII("image00000001.png")));
+ EXPECT_TRUE(base::ContentsEqual(
+ src_path,
+ dest_directory.AppendASCII("baz00000002.png")));
+ }
+ EXPECT_FALSE(base::PathExists(
+ dest_directory.AppendASCII("image00000003.png")));
}
TEST_F(FileCacheTest, Iterator) {
« no previous file with comments | « chrome/browser/chromeos/drive/file_cache.cc ('k') | chrome/browser/chromeos/drive/resource_metadata_storage.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698