Paul Stewart | e692740 | 2012-01-23 16:11:30 -0800 | [diff] [blame] | 1 | // Copyright (c) 2012 The Chromium OS Authors. All rights reserved. |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 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 "shill/dns_client.h" |
| 6 | |
| 7 | #include <netdb.h> |
| 8 | |
Ben Chan | cd47732 | 2014-10-17 14:19:30 -0700 | [diff] [blame] | 9 | #include <memory> |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 10 | #include <string> |
| 11 | #include <vector> |
| 12 | |
Eric Shienbrood | 3e20a23 | 2012-02-16 11:35:56 -0500 | [diff] [blame] | 13 | #include <base/bind.h> |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 14 | |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 15 | #include "shill/error.h" |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 16 | #include "shill/event_dispatcher.h" |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 17 | #include "shill/mock_ares.h" |
| 18 | #include "shill/mock_control.h" |
| 19 | #include "shill/mock_event_dispatcher.h" |
Peter Qiu | 8d6b597 | 2014-10-28 15:33:34 -0700 | [diff] [blame] | 20 | #include "shill/net/io_handler.h" |
| 21 | #include "shill/net/mock_time.h" |
Ben Chan | be277dd | 2014-02-05 17:26:47 -0800 | [diff] [blame] | 22 | #include "shill/testing.h" |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 23 | |
Eric Shienbrood | 3e20a23 | 2012-02-16 11:35:56 -0500 | [diff] [blame] | 24 | using base::Bind; |
| 25 | using base::Unretained; |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 26 | using std::string; |
| 27 | using std::vector; |
| 28 | using testing::_; |
| 29 | using testing::DoAll; |
Ben Chan | be277dd | 2014-02-05 17:26:47 -0800 | [diff] [blame] | 30 | using testing::Not; |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 31 | using testing::Return; |
| 32 | using testing::ReturnArg; |
| 33 | using testing::ReturnNew; |
| 34 | using testing::Test; |
| 35 | using testing::SetArgumentPointee; |
| 36 | using testing::StrEq; |
| 37 | using testing::StrictMock; |
| 38 | |
| 39 | namespace shill { |
| 40 | |
| 41 | namespace { |
| 42 | const char kGoodName[] = "all-systems.mcast.net"; |
| 43 | const char kResult[] = "224.0.0.1"; |
| 44 | const char kGoodServer[] = "8.8.8.8"; |
| 45 | const char kBadServer[] = "10.9xx8.7"; |
| 46 | const char kNetworkInterface[] = "eth0"; |
Han Shen | fc34925 | 2012-08-30 11:36:04 -0700 | [diff] [blame] | 47 | char kReturnAddressList0[] = { static_cast<char>(224), 0, 0, 1 }; |
Ben Chan | cc225ef | 2014-09-30 13:26:51 -0700 | [diff] [blame] | 48 | char *kReturnAddressList[] = { kReturnAddressList0, nullptr }; |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 49 | char kFakeAresChannelData = 0; |
| 50 | const ares_channel kAresChannel = |
| 51 | reinterpret_cast<ares_channel>(&kFakeAresChannelData); |
| 52 | const int kAresFd = 10203; |
| 53 | const int kAresTimeoutMS = 2000; // ARES transaction timeout |
| 54 | const int kAresWaitMS = 1000; // Time period ARES asks caller to wait |
Alex Vakulenko | 8a53229 | 2014-06-16 17:18:44 -0700 | [diff] [blame] | 55 | } // namespace |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 56 | |
| 57 | class DNSClientTest : public Test { |
| 58 | public: |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 59 | DNSClientTest() |
| 60 | : ares_result_(ARES_SUCCESS), address_result_(IPAddress::kFamilyUnknown) { |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 61 | time_val_.tv_sec = 0; |
| 62 | time_val_.tv_usec = 0; |
| 63 | ares_timeout_.tv_sec = kAresWaitMS / 1000; |
| 64 | ares_timeout_.tv_usec = (kAresWaitMS % 1000) * 1000; |
| 65 | hostent_.h_addrtype = IPAddress::kFamilyIPv4; |
| 66 | hostent_.h_length = sizeof(kReturnAddressList0); |
| 67 | hostent_.h_addr_list = kReturnAddressList; |
| 68 | } |
| 69 | |
| 70 | virtual void SetUp() { |
Paul Stewart | e692740 | 2012-01-23 16:11:30 -0800 | [diff] [blame] | 71 | EXPECT_CALL(time_, GetTimeMonotonic(_)) |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 72 | .WillRepeatedly(DoAll(SetArgumentPointee<0>(time_val_), Return(0))); |
| 73 | SetInActive(); |
| 74 | } |
| 75 | |
| 76 | virtual void TearDown() { |
| 77 | // We need to make sure the dns_client instance releases ares_ |
| 78 | // before the destructor for DNSClientTest deletes ares_. |
| 79 | if (dns_client_.get()) { |
| 80 | dns_client_->Stop(); |
| 81 | } |
| 82 | } |
| 83 | |
| 84 | void AdvanceTime(int time_ms) { |
| 85 | struct timeval adv_time = { time_ms/1000, (time_ms % 1000) * 1000 }; |
| 86 | timeradd(&time_val_, &adv_time, &time_val_); |
Paul Stewart | e692740 | 2012-01-23 16:11:30 -0800 | [diff] [blame] | 87 | EXPECT_CALL(time_, GetTimeMonotonic(_)) |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 88 | .WillRepeatedly(DoAll(SetArgumentPointee<0>(time_val_), Return(0))); |
| 89 | } |
| 90 | |
| 91 | void CallReplyCB() { |
| 92 | dns_client_->ReceiveDNSReplyCB(dns_client_.get(), ares_result_, 0, |
| 93 | &hostent_); |
| 94 | } |
| 95 | |
| 96 | void CallDNSRead() { |
| 97 | dns_client_->HandleDNSRead(kAresFd); |
| 98 | } |
| 99 | |
| 100 | void CallDNSWrite() { |
| 101 | dns_client_->HandleDNSWrite(kAresFd); |
| 102 | } |
| 103 | |
| 104 | void CallTimeout() { |
| 105 | dns_client_->HandleTimeout(); |
| 106 | } |
| 107 | |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 108 | void CallCompletion() { |
| 109 | dns_client_->HandleCompletion(); |
| 110 | } |
| 111 | |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 112 | void CreateClient(const vector<string> &dns_servers, int timeout_ms) { |
| 113 | dns_client_.reset(new DNSClient(IPAddress::kFamilyIPv4, |
| 114 | kNetworkInterface, |
| 115 | dns_servers, |
| 116 | timeout_ms, |
| 117 | &dispatcher_, |
| 118 | callback_target_.callback())); |
| 119 | dns_client_->ares_ = &ares_; |
| 120 | dns_client_->time_ = &time_; |
| 121 | } |
| 122 | |
| 123 | void SetActive() { |
| 124 | // Returns that socket kAresFd is readable. |
| 125 | EXPECT_CALL(ares_, GetSock(_, _, _)) |
| 126 | .WillRepeatedly(DoAll(SetArgumentPointee<1>(kAresFd), Return(1))); |
| 127 | EXPECT_CALL(ares_, Timeout(_, _, _)) |
| 128 | .WillRepeatedly( |
| 129 | DoAll(SetArgumentPointee<2>(ares_timeout_), ReturnArg<2>())); |
| 130 | } |
| 131 | |
| 132 | void SetInActive() { |
| 133 | EXPECT_CALL(ares_, GetSock(_, _, _)) |
| 134 | .WillRepeatedly(Return(0)); |
| 135 | EXPECT_CALL(ares_, Timeout(_, _, _)) |
| 136 | .WillRepeatedly(ReturnArg<1>()); |
| 137 | } |
| 138 | |
| 139 | void SetupRequest(const string &name, const string &server) { |
| 140 | vector<string> dns_servers; |
| 141 | dns_servers.push_back(server); |
| 142 | CreateClient(dns_servers, kAresTimeoutMS); |
| 143 | // These expectations are fulfilled when dns_client_->Start() is called. |
| 144 | EXPECT_CALL(ares_, InitOptions(_, _, _)) |
| 145 | .WillOnce(DoAll(SetArgumentPointee<0>(kAresChannel), |
| 146 | Return(ARES_SUCCESS))); |
Peter Qiu | f3a8f90 | 2014-08-20 10:05:42 -0700 | [diff] [blame] | 147 | EXPECT_CALL(ares_, SetServersCsv(_, _)) |
| 148 | .WillOnce(Return(ARES_SUCCESS)); |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 149 | EXPECT_CALL(ares_, SetLocalDev(kAresChannel, StrEq(kNetworkInterface))) |
| 150 | .Times(1); |
| 151 | EXPECT_CALL(ares_, GetHostByName(kAresChannel, StrEq(name), _, _, _)); |
| 152 | } |
| 153 | |
| 154 | void StartValidRequest() { |
| 155 | SetupRequest(kGoodName, kGoodServer); |
| 156 | EXPECT_CALL(dispatcher_, |
| 157 | CreateReadyHandler(kAresFd, IOHandler::kModeInput, _)) |
| 158 | .WillOnce(ReturnNew<IOHandler>()); |
| 159 | SetActive(); |
| 160 | EXPECT_CALL(dispatcher_, PostDelayedTask(_, kAresWaitMS)); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 161 | Error error; |
| 162 | ASSERT_TRUE(dns_client_->Start(kGoodName, &error)); |
| 163 | EXPECT_TRUE(error.IsSuccess()); |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 164 | EXPECT_CALL(ares_, Destroy(kAresChannel)); |
| 165 | } |
| 166 | |
| 167 | void TestValidCompletion() { |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 168 | EXPECT_CALL(ares_, ProcessFd(kAresChannel, kAresFd, ARES_SOCKET_BAD)) |
| 169 | .WillOnce(InvokeWithoutArgs(this, &DNSClientTest::CallReplyCB)); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 170 | ExpectPostCompletionTask(); |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 171 | CallDNSRead(); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 172 | |
| 173 | // Make sure that the address value is correct as held in the DNSClient. |
| 174 | ASSERT_TRUE(dns_client_->address_.IsValid()); |
| 175 | IPAddress ipaddr(dns_client_->address_.family()); |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 176 | ASSERT_TRUE(ipaddr.SetAddressFromString(kResult)); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 177 | EXPECT_TRUE(ipaddr.Equals(dns_client_->address_)); |
| 178 | |
| 179 | // Make sure the callback gets called with a success result, and save |
| 180 | // the callback address argument in |address_result_|. |
Ben Chan | be277dd | 2014-02-05 17:26:47 -0800 | [diff] [blame] | 181 | EXPECT_CALL(callback_target_, CallTarget(IsSuccess(), _)) |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 182 | .WillOnce(Invoke(this, &DNSClientTest::SaveCallbackArgs)); |
| 183 | CallCompletion(); |
| 184 | |
| 185 | // Make sure the address was successfully passed to the callback. |
| 186 | EXPECT_TRUE(ipaddr.Equals(address_result_)); |
| 187 | EXPECT_TRUE(dns_client_->address_.IsDefault()); |
| 188 | } |
| 189 | |
| 190 | void SaveCallbackArgs(const Error &error, const IPAddress &address) { |
| 191 | error_result_.CopyFrom(error); |
| 192 | address_result_ = address; |
| 193 | } |
| 194 | |
| 195 | void ExpectPostCompletionTask() { |
| 196 | EXPECT_CALL(dispatcher_, PostTask(_)); |
| 197 | } |
| 198 | |
| 199 | void ExpectReset() { |
| 200 | EXPECT_TRUE(dns_client_->address_.family() == IPAddress::kFamilyIPv4); |
| 201 | EXPECT_TRUE(dns_client_->address_.IsDefault()); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 202 | EXPECT_FALSE(dns_client_->resolver_state_.get()); |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 203 | } |
| 204 | |
| 205 | protected: |
| 206 | class DNSCallbackTarget { |
| 207 | public: |
| 208 | DNSCallbackTarget() |
Eric Shienbrood | 3e20a23 | 2012-02-16 11:35:56 -0500 | [diff] [blame] | 209 | : callback_(Bind(&DNSCallbackTarget::CallTarget, Unretained(this))) {} |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 210 | |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 211 | MOCK_METHOD2(CallTarget, void(const Error &error, |
| 212 | const IPAddress &address)); |
Eric Shienbrood | 3e20a23 | 2012-02-16 11:35:56 -0500 | [diff] [blame] | 213 | const DNSClient::ClientCallback &callback() { return callback_; } |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 214 | |
| 215 | private: |
Eric Shienbrood | 3e20a23 | 2012-02-16 11:35:56 -0500 | [diff] [blame] | 216 | DNSClient::ClientCallback callback_; |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 217 | }; |
| 218 | |
Ben Chan | cd47732 | 2014-10-17 14:19:30 -0700 | [diff] [blame] | 219 | std::unique_ptr<DNSClient> dns_client_; |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 220 | StrictMock<MockEventDispatcher> dispatcher_; |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 221 | string queued_request_; |
| 222 | StrictMock<DNSCallbackTarget> callback_target_; |
| 223 | StrictMock<MockAres> ares_; |
| 224 | StrictMock<MockTime> time_; |
| 225 | struct timeval time_val_; |
| 226 | struct timeval ares_timeout_; |
| 227 | struct hostent hostent_; |
| 228 | int ares_result_; |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 229 | Error error_result_; |
| 230 | IPAddress address_result_; |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 231 | }; |
| 232 | |
| 233 | class SentinelIOHandler : public IOHandler { |
| 234 | public: |
| 235 | MOCK_METHOD0(Die, void()); |
| 236 | virtual ~SentinelIOHandler() { Die(); } |
| 237 | }; |
| 238 | |
| 239 | TEST_F(DNSClientTest, Constructor) { |
| 240 | vector<string> dns_servers; |
| 241 | dns_servers.push_back(kGoodServer); |
| 242 | CreateClient(dns_servers, kAresTimeoutMS); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 243 | ExpectReset(); |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 244 | } |
| 245 | |
| 246 | // Receive error because no DNS servers were specified. |
| 247 | TEST_F(DNSClientTest, NoServers) { |
| 248 | CreateClient(vector<string>(), kAresTimeoutMS); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 249 | Error error; |
| 250 | EXPECT_FALSE(dns_client_->Start(kGoodName, &error)); |
| 251 | EXPECT_EQ(Error::kInvalidArguments, error.type()); |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 252 | } |
| 253 | |
Peter Qiu | f3a8f90 | 2014-08-20 10:05:42 -0700 | [diff] [blame] | 254 | // Setup error because SetServersCsv failed due to invalid DNS servers. |
| 255 | TEST_F(DNSClientTest, SetServersCsvInvalidServer) { |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 256 | vector<string> dns_servers; |
| 257 | dns_servers.push_back(kBadServer); |
| 258 | CreateClient(dns_servers, kAresTimeoutMS); |
Peter Qiu | f3a8f90 | 2014-08-20 10:05:42 -0700 | [diff] [blame] | 259 | EXPECT_CALL(ares_, InitOptions(_, _, _)) |
| 260 | .WillOnce(Return(ARES_SUCCESS)); |
| 261 | EXPECT_CALL(ares_, SetServersCsv(_, _)) |
| 262 | .WillOnce(Return(ARES_EBADSTR)); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 263 | Error error; |
Peter Qiu | f3a8f90 | 2014-08-20 10:05:42 -0700 | [diff] [blame] | 264 | EXPECT_FALSE(dns_client_->Start(kGoodName, &error)); |
| 265 | EXPECT_EQ(Error::kOperationFailed, error.type()); |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 266 | } |
| 267 | |
| 268 | // Setup error because InitOptions failed. |
| 269 | TEST_F(DNSClientTest, InitOptionsFailure) { |
| 270 | vector<string> dns_servers; |
| 271 | dns_servers.push_back(kGoodServer); |
| 272 | CreateClient(dns_servers, kAresTimeoutMS); |
| 273 | EXPECT_CALL(ares_, InitOptions(_, _, _)) |
| 274 | .WillOnce(Return(ARES_EBADFLAGS)); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 275 | Error error; |
| 276 | EXPECT_FALSE(dns_client_->Start(kGoodName, &error)); |
| 277 | EXPECT_EQ(Error::kOperationFailed, error.type()); |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 278 | } |
| 279 | |
| 280 | // Fail a second request because one is already in progress. |
| 281 | TEST_F(DNSClientTest, MultipleRequest) { |
| 282 | StartValidRequest(); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 283 | Error error; |
| 284 | ASSERT_FALSE(dns_client_->Start(kGoodName, &error)); |
| 285 | EXPECT_EQ(Error::kInProgress, error.type()); |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 286 | } |
| 287 | |
| 288 | TEST_F(DNSClientTest, GoodRequest) { |
| 289 | StartValidRequest(); |
| 290 | TestValidCompletion(); |
| 291 | } |
| 292 | |
| 293 | TEST_F(DNSClientTest, GoodRequestWithTimeout) { |
| 294 | StartValidRequest(); |
| 295 | // Insert an intermediate HandleTimeout callback. |
| 296 | AdvanceTime(kAresWaitMS); |
| 297 | EXPECT_CALL(ares_, ProcessFd(kAresChannel, ARES_SOCKET_BAD, ARES_SOCKET_BAD)); |
| 298 | EXPECT_CALL(dispatcher_, PostDelayedTask(_, kAresWaitMS)); |
| 299 | CallTimeout(); |
| 300 | AdvanceTime(kAresWaitMS); |
| 301 | TestValidCompletion(); |
| 302 | } |
| 303 | |
| 304 | TEST_F(DNSClientTest, GoodRequestWithDNSRead) { |
| 305 | StartValidRequest(); |
| 306 | // Insert an intermediate HandleDNSRead callback. |
| 307 | AdvanceTime(kAresWaitMS); |
| 308 | EXPECT_CALL(ares_, ProcessFd(kAresChannel, kAresFd, ARES_SOCKET_BAD)); |
| 309 | EXPECT_CALL(dispatcher_, PostDelayedTask(_, kAresWaitMS)); |
| 310 | CallDNSRead(); |
| 311 | AdvanceTime(kAresWaitMS); |
| 312 | TestValidCompletion(); |
| 313 | } |
| 314 | |
| 315 | TEST_F(DNSClientTest, GoodRequestWithDNSWrite) { |
| 316 | StartValidRequest(); |
| 317 | // Insert an intermediate HandleDNSWrite callback. |
| 318 | AdvanceTime(kAresWaitMS); |
| 319 | EXPECT_CALL(ares_, ProcessFd(kAresChannel, ARES_SOCKET_BAD, kAresFd)); |
| 320 | EXPECT_CALL(dispatcher_, PostDelayedTask(_, kAresWaitMS)); |
| 321 | CallDNSWrite(); |
| 322 | AdvanceTime(kAresWaitMS); |
| 323 | TestValidCompletion(); |
| 324 | } |
| 325 | |
| 326 | // Failure due to the timeout occurring during first call to RefreshHandles. |
| 327 | TEST_F(DNSClientTest, TimeoutFirstRefresh) { |
| 328 | SetupRequest(kGoodName, kGoodServer); |
| 329 | struct timeval init_time_val = time_val_; |
| 330 | AdvanceTime(kAresTimeoutMS); |
Paul Stewart | e692740 | 2012-01-23 16:11:30 -0800 | [diff] [blame] | 331 | EXPECT_CALL(time_, GetTimeMonotonic(_)) |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 332 | .WillOnce(DoAll(SetArgumentPointee<0>(init_time_val), Return(0))) |
| 333 | .WillRepeatedly(DoAll(SetArgumentPointee<0>(time_val_), Return(0))); |
Ben Chan | be277dd | 2014-02-05 17:26:47 -0800 | [diff] [blame] | 334 | EXPECT_CALL(callback_target_, CallTarget(Not(IsSuccess()), _)) |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 335 | .Times(0); |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 336 | EXPECT_CALL(ares_, Destroy(kAresChannel)); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 337 | Error error; |
| 338 | // Expect the DNSClient to post a completion task. However this task will |
| 339 | // never run since the Stop() gets called before returning. We confirm |
| 340 | // that the task indeed gets canceled below in ExpectReset(). |
| 341 | ExpectPostCompletionTask(); |
| 342 | ASSERT_FALSE(dns_client_->Start(kGoodName, &error)); |
| 343 | EXPECT_EQ(Error::kOperationTimeout, error.type()); |
| 344 | EXPECT_EQ(string(DNSClient::kErrorTimedOut), error.message()); |
| 345 | ExpectReset(); |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 346 | } |
| 347 | |
| 348 | // Failed request due to timeout within the dns_client. |
| 349 | TEST_F(DNSClientTest, TimeoutDispatcherEvent) { |
| 350 | StartValidRequest(); |
| 351 | EXPECT_CALL(ares_, ProcessFd(kAresChannel, |
| 352 | ARES_SOCKET_BAD, ARES_SOCKET_BAD)); |
| 353 | AdvanceTime(kAresTimeoutMS); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 354 | ExpectPostCompletionTask(); |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 355 | CallTimeout(); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 356 | EXPECT_CALL(callback_target_, CallTarget( |
Ben Chan | be277dd | 2014-02-05 17:26:47 -0800 | [diff] [blame] | 357 | ErrorIs(Error::kOperationTimeout, DNSClient::kErrorTimedOut), _)); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 358 | CallCompletion(); |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 359 | } |
| 360 | |
| 361 | // Failed request due to timeout reported by ARES. |
| 362 | TEST_F(DNSClientTest, TimeoutFromARES) { |
| 363 | StartValidRequest(); |
| 364 | AdvanceTime(kAresWaitMS); |
| 365 | ares_result_ = ARES_ETIMEOUT; |
| 366 | EXPECT_CALL(ares_, ProcessFd(kAresChannel, ARES_SOCKET_BAD, ARES_SOCKET_BAD)) |
| 367 | .WillOnce(InvokeWithoutArgs(this, &DNSClientTest::CallReplyCB)); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 368 | ExpectPostCompletionTask(); |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 369 | CallTimeout(); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 370 | EXPECT_CALL(callback_target_, CallTarget( |
Ben Chan | be277dd | 2014-02-05 17:26:47 -0800 | [diff] [blame] | 371 | ErrorIs(Error::kOperationTimeout, DNSClient::kErrorTimedOut), _)); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 372 | CallCompletion(); |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 373 | } |
| 374 | |
| 375 | // Failed request due to "host not found" reported by ARES. |
| 376 | TEST_F(DNSClientTest, HostNotFound) { |
| 377 | StartValidRequest(); |
| 378 | AdvanceTime(kAresWaitMS); |
| 379 | ares_result_ = ARES_ENOTFOUND; |
| 380 | EXPECT_CALL(ares_, ProcessFd(kAresChannel, kAresFd, ARES_SOCKET_BAD)) |
| 381 | .WillOnce(InvokeWithoutArgs(this, &DNSClientTest::CallReplyCB)); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 382 | ExpectPostCompletionTask(); |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 383 | CallDNSRead(); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 384 | EXPECT_CALL(callback_target_, CallTarget( |
Ben Chan | be277dd | 2014-02-05 17:26:47 -0800 | [diff] [blame] | 385 | ErrorIs(Error::kOperationFailed, DNSClient::kErrorNotFound), _)); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 386 | CallCompletion(); |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 387 | } |
| 388 | |
| 389 | // Make sure IOHandles are deallocated when GetSock() reports them gone. |
| 390 | TEST_F(DNSClientTest, IOHandleDeallocGetSock) { |
| 391 | SetupRequest(kGoodName, kGoodServer); |
| 392 | // This isn't any kind of scoped/ref pointer because we are tracking dealloc. |
| 393 | SentinelIOHandler *io_handler = new SentinelIOHandler(); |
| 394 | EXPECT_CALL(dispatcher_, |
| 395 | CreateReadyHandler(kAresFd, IOHandler::kModeInput, _)) |
| 396 | .WillOnce(Return(io_handler)); |
| 397 | EXPECT_CALL(dispatcher_, PostDelayedTask(_, kAresWaitMS)); |
| 398 | SetActive(); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 399 | Error error; |
| 400 | ASSERT_TRUE(dns_client_->Start(kGoodName, &error)); |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 401 | AdvanceTime(kAresWaitMS); |
| 402 | SetInActive(); |
| 403 | EXPECT_CALL(*io_handler, Die()); |
| 404 | EXPECT_CALL(ares_, ProcessFd(kAresChannel, kAresFd, ARES_SOCKET_BAD)); |
| 405 | EXPECT_CALL(dispatcher_, PostDelayedTask(_, kAresWaitMS)); |
| 406 | CallDNSRead(); |
| 407 | EXPECT_CALL(ares_, Destroy(kAresChannel)); |
| 408 | } |
| 409 | |
| 410 | // Make sure IOHandles are deallocated when Stop() is called. |
| 411 | TEST_F(DNSClientTest, IOHandleDeallocStop) { |
| 412 | SetupRequest(kGoodName, kGoodServer); |
| 413 | // This isn't any kind of scoped/ref pointer because we are tracking dealloc. |
| 414 | SentinelIOHandler *io_handler = new SentinelIOHandler(); |
| 415 | EXPECT_CALL(dispatcher_, |
| 416 | CreateReadyHandler(kAresFd, IOHandler::kModeInput, _)) |
| 417 | .WillOnce(Return(io_handler)); |
| 418 | EXPECT_CALL(dispatcher_, PostDelayedTask(_, kAresWaitMS)); |
| 419 | SetActive(); |
Paul Stewart | bdb02e6 | 2012-02-22 16:24:33 -0800 | [diff] [blame] | 420 | Error error; |
| 421 | ASSERT_TRUE(dns_client_->Start(kGoodName, &error)); |
Paul Stewart | c2350ee | 2011-10-19 12:28:40 -0700 | [diff] [blame] | 422 | EXPECT_CALL(*io_handler, Die()); |
| 423 | EXPECT_CALL(ares_, Destroy(kAresChannel)); |
| 424 | dns_client_->Stop(); |
| 425 | } |
| 426 | |
| 427 | } // namespace shill |