Create ReadingListSuggestionsProvider

components/ntp_snippets/reading_list/reading_list_suggestions_provider_unittest.cc

Index: components/ntp_snippets/reading_list/reading_list_suggestions_provider_unittest.cc
diff --git a/components/ntp_snippets/reading_list/reading_list_suggestions_provider_unittest.cc b/components/ntp_snippets/reading_list/reading_list_suggestions_provider_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..23b0f8c267bc22c99016c708d5eb6ffabd408f5f
--- /dev/null
+++ b/components/ntp_snippets/reading_list/reading_list_suggestions_provider_unittest.cc
@@ -0,0 +1,50 @@
+// Copyright 2017 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/ntp_snippets/reading_list/reading_list_suggestions_provider.h"
+
+#include "base/memory/ptr_util.h"
+#include "base/test/simple_test_clock.h"
+#include "components/ntp_snippets/mock_content_suggestions_provider_observer.h"
+#include "components/reading_list/core/reading_list_model_impl.h"
+#include "testing/gmock/include/gmock/gmock.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace ntp_snippets {
+
+namespace {
+
+class ReadingListSuggestionsProviderTest : public ::testing::Test {
+ public:
+  ReadingListSuggestionsProviderTest() {
+    model_ = base::MakeUnique<ReadingListModelImpl>(

[Marc Treib, 2017/03/23 19:01:49]

optional: If this is created in the ctor anyway, it doesn't have to be a
pointer, it could just be an instance, created in the initializer list.

[gambard, 2017/03/24 09:08:51]

The clock will be used in future tests.

+        /*storage_layer=*/nullptr, /*pref_service=*/nullptr,
+        base::MakeUnique<base::SimpleTestClock>());
+    EXPECT_CALL(observer_,
+                OnCategoryStatusChanged(testing::_, Category(),
+                                        CategoryStatus::AVAILABLE_LOADING))
+        .RetiresOnSaturation();
+
+    provider_ = base::MakeUnique<ReadingListSuggestionsProvider>(&observer_,
+                                                                 model_.get());
+  }
+
+  Category Category() {

[Marc Treib, 2017/03/23 19:01:49]

nit: I find it confusing to overload the name between the type and the "getter".
Maybe ReadingListCategory(), or just GetCategory()?

[gambard, 2017/03/24 09:08:51]

Done.

+    return Category::FromKnownCategory(KnownCategories::READING_LIST);
+  }
+
+ protected:
+  std::unique_ptr<ReadingListModelImpl> model_;
+  testing::StrictMock<MockContentSuggestionsProviderObserver> observer_;
+  std::unique_ptr<ReadingListSuggestionsProvider> provider_;
+};
+
+TEST_F(ReadingListSuggestionsProviderTest, CategoryInfo) {
+  CategoryInfo categoryInfo = provider_->GetCategoryInfo(Category());
+  EXPECT_EQ(ContentSuggestionsAdditionalAction::VIEW_ALL,
+            categoryInfo.additional_action());
+}
+
+}  // namespace
+}  // namespace ntp_snippets