[Offline pages] Hooking up Archive Manager to Offline Page Model

components/offline_pages/archive_manager_unittest.cc

 // Copyright 2015 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 "components/offline_pages/archive_manager.h"
 
 #include <memory>
+#include <set>
 
 #include "base/bind.h"
 #include "base/files/file_path.h"
 #include "base/files/file_util.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/test/test_simple_task_runner.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace offline_pages {
@@ skipping 12 matching lines @@
 class ArchiveManagerTest : public testing::Test {
  public:
   ArchiveManagerTest();
   void SetUp() override;
 
   void PumpLoop();
   void ResetResults();
 
   void ResetManager(const base::FilePath& file_path);
   void Callback(bool result);
+  void GetAllArchivesCallback(const std::set<base::FilePath>& archive_paths);
 
   ArchiveManager* manager() { return manager_.get(); }
   const base::FilePath& temp_path() const { return temp_dir_.path(); }
   CallbackStatus callback_status() const { return callback_status_; }
+  const std::set<base::FilePath>& last_archvie_paths() const {
+    return last_archvie_paths_;
+  }
 
  private:
   scoped_refptr<base::TestSimpleTaskRunner> task_runner_;
   base::ThreadTaskRunnerHandle task_runner_handle_;
   base::ScopedTempDir temp_dir_;
 
   std::unique_ptr<ArchiveManager> manager_;
   CallbackStatus callback_status_;
+  std::set<base::FilePath> last_archvie_paths_;
 };
 
 ArchiveManagerTest::ArchiveManagerTest()
     : task_runner_(new base::TestSimpleTaskRunner),
       task_runner_handle_(task_runner_),
       callback_status_(CallbackStatus::NOT_CALLED) {}
 
 void ArchiveManagerTest::SetUp() {
   ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
   ResetManager(temp_dir_.path());
 }
 
 void ArchiveManagerTest::PumpLoop() {
   task_runner_->RunUntilIdle();
 }
 
 void ArchiveManagerTest::ResetResults() {
   callback_status_ = CallbackStatus::NOT_CALLED;
+  last_archvie_paths_.clear();
 }
 
 void ArchiveManagerTest::ResetManager(const base::FilePath& file_path) {
   manager_.reset(
       new ArchiveManager(file_path, base::ThreadTaskRunnerHandle::Get()));
 }
 
 void ArchiveManagerTest::Callback(bool result) {
   callback_status_ =
       result ? CallbackStatus::CALLED_TRUE : CallbackStatus::CALLED_FALSE;
 }
 
+void ArchiveManagerTest::GetAllArchivesCallback(
+    const std::set<base::FilePath>& archive_paths) {
+  last_archvie_paths_ = archive_paths;
+}
+
 TEST_F(ArchiveManagerTest, EnsureArchivesDirCreated) {
   base::FilePath archive_dir =
       temp_path().Append(FILE_PATH_LITERAL("test_path"));
   ResetManager(archive_dir);
   EXPECT_FALSE(base::PathExists(archive_dir));
 
   // Ensure archives dir exists, when it doesn't.
   manager()->EnsureArchivesDirCreated(
       base::Bind(&ArchiveManagerTest::Callback, base::Unretained(this), true));
   PumpLoop();
@@ skipping 107 matching lines @@
   EXPECT_FALSE(base::PathExists(archive_path));
 
   manager()->DeleteArchive(
       archive_path,
       base::Bind(&ArchiveManagerTest::Callback, base::Unretained(this)));
   PumpLoop();
   EXPECT_EQ(CallbackStatus::CALLED_TRUE, callback_status());
   EXPECT_FALSE(base::PathExists(archive_path));
 }
 
+TEST_F(ArchiveManagerTest, GetAllArchives) {
+  base::FilePath archive_path_1;
+  EXPECT_TRUE(base::CreateTemporaryFileInDir(temp_path(), &archive_path_1));
+  base::FilePath archive_path_2;
+  EXPECT_TRUE(base::CreateTemporaryFileInDir(temp_path(), &archive_path_2));
+  base::FilePath archive_path_3;
+  EXPECT_TRUE(base::CreateTemporaryFileInDir(temp_path(), &archive_path_3));
+
+  manager()->GetAllArchives(
+      base::Bind(&ArchiveManagerTest::GetAllArchivesCallback,
+                 base::Unretained(this)));
+  PumpLoop();
+  EXPECT_EQ(3UL, last_archvie_paths().size());

[jianli, 2016/05/17 21:48:13]

nit: ASSERT_EQ

[fgorski, 2016/05/17 22:50:29]

Done.

+  EXPECT_EQ(1UL, last_archvie_paths().count(archive_path_1));
+  EXPECT_EQ(1UL, last_archvie_paths().count(archive_path_2));
+  EXPECT_EQ(1UL, last_archvie_paths().count(archive_path_3));
+}
+
 }  // namespace offline_pages