Renamed classes for consistency.

net/disk_cache/flash/log_store_unittest.cc

Index: net/disk_cache/flash/log_store_unittest.cc
diff --git a/net/disk_cache/flash/log_store_unittest.cc b/net/disk_cache/flash/log_store_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..e57eedc4a9eeaab13a2bfba0c282c5bdb2b0e8b1
--- /dev/null
+++ b/net/disk_cache/flash/log_store_unittest.cc
@@ -0,0 +1,114 @@
+// Copyright (c) 2012 The Chromium Authors. All rights reserved.

[rvargas (doing something else), 2012/12/11 20:40:42]

Looks like this file was not renamed correctly (will lose history... A vs A+).

[agayev, 2012/12/11 21:12:41]

Could not fix it.  I attempted once again and treated this file exactly like I
did the others, yet the outcome is A and not A+.  Looks like there is too much
change in it for git to recognize it as a rename.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "net/disk_cache/flash/flash_cache_test_base.h"
+#include "net/disk_cache/flash/format.h"
+#include "net/disk_cache/flash/log_store.h"
+#include "net/disk_cache/flash/segment.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace disk_cache {
+
+TEST_F(FlashCacheTest, LogStoreCreateEntry) {
+  const int32 kSize = 100;
+  const std::string buf(kSize, 0);
+
+  int32 id;
+  EXPECT_TRUE(log_store_->CreateEntry(kSize, &id));
+  EXPECT_TRUE(log_store_->WriteData(buf.data(), kSize/2));
+  EXPECT_TRUE(log_store_->WriteData(buf.data(), kSize/2));
+  log_store_->CloseEntry(id);
+}
+
+// Also tests reading from current segment.
+TEST_F(FlashCacheTest, LogStoreOpenEntry) {
+  const int32 kSize = 100;
+  const std::vector<char> expected(kSize, 'b');
+
+  int32 id;
+  EXPECT_TRUE(log_store_->CreateEntry(kSize, &id));
+  EXPECT_TRUE(log_store_->WriteData(&expected[0], kSize));
+  log_store_->CloseEntry(id);
+
+  EXPECT_TRUE(log_store_->OpenEntry(id));
+  std::vector<char> actual(kSize, 0);
+  EXPECT_TRUE(log_store_->ReadData(id, &actual[0], kSize, 0));
+  log_store_->CloseEntry(id);
+
+  EXPECT_EQ(expected, actual);
+}
+
+// Also tests that writing advances segments.
+TEST_F(FlashCacheTest, LogStoreReadFromClosedSegment) {
+  const int32 kSize = disk_cache::kFlashSegmentFreeSpace;
+  const std::vector<char> expected(kSize, 'a');
+
+  // First two entries go to segment 0.
+  int32 id1;
+  EXPECT_EQ(0, log_store_->write_index_);
+  EXPECT_TRUE(log_store_->CreateEntry(kSize/2, &id1));
+  EXPECT_TRUE(log_store_->WriteData(&expected[0], kSize/2));
+  log_store_->CloseEntry(id1);
+
+  int32 id2;
+  EXPECT_EQ(0, log_store_->write_index_);
+  EXPECT_TRUE(log_store_->CreateEntry(kSize/2, &id2));
+  EXPECT_TRUE(log_store_->WriteData(&expected[0], kSize/2));
+  log_store_->CloseEntry(id2);
+
+  // This entry goes to segment 1.
+  int32 id3;
+  EXPECT_TRUE(log_store_->CreateEntry(kSize, &id3));
+  EXPECT_EQ(1, log_store_->write_index_);
+  EXPECT_TRUE(log_store_->WriteData(&expected[0], kSize));
+  log_store_->CloseEntry(id3);
+
+  // We read from segment 0.
+  EXPECT_TRUE(log_store_->OpenEntry(id1));
+  std::vector<char> actual(kSize, 0);
+  EXPECT_TRUE(log_store_->ReadData(id1, &actual[0], kSize, id1));
+  log_store_->CloseEntry(id1);
+
+  EXPECT_EQ(expected, actual);
+}
+
+TEST_F(FlashCacheTest, LogStoreReadFromCurrentAfterClose) {
+  const int32 kSize = disk_cache::kFlashSegmentFreeSpace;
+  const std::vector<char> expected(kSize, 'a');
+
+  int32 id1;
+  EXPECT_EQ(0, log_store_->write_index_);
+  EXPECT_TRUE(log_store_->CreateEntry(kSize/2, &id1));
+  EXPECT_TRUE(log_store_->WriteData(&expected[0], kSize/2));
+  log_store_->CloseEntry(id1);
+
+  // Create a reference to above entry.
+  EXPECT_TRUE(log_store_->OpenEntry(id1));
+
+  // This entry fills the first segment.
+  int32 id2;
+  EXPECT_EQ(0, log_store_->write_index_);
+  EXPECT_TRUE(log_store_->CreateEntry(kSize/2, &id2));
+  EXPECT_TRUE(log_store_->WriteData(&expected[0], kSize/2));
+  log_store_->CloseEntry(id2);
+
+  // Creating this entry forces closing of the first segment.
+  int32 id3;
+  EXPECT_TRUE(log_store_->CreateEntry(kSize, &id3));
+  EXPECT_EQ(1, log_store_->write_index_);
+  EXPECT_TRUE(log_store_->WriteData(&expected[0], kSize));
+  log_store_->CloseEntry(id3);
+
+  // Now attempt to read from the closed segment.
+  std::vector<char> actual(kSize, 0);
+  EXPECT_TRUE(log_store_->ReadData(id1, &actual[0], kSize, id1));
+  log_store_->CloseEntry(id1);
+
+  EXPECT_EQ(expected, actual);
+}
+
+// TODO(agayev): Add a test that confirms that in-use segment is not selected as
+// the next write segment.
+
+}  // namespace disk_cache