sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 1 | // Copyright 2014 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 | |
dcheng | d0fc6aa9 | 2016-04-22 18:03:12 | [diff] [blame^] | 5 | #include "components/update_client/request_sender.h" |
| 6 | |
| 7 | #include <memory> |
| 8 | |
sorin | 5cb1f549 | 2014-09-23 04:07:44 | [diff] [blame] | 9 | #include "base/macros.h" |
sorin | 9797aba | 2015-04-17 17:15:03 | [diff] [blame] | 10 | #include "base/memory/ref_counted.h" |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 11 | #include "base/run_loop.h" |
sorin | 1bc5eff | 2016-02-17 18:45:17 | [diff] [blame] | 12 | #include "base/strings/string_util.h" |
skyostil | b0daa01 | 2015-06-02 19:03:48 | [diff] [blame] | 13 | #include "base/thread_task_runner_handle.h" |
sorin | b120440b | 2015-04-27 16:34:15 | [diff] [blame] | 14 | #include "components/update_client/test_configurator.h" |
| 15 | #include "components/update_client/url_request_post_interceptor.h" |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 16 | #include "net/url_request/url_fetcher.h" |
| 17 | #include "testing/gtest/include/gtest/gtest.h" |
| 18 | |
sorin | 52ac088 | 2015-01-24 01:15:00 | [diff] [blame] | 19 | namespace update_client { |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 20 | |
| 21 | namespace { |
| 22 | |
| 23 | const char kUrl1[] = "https://localhost2/path1"; |
| 24 | const char kUrl2[] = "https://localhost2/path2"; |
| 25 | const char kUrlPath1[] = "path1"; |
| 26 | const char kUrlPath2[] = "path2"; |
| 27 | |
sorin | 1bc5eff | 2016-02-17 18:45:17 | [diff] [blame] | 28 | // TODO(sorin): refactor as a utility function for unit tests. |
| 29 | base::FilePath test_file(const char* file) { |
| 30 | base::FilePath path; |
| 31 | PathService::Get(base::DIR_SOURCE_ROOT, &path); |
| 32 | return path.AppendASCII("components") |
| 33 | .AppendASCII("test") |
| 34 | .AppendASCII("data") |
| 35 | .AppendASCII("update_client") |
| 36 | .AppendASCII(file); |
| 37 | } |
| 38 | |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 39 | } // namespace |
| 40 | |
| 41 | class RequestSenderTest : public testing::Test { |
| 42 | public: |
| 43 | RequestSenderTest(); |
dcheng | 30a1b154 | 2014-10-29 21:27:50 | [diff] [blame] | 44 | ~RequestSenderTest() override; |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 45 | |
| 46 | // Overrides from testing::Test. |
dcheng | 30a1b154 | 2014-10-29 21:27:50 | [diff] [blame] | 47 | void SetUp() override; |
| 48 | void TearDown() override; |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 49 | |
sorin | fccbf2d | 2016-04-04 20:34:34 | [diff] [blame] | 50 | void RequestSenderComplete(int error, |
| 51 | const std::string& response, |
| 52 | int retry_after_sec); |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 53 | |
| 54 | protected: |
| 55 | void Quit(); |
| 56 | void RunThreads(); |
| 57 | void RunThreadsUntilIdle(); |
| 58 | |
sorin | 9797aba | 2015-04-17 17:15:03 | [diff] [blame] | 59 | scoped_refptr<TestConfigurator> config_; |
dcheng | d0fc6aa9 | 2016-04-22 18:03:12 | [diff] [blame^] | 60 | std::unique_ptr<RequestSender> request_sender_; |
| 61 | std::unique_ptr<InterceptorFactory> interceptor_factory_; |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 62 | |
sorin | 5cabcf0b | 2015-06-09 00:54:44 | [diff] [blame] | 63 | URLRequestPostInterceptor* post_interceptor_1_; // Owned by the factory. |
| 64 | URLRequestPostInterceptor* post_interceptor_2_; // Owned by the factory. |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 65 | |
sorin | 1bc5eff | 2016-02-17 18:45:17 | [diff] [blame] | 66 | int error_; |
| 67 | std::string response_; |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 68 | |
| 69 | private: |
| 70 | base::MessageLoopForIO loop_; |
| 71 | base::Closure quit_closure_; |
| 72 | |
| 73 | DISALLOW_COPY_AND_ASSIGN(RequestSenderTest); |
| 74 | }; |
| 75 | |
| 76 | RequestSenderTest::RequestSenderTest() |
sorin | 1bc5eff | 2016-02-17 18:45:17 | [diff] [blame] | 77 | : post_interceptor_1_(nullptr), post_interceptor_2_(nullptr), error_(0) {} |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 78 | |
sorin | 1bc5eff | 2016-02-17 18:45:17 | [diff] [blame] | 79 | RequestSenderTest::~RequestSenderTest() {} |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 80 | |
| 81 | void RequestSenderTest::SetUp() { |
skyostil | b0daa01 | 2015-06-02 19:03:48 | [diff] [blame] | 82 | config_ = new TestConfigurator(base::ThreadTaskRunnerHandle::Get(), |
| 83 | base::ThreadTaskRunnerHandle::Get()); |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 84 | interceptor_factory_.reset( |
skyostil | b0daa01 | 2015-06-02 19:03:48 | [diff] [blame] | 85 | new InterceptorFactory(base::ThreadTaskRunnerHandle::Get())); |
sorin | 5cabcf0b | 2015-06-09 00:54:44 | [diff] [blame] | 86 | post_interceptor_1_ = |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 87 | interceptor_factory_->CreateInterceptorForPath(kUrlPath1); |
sorin | 5cabcf0b | 2015-06-09 00:54:44 | [diff] [blame] | 88 | post_interceptor_2_ = |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 89 | interceptor_factory_->CreateInterceptorForPath(kUrlPath2); |
sorin | 5cabcf0b | 2015-06-09 00:54:44 | [diff] [blame] | 90 | EXPECT_TRUE(post_interceptor_1_); |
| 91 | EXPECT_TRUE(post_interceptor_2_); |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 92 | |
| 93 | request_sender_.reset(); |
| 94 | } |
| 95 | |
| 96 | void RequestSenderTest::TearDown() { |
| 97 | request_sender_.reset(); |
| 98 | |
sorin | 5cabcf0b | 2015-06-09 00:54:44 | [diff] [blame] | 99 | post_interceptor_1_ = nullptr; |
| 100 | post_interceptor_2_ = nullptr; |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 101 | |
| 102 | interceptor_factory_.reset(); |
| 103 | |
sorin | 9797aba | 2015-04-17 17:15:03 | [diff] [blame] | 104 | config_ = nullptr; |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 105 | |
| 106 | RunThreadsUntilIdle(); |
| 107 | } |
| 108 | |
| 109 | void RequestSenderTest::RunThreads() { |
| 110 | base::RunLoop runloop; |
| 111 | quit_closure_ = runloop.QuitClosure(); |
| 112 | runloop.Run(); |
| 113 | |
| 114 | // Since some tests need to drain currently enqueued tasks such as network |
| 115 | // intercepts on the IO thread, run the threads until they are |
| 116 | // idle. The component updater service won't loop again until the loop count |
| 117 | // is set and the service is started. |
| 118 | RunThreadsUntilIdle(); |
| 119 | } |
| 120 | |
| 121 | void RequestSenderTest::RunThreadsUntilIdle() { |
| 122 | base::RunLoop().RunUntilIdle(); |
| 123 | } |
| 124 | |
| 125 | void RequestSenderTest::Quit() { |
| 126 | if (!quit_closure_.is_null()) |
| 127 | quit_closure_.Run(); |
| 128 | } |
| 129 | |
sorin | 1bc5eff | 2016-02-17 18:45:17 | [diff] [blame] | 130 | void RequestSenderTest::RequestSenderComplete(int error, |
sorin | fccbf2d | 2016-04-04 20:34:34 | [diff] [blame] | 131 | const std::string& response, |
| 132 | int retry_after_sec) { |
sorin | 1bc5eff | 2016-02-17 18:45:17 | [diff] [blame] | 133 | error_ = error; |
| 134 | response_ = response; |
| 135 | |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 136 | Quit(); |
| 137 | } |
| 138 | |
| 139 | // Tests that when a request to the first url succeeds, the subsequent urls are |
| 140 | // not tried. |
| 141 | TEST_F(RequestSenderTest, RequestSendSuccess) { |
sorin | 1bc5eff | 2016-02-17 18:45:17 | [diff] [blame] | 142 | EXPECT_TRUE(post_interceptor_1_->ExpectRequest( |
| 143 | new PartialMatch("test"), test_file("updatecheck_reply_1.xml"))); |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 144 | |
| 145 | std::vector<GURL> urls; |
| 146 | urls.push_back(GURL(kUrl1)); |
| 147 | urls.push_back(GURL(kUrl2)); |
sorin | 958b5d3 | 2016-01-09 02:00:20 | [diff] [blame] | 148 | request_sender_.reset(new RequestSender(config_)); |
sorin | 1bc5eff | 2016-02-17 18:45:17 | [diff] [blame] | 149 | request_sender_->Send(false, "test", urls, |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 150 | base::Bind(&RequestSenderTest::RequestSenderComplete, |
| 151 | base::Unretained(this))); |
| 152 | RunThreads(); |
| 153 | |
sorin | 5cabcf0b | 2015-06-09 00:54:44 | [diff] [blame] | 154 | EXPECT_EQ(1, post_interceptor_1_->GetHitCount()) |
| 155 | << post_interceptor_1_->GetRequestsAsString(); |
| 156 | EXPECT_EQ(1, post_interceptor_1_->GetCount()) |
| 157 | << post_interceptor_1_->GetRequestsAsString(); |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 158 | |
sorin | 1bc5eff | 2016-02-17 18:45:17 | [diff] [blame] | 159 | // Sanity check the request. |
sorin | 5cabcf0b | 2015-06-09 00:54:44 | [diff] [blame] | 160 | EXPECT_STREQ("test", post_interceptor_1_->GetRequests()[0].c_str()); |
sorin | 1bc5eff | 2016-02-17 18:45:17 | [diff] [blame] | 161 | |
| 162 | // Check the response post conditions. |
| 163 | EXPECT_EQ(0, error_); |
| 164 | EXPECT_TRUE(base::StartsWith(response_, |
| 165 | "<?xml version='1.0' encoding='UTF-8'?>", |
| 166 | base::CompareCase::SENSITIVE)); |
| 167 | EXPECT_EQ(443ul, response_.size()); |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 168 | } |
| 169 | |
| 170 | // Tests that the request succeeds using the second url after the first url |
| 171 | // has failed. |
| 172 | TEST_F(RequestSenderTest, RequestSendSuccessWithFallback) { |
sorin | 5cabcf0b | 2015-06-09 00:54:44 | [diff] [blame] | 173 | EXPECT_TRUE( |
| 174 | post_interceptor_1_->ExpectRequest(new PartialMatch("test"), 403)); |
| 175 | EXPECT_TRUE(post_interceptor_2_->ExpectRequest(new PartialMatch("test"))); |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 176 | |
| 177 | std::vector<GURL> urls; |
| 178 | urls.push_back(GURL(kUrl1)); |
| 179 | urls.push_back(GURL(kUrl2)); |
sorin | 958b5d3 | 2016-01-09 02:00:20 | [diff] [blame] | 180 | request_sender_.reset(new RequestSender(config_)); |
sorin | 1bc5eff | 2016-02-17 18:45:17 | [diff] [blame] | 181 | request_sender_->Send(false, "test", urls, |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 182 | base::Bind(&RequestSenderTest::RequestSenderComplete, |
| 183 | base::Unretained(this))); |
| 184 | RunThreads(); |
| 185 | |
sorin | 5cabcf0b | 2015-06-09 00:54:44 | [diff] [blame] | 186 | EXPECT_EQ(1, post_interceptor_1_->GetHitCount()) |
| 187 | << post_interceptor_1_->GetRequestsAsString(); |
| 188 | EXPECT_EQ(1, post_interceptor_1_->GetCount()) |
| 189 | << post_interceptor_1_->GetRequestsAsString(); |
| 190 | EXPECT_EQ(1, post_interceptor_2_->GetHitCount()) |
| 191 | << post_interceptor_2_->GetRequestsAsString(); |
| 192 | EXPECT_EQ(1, post_interceptor_2_->GetCount()) |
| 193 | << post_interceptor_2_->GetRequestsAsString(); |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 194 | |
sorin | 5cabcf0b | 2015-06-09 00:54:44 | [diff] [blame] | 195 | EXPECT_STREQ("test", post_interceptor_1_->GetRequests()[0].c_str()); |
| 196 | EXPECT_STREQ("test", post_interceptor_2_->GetRequests()[0].c_str()); |
sorin | 1bc5eff | 2016-02-17 18:45:17 | [diff] [blame] | 197 | EXPECT_EQ(0, error_); |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 198 | } |
| 199 | |
| 200 | // Tests that the request fails when both urls have failed. |
| 201 | TEST_F(RequestSenderTest, RequestSendFailed) { |
sorin | 5cabcf0b | 2015-06-09 00:54:44 | [diff] [blame] | 202 | EXPECT_TRUE( |
| 203 | post_interceptor_1_->ExpectRequest(new PartialMatch("test"), 403)); |
| 204 | EXPECT_TRUE( |
| 205 | post_interceptor_2_->ExpectRequest(new PartialMatch("test"), 403)); |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 206 | |
| 207 | std::vector<GURL> urls; |
| 208 | urls.push_back(GURL(kUrl1)); |
| 209 | urls.push_back(GURL(kUrl2)); |
sorin | 958b5d3 | 2016-01-09 02:00:20 | [diff] [blame] | 210 | request_sender_.reset(new RequestSender(config_)); |
sorin | 1bc5eff | 2016-02-17 18:45:17 | [diff] [blame] | 211 | request_sender_->Send(false, "test", urls, |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 212 | base::Bind(&RequestSenderTest::RequestSenderComplete, |
| 213 | base::Unretained(this))); |
| 214 | RunThreads(); |
| 215 | |
sorin | 5cabcf0b | 2015-06-09 00:54:44 | [diff] [blame] | 216 | EXPECT_EQ(1, post_interceptor_1_->GetHitCount()) |
| 217 | << post_interceptor_1_->GetRequestsAsString(); |
| 218 | EXPECT_EQ(1, post_interceptor_1_->GetCount()) |
| 219 | << post_interceptor_1_->GetRequestsAsString(); |
| 220 | EXPECT_EQ(1, post_interceptor_2_->GetHitCount()) |
| 221 | << post_interceptor_2_->GetRequestsAsString(); |
| 222 | EXPECT_EQ(1, post_interceptor_2_->GetCount()) |
| 223 | << post_interceptor_2_->GetRequestsAsString(); |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 224 | |
sorin | 5cabcf0b | 2015-06-09 00:54:44 | [diff] [blame] | 225 | EXPECT_STREQ("test", post_interceptor_1_->GetRequests()[0].c_str()); |
| 226 | EXPECT_STREQ("test", post_interceptor_2_->GetRequests()[0].c_str()); |
sorin | 1bc5eff | 2016-02-17 18:45:17 | [diff] [blame] | 227 | EXPECT_EQ(403, error_); |
sorin | 395c2ac | 2014-09-16 21:31:07 | [diff] [blame] | 228 | } |
| 229 | |
sorin | 5cabcf0b | 2015-06-09 00:54:44 | [diff] [blame] | 230 | // Tests that the request fails when no urls are provided. |
| 231 | TEST_F(RequestSenderTest, RequestSendFailedNoUrls) { |
| 232 | std::vector<GURL> urls; |
sorin | 958b5d3 | 2016-01-09 02:00:20 | [diff] [blame] | 233 | request_sender_.reset(new RequestSender(config_)); |
sorin | 1bc5eff | 2016-02-17 18:45:17 | [diff] [blame] | 234 | request_sender_->Send(false, "test", urls, |
sorin | 5cabcf0b | 2015-06-09 00:54:44 | [diff] [blame] | 235 | base::Bind(&RequestSenderTest::RequestSenderComplete, |
| 236 | base::Unretained(this))); |
| 237 | RunThreads(); |
| 238 | |
sorin | 1bc5eff | 2016-02-17 18:45:17 | [diff] [blame] | 239 | EXPECT_EQ(-1, error_); |
| 240 | } |
| 241 | |
| 242 | // Tests that a CUP request fails if the response is not signed. |
| 243 | TEST_F(RequestSenderTest, RequestSendCupError) { |
| 244 | EXPECT_TRUE(post_interceptor_1_->ExpectRequest( |
| 245 | new PartialMatch("test"), test_file("updatecheck_reply_1.xml"))); |
| 246 | |
| 247 | std::vector<GURL> urls; |
| 248 | urls.push_back(GURL(kUrl1)); |
| 249 | request_sender_.reset(new RequestSender(config_)); |
| 250 | request_sender_->Send(true, "test", urls, |
| 251 | base::Bind(&RequestSenderTest::RequestSenderComplete, |
| 252 | base::Unretained(this))); |
| 253 | RunThreads(); |
| 254 | |
| 255 | EXPECT_EQ(1, post_interceptor_1_->GetHitCount()) |
| 256 | << post_interceptor_1_->GetRequestsAsString(); |
| 257 | EXPECT_EQ(1, post_interceptor_1_->GetCount()) |
| 258 | << post_interceptor_1_->GetRequestsAsString(); |
| 259 | |
| 260 | EXPECT_STREQ("test", post_interceptor_1_->GetRequests()[0].c_str()); |
| 261 | EXPECT_EQ(RequestSender::kErrorResponseNotTrusted, error_); |
| 262 | EXPECT_TRUE(response_.empty()); |
sorin | 5cabcf0b | 2015-06-09 00:54:44 | [diff] [blame] | 263 | } |
| 264 | |
sorin | 52ac088 | 2015-01-24 01:15:00 | [diff] [blame] | 265 | } // namespace update_client |