Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1551)

Side by Side Diff: components/doodle/doodle_fetcher_unittest.cc

Issue 2660883002: Introduce a Doodle Fetcher for NTP (Closed)
Patch Set: CMD line BaseUrl. [Rebase, Weak tests] Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "components/doodle/doodle_fetcher.h"
6
7 #include <string>
8 #include <utility>
9
10 #include "base/bind.h"
11 #include "base/json/json_reader.h"
12 #include "base/memory/ptr_util.h"
13 #include "base/test/test_mock_time_task_runner.h"
14 #include "base/threading/sequenced_task_runner_handle.h"
15 #include "base/threading/thread_task_runner_handle.h"
16 #include "base/values.h"
17 #include "net/http/http_status_code.h"
18 #include "net/url_request/test_url_fetcher_factory.h"
19 #include "net/url_request/url_request_status.h"
20 #include "net/url_request/url_request_test_util.h"
21 #include "testing/gmock/include/gmock/gmock.h"
22 #include "testing/gtest/include/gtest/gtest.h"
23
24 using testing::Eq;
25 using testing::IsEmpty;
26
27 namespace doodle {
28
29 namespace {
30 const char kSampleResponse[] = R"json()]}'{
31 "ddljson": {
32 "alt_text":"Mouseover Text",
33 "doodle_type":"SIMPLE",
34 "id":0,
35 "search_url":"/search?q\u003dtest",
36 "share_text":"Share Text #GoogleDoodle\nhttps://g.co/doodle/8hfqzq",
37 "short_link":"//g.co/doodle/8hfqzq",
38 "show_now_header_search_affordance":false,
39 "show_now_header_share_button":true,
40 "target_url":"/search?q\u003dtest\u0026sa\u003dX\u0026ved\u003d0ahUKEwjm",
41 "time_to_live_ms":514220215000,
42 "large_image": {
43 "background_color":"#ffffff",
44 "height":225,
45 "image_id":0,
46 "is_animated_gif":true,
47 "is_cta":false,
48 "slot":1,
49 "url":"/logos/doodles/2015/new-years-eve-2015-5985438795825152-hp.gif",
50 "width":489
51 },
52 "large_cta_image": {
53 "background_color":"#ffffff",
54 "height":225,
55 "image_id":0,
56 "is_animated_gif":true,
57 "is_cta":true,
58 "slot":8,
59 "url":"/logos/doodles/2015/new-years-eve-2015-5985438795825152-cta.gif",
60 "width":489
61 },
62 "transparent_large_image": {
63 "background_color":"",
64 "height":225,
65 "image_id":0,
66 "is_animated_gif":false,
67 "is_cta":false,
68 "slot":2,
69 "url":"/logos/doodles/2015/new-years-eve-2015-5985438795825152-thp.png",
70 "width":510
71 }
72 }})json";
73
74 const char kBaseUrl[] = "https://www.google.com/";
75
76 std::string Resolve(const std::string& relative_url) {
77 return GURL(kBaseUrl).Resolve(relative_url).spec();
78 }
79
80 void ParseJson(
81 const std::string& json,
82 const base::Callback<void(std::unique_ptr<base::Value> json)>& success,
83 const base::Callback<void(const std::string&)>& error) {
84 base::JSONReader json_reader;
85 std::unique_ptr<base::Value> value = json_reader.ReadToValue(json);
86 if (value) {
87 success.Run(std::move(value));
88 } else {
89 error.Run(json_reader.GetErrorMessage());
90 }
91 }
92
93 } // namespace
94
95 class DoodleFetcherTest : public ::testing::Test {
96 public:
97 DoodleFetcherTest()
98 : url_(GURL(kBaseUrl).Resolve(kDoodleConfigPath)),
99 // Constructor of FakeURLFetcherFactory uses UrlFetcherImpl as fallback
100 // for instantiating fetchers for undefined paths.
101 // TODO(fhorschig): Consider FailingUrlFetcher when testing BaseUrl.
102 url_fetcher_factory_(/*default_implementation=*/nullptr),
103 mock_task_runner_(new base::TestMockTimeTaskRunner()),
104 mock_task_runner_handle_(mock_task_runner_),
105 context_getter(
106 new net::TestURLRequestContextGetter(mock_task_runner_.get())) {
107 // Random difference to 0 ensures that expiry_dates are really relative.
108 mock_task_runner_->FastForwardBy(base::TimeDelta::FromMilliseconds(80082));
109 }
110
111 void SetResponse(const std::string& data) {
112 url_fetcher_factory_.SetFakeResponse(url_, data, net::HTTP_OK,
113 net::URLRequestStatus::SUCCESS);
114 }
115
116 void SetResponseToError() {
117 url_fetcher_factory_.SetFakeResponse(url_, "", net::HTTP_NOT_FOUND,
118 net::URLRequestStatus::FAILED);
119 }
120
121 DoodleState FetchDoodle(base::Optional<DoodleConfig>* config) {
122 DoodleState state;
123 std::unique_ptr<DoodleFetcher> fetcher = CreateDoodleFetcher();
124 TriggerFetch(fetcher.get(), &state, config);
125 WaitForCallbacksToReturn();
126 return state;
127 }
128
129 std::unique_ptr<DoodleFetcher> CreateDoodleFetcher() {
130 // TODO(fhorschig): Create Fake for GoogleUrlTracker.
131 std::unique_ptr<DoodleFetcher> fetcher = base::MakeUnique<DoodleFetcher>(
132 context_getter.get(),
133 /*google_url_tracker=*/nullptr, base::Bind(ParseJson));
134 fetcher->SetClockForTesting(mock_task_runner_->GetMockClock());
135 return fetcher;
136 }
137
138 void TriggerFetch(DoodleFetcher* fetcher,
139 DoodleState* state,
140 base::Optional<DoodleConfig>* config) {
141 fetcher->FetchDoodle(base::BindOnce(
142 [](DoodleState* state_out, base::Optional<DoodleConfig>* config_out,
143 DoodleState state, base::Optional<DoodleConfig> config) {
144 *state_out = std::move(state);
145 *config_out = std::move(config);
146 },
147 state, config));
148 }
149
150 base::Time TimeFromNow(uint64_t milliseconds) {
151 return mock_task_runner_->GetMockClock()->Now() +
152 base::TimeDelta::FromMilliseconds(milliseconds);
153 }
154
155 void WaitForCallbacksToReturn() {
156 mock_task_runner_->FastForwardUntilNoTasksRemain();
157 }
158
159 size_t NumberOfPendingTasks() {
160 return mock_task_runner_->GetPendingTaskCount();
161 }
162
163 private:
164 GURL url_;
165 net::FakeURLFetcherFactory url_fetcher_factory_;
166 scoped_refptr<base::TestMockTimeTaskRunner> mock_task_runner_;
167 base::ThreadTaskRunnerHandle mock_task_runner_handle_;
168 scoped_refptr<net::TestURLRequestContextGetter> context_getter;
169 };
170
171 // TODO(fhorschig): Test invalid image urls.
172 // TODO(fhorschig): Test invalid timestamps.
173 // TODO(fhorschig): Test (CMD, Tracker) BaseUrls.
174
175 TEST_F(DoodleFetcherTest, ReturnsFromFetchWithoutError) {
176 SetResponse(kSampleResponse);
177
178 base::Optional<DoodleConfig> response;
179 EXPECT_THAT(FetchDoodle(&response), Eq(DoodleState::AVAILABLE));
180 EXPECT_TRUE(response.has_value());
181 }
182
183 TEST_F(DoodleFetcherTest, ReturnsFrom404FetchWithError) {
184 SetResponseToError();
185
186 base::Optional<DoodleConfig> response;
187 EXPECT_THAT(FetchDoodle(&response), Eq(DoodleState::DOWNLOAD_ERROR));
188 EXPECT_FALSE(response.has_value());
189 }
190
191 TEST_F(DoodleFetcherTest, ReturnsErrorForInvalidJson) {
192 SetResponse("{}");
193
194 base::Optional<DoodleConfig> response;
195 EXPECT_THAT(FetchDoodle(&response), Eq(DoodleState::PARSING_ERROR));
196 EXPECT_FALSE(response.has_value());
197 }
198
199 TEST_F(DoodleFetcherTest, ResponseContainsValidBaseInformation) {
200 SetResponse(kSampleResponse);
201
202 base::Optional<DoodleConfig> response;
203 EXPECT_THAT(FetchDoodle(&response), Eq(DoodleState::AVAILABLE));
204 ASSERT_TRUE(response.has_value());
205 DoodleConfig config = response.value();
206
207 EXPECT_TRUE(config.search_url.is_valid());
208 EXPECT_THAT(config.search_url, Eq(Resolve("/search?q\u003dtest")));
209 EXPECT_THAT(config.fullpage_interactive_url, Eq(GURL()));
210 EXPECT_TRUE(config.target_url.is_valid());
211 EXPECT_THAT(config.target_url,
212 Eq(Resolve("/search?q\u003dtest\u0026sa\u003dX\u0026ved\u003d"
213 "0ahUKEwjm")));
214 EXPECT_THAT(config.doodle_type, Eq(DoodleType::SIMPLE));
215 EXPECT_THAT(config.alt_text, Eq("Mouseover Text"));
216
217 EXPECT_THAT(config.expiry_date, Eq(TimeFromNow(514220215000)));
218 }
219
220 TEST_F(DoodleFetcherTest, EmptyResponsesCausesNoDoodleState) {
221 SetResponse("{\"ddljson\":{}}");
222
223 base::Optional<DoodleConfig> response;
224 EXPECT_THAT(FetchDoodle(&response), Eq(DoodleState::NO_DOODLE));
225 EXPECT_FALSE(response.has_value());
226 }
227
228 TEST_F(DoodleFetcherTest, ResponseContainsExactlyTheSampleImages) {
229 SetResponse(kSampleResponse);
230
231 base::Optional<DoodleConfig> response;
232 EXPECT_THAT(FetchDoodle(&response), Eq(DoodleState::AVAILABLE));
233 ASSERT_TRUE(response.has_value());
234 DoodleConfig config = response.value();
235
236 EXPECT_TRUE(config.transparent_large_image.url.is_valid());
237 EXPECT_THAT(config.transparent_large_image.url.spec(),
238 Eq(Resolve("/logos/doodles/2015/new-years-eve-2015-5985438795"
239 "825152-thp.png")));
240 EXPECT_THAT(config.transparent_large_image.width, Eq(510));
241 EXPECT_THAT(config.transparent_large_image.height, Eq(225));
242 EXPECT_FALSE(config.transparent_large_image.is_animated_gif);
243 EXPECT_FALSE(config.transparent_large_image.is_cta);
244
245 EXPECT_TRUE(config.large_image.url.is_valid());
246 EXPECT_THAT(config.large_image.url.spec(),
247 Eq(Resolve("/logos/doodles/2015/new-years-eve-2015-5985438795"
248 "825152-hp.gif")));
249 EXPECT_THAT(config.large_image.width, Eq(489));
250 EXPECT_THAT(config.large_image.height, Eq(225));
251 EXPECT_TRUE(config.large_image.is_animated_gif);
252 EXPECT_FALSE(config.large_image.is_cta);
253
254 EXPECT_TRUE(config.large_cta_image.url.is_valid());
255 EXPECT_THAT(config.large_cta_image.url.spec(),
256 Eq(Resolve("/logos/doodles/2015/new-years-eve-2015-5985438795"
257 "825152-cta.gif")));
258 EXPECT_THAT(config.large_cta_image.width, Eq(489));
259 EXPECT_THAT(config.large_cta_image.height, Eq(225));
260 EXPECT_TRUE(config.large_cta_image.is_animated_gif);
261 EXPECT_TRUE(config.large_cta_image.is_cta);
262 }
263
264 TEST_F(DoodleFetcherTest, RespondsToMultipleRequestsWithSameResponse) {
265 SetResponse(kSampleResponse);
266
267 DoodleState state1;
268 DoodleState state2;
269 base::Optional<DoodleConfig> response1;
270 base::Optional<DoodleConfig> response2;
271 std::unique_ptr<DoodleFetcher> fetcher = CreateDoodleFetcher();
272
273 TriggerFetch(fetcher.get(), &state1, &response1);
274 TriggerFetch(fetcher.get(), &state2, &response2);
275
276 EXPECT_THAT(NumberOfPendingTasks(), Eq(1ul));
277 WaitForCallbacksToReturn();
278
279 EXPECT_THAT(state1, Eq(DoodleState::AVAILABLE));
280 EXPECT_THAT(state2, Eq(DoodleState::AVAILABLE));
281 EXPECT_TRUE(response1.has_value());
282 EXPECT_TRUE(response2.has_value());
283 }
284
285 } // namespace doodle
OLDNEW
« components/doodle/doodle_fetcher.cc ('K') | « components/doodle/doodle_fetcher.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698