[NTP Snippets] Persist non-article remote suggestions in the DB

components/ntp_snippets/remote/ntp_snippet_unittest.cc

 // Copyright 2016 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/remote/ntp_snippet.h"
 
 #include "base/json/json_reader.h"
 #include "base/values.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace ntp_snippets {
 namespace {
 
 std::unique_ptr<NTPSnippet> SnippetFromContentSuggestionJSON(
     const std::string& json) {
   auto json_value = base::JSONReader::Read(json);
   base::DictionaryValue* json_dict;
   if (!json_value->GetAsDictionary(&json_dict)) {
     return nullptr;
   }
-  return NTPSnippet::CreateFromContentSuggestionsDictionary(*json_dict);
+  return NTPSnippet::CreateFromContentSuggestionsDictionary(*json_dict, 1);

[Bernhard Bauer, 2016/10/10 13:54:54]

You could expose the remote category in the header instead of hard-coding it?

[Marc Treib, 2016/10/10 15:54:47]

Done.

 }
 
 TEST(NTPSnippetTest, FromChromeContentSuggestionsDictionary) {
   const std::string kJsonStr =
       "{"
       "  \"ids\" : [\"http://localhost/foobar\"],"
       "  \"title\" : \"Foo Barred from Baz\","
       "  \"snippet\" : \"...\","
       "  \"fullPageUrl\" : \"http://localhost/foobar\","
       "  \"creationTime\" : \"2016-06-30T11:01:37.000Z\","
@@ skipping 225 matching lines @@
 
   EXPECT_EQ(snippet->sources().size(), 3u);
   EXPECT_EQ(snippet->id(), "http://url.com");
   EXPECT_EQ(snippet->best_source().url, GURL("http://source1.com"));
   EXPECT_EQ(snippet->best_source().publisher_name, std::string("Source 1"));
   EXPECT_EQ(snippet->best_source().amp_url, GURL("http://source1.amp.com"));
 }
 
 }  // namespace
 }  // namespace ntp_snippets