Fetch snippets from ChromeReader and show them on the NTP

components/ntp_snippets/ntp_snippets_service_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 "base/macros.h"
+#include "base/message_loop/message_loop.h"
+#include "base/thread_task_runner_handle.h"
 #include "base/time/time.h"
 #include "components/ntp_snippets/ntp_snippet.h"
+#include "components/ntp_snippets/ntp_snippets_fetcher.h"
 #include "components/ntp_snippets/ntp_snippets_service.h"
+#include "components/signin/core/browser/account_tracker_service.h"
+#include "components/signin/core/browser/fake_profile_oauth2_token_service.h"
+#include "components/signin/core/browser/fake_signin_manager.h"
+#include "components/signin/core/browser/test_signin_client.h"
+#include "net/url_request/url_request_test_util.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
-namespace {
+namespace ntp_snippets {
 
-class SnippetObserver : public ntp_snippets::NTPSnippetsServiceObserver {
+class SnippetObserver : public NTPSnippetsServiceObserver {
  public:
   SnippetObserver() : loaded_(false), shutdown_(false) {}
   ~SnippetObserver() override {}
 
-  void NTPSnippetsServiceLoaded(
-      ntp_snippets::NTPSnippetsService* service) override {
+  void NTPSnippetsServiceLoaded(NTPSnippetsService* service) override {
     loaded_ = true;
   }
 
-  void NTPSnippetsServiceShutdown(
-      ntp_snippets::NTPSnippetsService* service) override {
+  void NTPSnippetsServiceShutdown(NTPSnippetsService* service) override {
     shutdown_ = true;
     loaded_ = false;
   }
 
   bool loaded_;
   bool shutdown_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(SnippetObserver);
 };
 
 class NTPSnippetsServiceTest : public testing::Test {
  public:
   NTPSnippetsServiceTest() {}
   ~NTPSnippetsServiceTest() override {}
 
  private:
+  base::MessageLoop message_loop_;
   DISALLOW_COPY_AND_ASSIGN(NTPSnippetsServiceTest);
 };
 
+scoped_ptr<NTPSnippetsService> CreateSnippetService(
+  const std::string& language_code) {
+
+  scoped_refptr<base::SingleThreadTaskRunner> task_runner(
+      base::ThreadTaskRunnerHandle::Get());
+  scoped_refptr<net::TestURLRequestContextGetter> request_context_getter =
+      new net::TestURLRequestContextGetter(task_runner.get());
+  FakeProfileOAuth2TokenService* token_service =
+      new FakeProfileOAuth2TokenService();
+  scoped_ptr<TestSigninClient> signin_client(new TestSigninClient(nullptr));
+  scoped_ptr<AccountTrackerService> account_tracker(
+      new AccountTrackerService());

[Marc Treib, 2016/02/23 15:28:21]

Waaait, now these will be deleted when CreateSnippetService returns, but the
signin manager will retain raw pointers to them?!

[May, 2016/02/23 21:34:19]

You're right, I didn't look into what the FakeSigninManager actually did.
Changed.

+  FakeSigninManagerBase* signin_manager =  new FakeSigninManagerBase(
+      signin_client.get(), account_tracker.get());
+
+  scoped_ptr<NTPSnippetsService> service(
+      new NTPSnippetsService(task_runner.get(), language_code,
+        make_scoped_ptr(new NTPSnippetsFetcher(task_runner.get(),
+          signin_manager, token_service, request_context_getter,
+          base::FilePath()))));
+  return service;
+}
+
 TEST_F(NTPSnippetsServiceTest, Create) {
   std::string language_code("fr");

[noyau (Ping after 24h), 2016/02/23 14:23:45]

Right now it's always passing language code "fr" so you could move this into the
CreateSnippetservice function as well.

[May, 2016/02/23 21:34:18]

Done.

-  scoped_ptr<ntp_snippets::NTPSnippetsService> service(
-      new ntp_snippets::NTPSnippetsService(language_code));
-
+  scoped_ptr<NTPSnippetsService> service(CreateSnippetService(language_code));
   EXPECT_FALSE(service->is_loaded());
 }
 
 TEST_F(NTPSnippetsServiceTest, Loop) {
   std::string language_code("fr");
-  scoped_ptr<ntp_snippets::NTPSnippetsService> service(
-      new ntp_snippets::NTPSnippetsService(language_code));
+  scoped_ptr<NTPSnippetsService> service(CreateSnippetService(language_code));
 
   EXPECT_FALSE(service->is_loaded());
 
   std::string json_str(
       "{ \"recos\": [ "
       "{ \"contentInfo\": { \"url\" : \"http://localhost/foobar\" }}"
       "]}");
   EXPECT_TRUE(service->LoadFromJSONString(json_str));
 
   EXPECT_TRUE(service->is_loaded());
 
   // The same for loop without the '&' should not compile.
   for (auto& snippet : *service) {
     // Snippet here is a const.
     EXPECT_EQ(snippet.url(), GURL("http://localhost/foobar"));
   }
   // Without the const, this should not compile.
-  for (const ntp_snippets::NTPSnippet& snippet : *service) {
+  for (const NTPSnippet& snippet : *service) {
     EXPECT_EQ(snippet.url(), GURL("http://localhost/foobar"));
   }
 }
 
 TEST_F(NTPSnippetsServiceTest, Full) {
   std::string language_code("fr");
-  scoped_ptr<ntp_snippets::NTPSnippetsService> service(
-      new ntp_snippets::NTPSnippetsService(language_code));
+  scoped_ptr<NTPSnippetsService> service(CreateSnippetService(language_code));
 
   std::string json_str(
       "{ \"recos\": [ "
       "{ \"contentInfo\": {"
       "\"url\" : \"http://localhost/foobar\","
       "\"site_title\" : \"Site Title\","
       "\"favicon_url\" : \"http://localhost/favicon\","
       "\"title\" : \"Title\","
       "\"snippet\" : \"Snippet\","
       "\"thumbnailUrl\" : \"http://localhost/salient_image\","
@@ skipping 13 matching lines @@
     EXPECT_EQ(snippet.salient_image_url(),
               GURL("http://localhost/salient_image"));
     base::Time then =
         base::Time::FromUTCExploded({2015, 11, 4, 25, 13, 46, 45});
     EXPECT_EQ(then, snippet.publish_date());
   }
 }
 
 TEST_F(NTPSnippetsServiceTest, ObserverNotLoaded) {
   std::string language_code("fr");
-  scoped_ptr<ntp_snippets::NTPSnippetsService> service(
-      new ntp_snippets::NTPSnippetsService(language_code));
+  scoped_ptr<NTPSnippetsService> service(CreateSnippetService(language_code));
 
   SnippetObserver observer;
   service->AddObserver(&observer);
   EXPECT_FALSE(observer.loaded_);
 
   std::string json_str(
       "{ \"recos\": [ "
       "{ \"contentInfo\": { \"url\" : \"http://localhost/foobar\" }}"
       "]}");
   EXPECT_TRUE(service->LoadFromJSONString(json_str));
   EXPECT_TRUE(observer.loaded_);
 
   service->RemoveObserver(&observer);
 }
 
 TEST_F(NTPSnippetsServiceTest, ObserverLoaded) {
   std::string language_code("fr");
-  scoped_ptr<ntp_snippets::NTPSnippetsService> service(
-      new ntp_snippets::NTPSnippetsService(language_code));
+  scoped_ptr<NTPSnippetsService> service(CreateSnippetService(language_code));
 
   std::string json_str(
       "{ \"recos\": [ "
       "{ \"contentInfo\": { \"url\" : \"http://localhost/foobar\" }}"
       "]}");
   EXPECT_TRUE(service->LoadFromJSONString(json_str));
 
   SnippetObserver observer;
   service->AddObserver(&observer);
 
   EXPECT_TRUE(observer.loaded_);
 
   service->RemoveObserver(&observer);
 }
-}  // namespace
+}  // namespace ntp_snippets