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

Unified Diff: net/disk_cache/simple/simple_index_unittest.cc

Issue 14263005: Refactor our SimpleIndex file format and serialization to use Pickle instead of the previously bugg… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Sync Created 7 years, 8 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: net/disk_cache/simple/simple_index_unittest.cc
diff --git a/net/disk_cache/simple/simple_index_unittest.cc b/net/disk_cache/simple/simple_index_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..d1159678c80e2ab85fa97d35899731d34b948f30
--- /dev/null
+++ b/net/disk_cache/simple/simple_index_unittest.cc
@@ -0,0 +1,94 @@
+// Copyright (c) 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 "base/hash.h"
+#include "base/logging.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/pickle.h"
+#include "base/sha1.h"
+#include "base/stringprintf.h"
+#include "base/time.h"
+#include "net/disk_cache/simple/simple_index.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace {
+const uint64 kTestHashKey = 4364515;
+const int64 kTestLastUsedTimeInternal = 12345;
+const base::Time kTestLastUsedTime =
+ base::Time::FromInternalValue(kTestLastUsedTimeInternal);
+const uint64 kTestEntrySize = 789;
+}
+
+using disk_cache::EntryMetadata;
+
+
+class EntryMetadataTest : public testing::Test {
+ public:
+ EntryMetadata NewEntryMetadataWithValues() {
+ return EntryMetadata(kTestHashKey,
+ kTestLastUsedTime,
+ kTestEntrySize);
+ }
+
+ void CheckEntryMetadataValues(const EntryMetadata& entry_metadata) {
+ EXPECT_EQ(kTestLastUsedTime, entry_metadata.GetLastUsedTime());
+ EXPECT_EQ(kTestEntrySize, entry_metadata.GetEntrySize());
+ }
+};
+
+TEST_F(EntryMetadataTest, Basics) {
+ EntryMetadata entry_metadata;
+ EXPECT_EQ(base::Time::FromInternalValue(0), entry_metadata.GetLastUsedTime());
+ EXPECT_EQ(size_t(0), entry_metadata.GetEntrySize());
+
+ entry_metadata = NewEntryMetadataWithValues();
+ CheckEntryMetadataValues(entry_metadata);
+ EXPECT_EQ(kTestHashKey, entry_metadata.GetHashKey());
+
+ const base::Time new_time = base::Time::FromInternalValue(5);
+ entry_metadata.SetLastUsedTime(new_time);
+ EXPECT_EQ(new_time, entry_metadata.GetLastUsedTime());
+}
+
+TEST_F(EntryMetadataTest, Serialize) {
+ EntryMetadata entry_metadata = NewEntryMetadataWithValues();
+
+ Pickle pickle;
+ entry_metadata.Serialize(&pickle);
+
+ PickleIterator it(pickle);
+ EntryMetadata new_entry_metadata;
+ new_entry_metadata.Deserialize(&it);
+ CheckEntryMetadataValues(new_entry_metadata);
+ EXPECT_EQ(kTestHashKey, new_entry_metadata.GetHashKey());
+}
+
+TEST_F(EntryMetadataTest, Merge) {
+ EntryMetadata entry_metadata_a = NewEntryMetadataWithValues();
+ // MergeWith assumes the hash_key of both entries is the same, so we
+ // initialize it to be that way.
+ base::Time dummy_time = base::Time::FromInternalValue(0);
+ EntryMetadata entry_metadata_b(entry_metadata_a.GetHashKey(), dummy_time, 0);
+ entry_metadata_b.MergeWith(entry_metadata_a);
+ CheckEntryMetadataValues(entry_metadata_b);
+
+ EntryMetadata entry_metadata_c(entry_metadata_a.GetHashKey(), dummy_time, 0);
+ entry_metadata_a.MergeWith(entry_metadata_c);
+ CheckEntryMetadataValues(entry_metadata_a);
+
+ EntryMetadata entry_metadata_d(entry_metadata_a.GetHashKey(), dummy_time, 0);
+ const base::Time new_time = base::Time::FromInternalValue(5);
+ entry_metadata_d.SetLastUsedTime(new_time);
+ entry_metadata_d.MergeWith(entry_metadata_a);
+ EXPECT_EQ(entry_metadata_a.GetEntrySize(), entry_metadata_d.GetEntrySize());
+ EXPECT_EQ(new_time, entry_metadata_d.GetLastUsedTime());
+
+ EntryMetadata entry_metadata_e(entry_metadata_a.GetHashKey(), dummy_time, 0);
+ const uint64 entry_size = 9999999;
+ entry_metadata_e.SetEntrySize(entry_size);
+ entry_metadata_e.MergeWith(entry_metadata_a);
+ EXPECT_EQ(entry_size, entry_metadata_e.GetEntrySize());
+ EXPECT_EQ(entry_metadata_a.GetLastUsedTime(),
+ entry_metadata_e.GetLastUsedTime());
+}

Powered by Google App Engine
This is Rietveld 408576698