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