blob: 434b57c26405bdd3f952fc7755281badca42d614 [file] [log] [blame]
mukesh agrawal6bb9e7c2012-01-30 14:57:54 -08001// Copyright (c) 2012 The Chromium OS Authors. All rights reserved.
Darin Petkove7cb7f82011-06-03 13:21:51 -07002// Use of this source code is governed by a BSD-style license that can be
3// found in the LICENSE file.
4
Chris Masone8a7b8be2011-07-22 12:43:37 -07005#include "shill/dhcp_config.h"
6
Eric Shienbrood3e20a232012-02-16 11:35:56 -05007#include <base/bind.h>
Darin Petkov92c43902011-06-09 20:46:06 -07008#include <base/file_util.h>
Eric Shienbrood3e20a232012-02-16 11:35:56 -05009#include <base/scoped_temp_dir.h>
Darin Petkov92c43902011-06-09 20:46:06 -070010#include <base/stringprintf.h>
Chris Masone43b48a12011-07-01 13:37:07 -070011#include <chromeos/dbus/service_constants.h>
Chris Masone8a7b8be2011-07-22 12:43:37 -070012#include <gtest/gtest.h>
Darin Petkov92c43902011-06-09 20:46:06 -070013
Chris Masone43b48a12011-07-01 13:37:07 -070014#include "shill/dbus_adaptor.h"
Darin Petkovf7897bc2011-06-08 17:13:36 -070015#include "shill/dhcp_provider.h"
Paul Stewart26b327e2011-10-19 11:38:09 -070016#include "shill/event_dispatcher.h"
Chris Masone19e30402011-07-19 15:48:47 -070017#include "shill/mock_control.h"
Darin Petkov98dd6a02011-06-10 15:12:57 -070018#include "shill/mock_dhcp_proxy.h"
Darin Petkovf7897bc2011-06-08 17:13:36 -070019#include "shill/mock_glib.h"
Jorge Lucangeli Obesad43cc62012-04-11 16:25:43 -070020#include "shill/mock_minijail.h"
Chris Masone43b48a12011-07-01 13:37:07 -070021#include "shill/property_store_unittest.h"
Darin Petkova7b89492011-07-27 12:48:17 -070022#include "shill/proxy_factory.h"
Darin Petkove7cb7f82011-06-03 13:21:51 -070023
Eric Shienbrood3e20a232012-02-16 11:35:56 -050024using base::Bind;
25using base::Unretained;
Darin Petkove7cb7f82011-06-03 13:21:51 -070026using std::string;
27using std::vector;
Darin Petkovf7897bc2011-06-08 17:13:36 -070028using testing::_;
29using testing::Return;
30using testing::SetArgumentPointee;
Darin Petkove7cb7f82011-06-03 13:21:51 -070031using testing::Test;
32
33namespace shill {
34
Darin Petkov92c43902011-06-09 20:46:06 -070035namespace {
Darin Petkova7b89492011-07-27 12:48:17 -070036const char kDeviceName[] = "eth0";
Paul Stewartd32f4842012-01-11 16:08:13 -080037const char kHostName[] = "hostname";
Paul Stewartd408fdf2012-05-07 17:15:57 -070038const char kLeaseFileSuffix[] = "leasefilesuffix";
39const bool kArpGateway = true;
Darin Petkov92c43902011-06-09 20:46:06 -070040} // namespace {}
41
Chris Masone43b48a12011-07-01 13:37:07 -070042class DHCPConfigTest : public PropertyStoreTest {
Darin Petkove7cb7f82011-06-03 13:21:51 -070043 public:
Darin Petkovf7897bc2011-06-08 17:13:36 -070044 DHCPConfigTest()
Darin Petkovf65e9282011-06-21 14:29:56 -070045 : proxy_(new MockDHCPProxy()),
Darin Petkova7b89492011-07-27 12:48:17 -070046 proxy_factory_(this),
Jorge Lucangeli Obesad43cc62012-04-11 16:25:43 -070047 minijail_(new MockMinijail()),
Chris Masone19e30402011-07-19 15:48:47 -070048 config_(new DHCPConfig(&control_,
Chris Masone2176a882011-09-14 22:29:15 -070049 dispatcher(),
Chris Masone19e30402011-07-19 15:48:47 -070050 DHCPProvider::GetInstance(),
Darin Petkov92c43902011-06-09 20:46:06 -070051 kDeviceName,
Paul Stewartd32f4842012-01-11 16:08:13 -080052 kHostName,
Paul Stewartd408fdf2012-05-07 17:15:57 -070053 kLeaseFileSuffix,
54 kArpGateway,
Chris Masone2176a882011-09-14 22:29:15 -070055 glib())) {}
Darin Petkova7b89492011-07-27 12:48:17 -070056
57 virtual void SetUp() {
Darin Petkovab565bb2011-10-06 02:55:51 -070058 config_->proxy_factory_ = &proxy_factory_;
Jorge Lucangeli Obesad43cc62012-04-11 16:25:43 -070059 config_->minijail_ = minijail_.get();
Darin Petkova7b89492011-07-27 12:48:17 -070060 }
61
62 virtual void TearDown() {
Darin Petkovab565bb2011-10-06 02:55:51 -070063 config_->proxy_factory_ = NULL;
Jorge Lucangeli Obesad43cc62012-04-11 16:25:43 -070064 config_->minijail_ = NULL;
Darin Petkov98dd6a02011-06-10 15:12:57 -070065 }
Darin Petkove7cb7f82011-06-03 13:21:51 -070066
Jorge Lucangeli Obesad43cc62012-04-11 16:25:43 -070067 DHCPConfigRefPtr CreateMockMinijailConfig(const string &hostname,
68 const string &lease_suffix,
69 bool arp_gateway);
Paul Stewartd408fdf2012-05-07 17:15:57 -070070 DHCPConfigRefPtr CreateRunningConfig(const string &hostname,
71 const string &lease_suffix,
72 bool arp_gateway);
73 void StopRunningConfigAndExpect(DHCPConfigRefPtr config,
74 bool lease_file_exists);
75
Darin Petkove7cb7f82011-06-03 13:21:51 -070076 protected:
Darin Petkova7b89492011-07-27 12:48:17 -070077 class TestProxyFactory : public ProxyFactory {
78 public:
Paul Stewart7355ce12011-09-02 10:47:01 -070079 explicit TestProxyFactory(DHCPConfigTest *test) : test_(test) {}
Darin Petkova7b89492011-07-27 12:48:17 -070080
mukesh agrawal1830fa12011-09-26 14:31:40 -070081 virtual DHCPProxyInterface *CreateDHCPProxy(const string &/*service*/) {
Darin Petkova7b89492011-07-27 12:48:17 -070082 return test_->proxy_.release();
83 }
84
85 private:
86 DHCPConfigTest *test_;
87 };
88
Paul Stewartd408fdf2012-05-07 17:15:57 -070089 static const int kPID;
90 static const unsigned int kTag;
91
92 FilePath lease_file_;
93 FilePath pid_file_;
94 ScopedTempDir temp_dir_;
Darin Petkova7b89492011-07-27 12:48:17 -070095 scoped_ptr<MockDHCPProxy> proxy_;
96 TestProxyFactory proxy_factory_;
Chris Masone19e30402011-07-19 15:48:47 -070097 MockControl control_;
Jorge Lucangeli Obesad43cc62012-04-11 16:25:43 -070098 scoped_ptr<MockMinijail> minijail_;
Darin Petkovf7897bc2011-06-08 17:13:36 -070099 DHCPConfigRefPtr config_;
Darin Petkove7cb7f82011-06-03 13:21:51 -0700100};
101
Paul Stewartd408fdf2012-05-07 17:15:57 -0700102const int DHCPConfigTest::kPID = 123456;
103const unsigned int DHCPConfigTest::kTag = 77;
104
Jorge Lucangeli Obesad43cc62012-04-11 16:25:43 -0700105DHCPConfigRefPtr DHCPConfigTest::CreateMockMinijailConfig(
106 const string &hostname,
107 const string &lease_suffix,
108 bool arp_gateway) {
109 DHCPConfigRefPtr config(new DHCPConfig(&control_,
110 dispatcher(),
111 DHCPProvider::GetInstance(),
112 kDeviceName,
113 hostname,
114 lease_suffix,
115 arp_gateway,
116 glib()));
117 config->minijail_ = minijail_.get();
118 EXPECT_CALL(*minijail_, RunAndDestroy(_, _, _)).WillOnce(Return(false));
119
120 return config;
121}
122
Paul Stewartd408fdf2012-05-07 17:15:57 -0700123DHCPConfigRefPtr DHCPConfigTest::CreateRunningConfig(const string &hostname,
124 const string &lease_suffix,
125 bool arp_gateway) {
126 DHCPConfigRefPtr config(new DHCPConfig(&control_,
127 dispatcher(),
128 DHCPProvider::GetInstance(),
129 kDeviceName,
130 hostname,
131 lease_suffix,
132 arp_gateway,
133 glib()));
Jorge Lucangeli Obesad43cc62012-04-11 16:25:43 -0700134 config->minijail_ = minijail_.get();
135 EXPECT_CALL(*minijail_, RunAndDestroy(_, _, _))
136 .WillOnce(DoAll(SetArgumentPointee<2>(kPID), Return(true)));
Paul Stewartd408fdf2012-05-07 17:15:57 -0700137 EXPECT_CALL(*glib(), ChildWatchAdd(kPID, _, _)).WillOnce(Return(kTag));
138 EXPECT_TRUE(config->Start());
139 EXPECT_EQ(kPID, config->pid_);
140 EXPECT_EQ(config.get(), DHCPProvider::GetInstance()->GetConfig(kPID).get());
141 EXPECT_EQ(kTag, config->child_watch_tag_);
142
143 EXPECT_TRUE(temp_dir_.CreateUniqueTempDir());
144 config->root_ = temp_dir_.path();
Jorge Lucangeli Obes2f3169d2012-04-25 11:38:25 -0700145 FilePath varrun = temp_dir_.path().Append("var/run/dhcpcd");
Paul Stewartd408fdf2012-05-07 17:15:57 -0700146 EXPECT_TRUE(file_util::CreateDirectory(varrun));
147 pid_file_ = varrun.Append(base::StringPrintf("dhcpcd-%s.pid", kDeviceName));
148 FilePath varlib = temp_dir_.path().Append("var/lib/dhcpcd");
149 EXPECT_TRUE(file_util::CreateDirectory(varlib));
150 lease_file_ =
151 varlib.Append(base::StringPrintf("dhcpcd-%s.lease", kDeviceName));
152 EXPECT_EQ(0, file_util::WriteFile(pid_file_, "", 0));
153 EXPECT_EQ(0, file_util::WriteFile(lease_file_, "", 0));
154 EXPECT_TRUE(file_util::PathExists(pid_file_));
155 EXPECT_TRUE(file_util::PathExists(lease_file_));
156
157 return config;
158}
159
160void DHCPConfigTest::StopRunningConfigAndExpect(DHCPConfigRefPtr config,
161 bool lease_file_exists) {
Paul Stewartd408fdf2012-05-07 17:15:57 -0700162 DHCPConfig::ChildWatchCallback(kPID, 0, config.get());
163 EXPECT_EQ(NULL, DHCPProvider::GetInstance()->GetConfig(kPID).get());
164
165 EXPECT_FALSE(file_util::PathExists(pid_file_));
166 EXPECT_EQ(lease_file_exists, file_util::PathExists(lease_file_));
167}
168
Darin Petkove7cb7f82011-06-03 13:21:51 -0700169TEST_F(DHCPConfigTest, GetIPv4AddressString) {
Darin Petkovf7897bc2011-06-08 17:13:36 -0700170 EXPECT_EQ("255.255.255.255", config_->GetIPv4AddressString(0xffffffff));
171 EXPECT_EQ("0.0.0.0", config_->GetIPv4AddressString(0));
172 EXPECT_EQ("1.2.3.4", config_->GetIPv4AddressString(0x04030201));
Darin Petkove7cb7f82011-06-03 13:21:51 -0700173}
174
Darin Petkova7b89492011-07-27 12:48:17 -0700175TEST_F(DHCPConfigTest, InitProxy) {
176 static const char kService[] = ":1.200";
Darin Petkova7b89492011-07-27 12:48:17 -0700177 EXPECT_TRUE(proxy_.get());
178 EXPECT_FALSE(config_->proxy_.get());
Eric Shienbrood9a245532012-03-07 14:20:39 -0500179 config_->InitProxy(kService);
Darin Petkova7b89492011-07-27 12:48:17 -0700180 EXPECT_FALSE(proxy_.get());
181 EXPECT_TRUE(config_->proxy_.get());
182
183 config_->InitProxy(kService);
Darin Petkova7b89492011-07-27 12:48:17 -0700184}
185
Darin Petkove7cb7f82011-06-03 13:21:51 -0700186TEST_F(DHCPConfigTest, ParseConfiguration) {
187 DHCPConfig::Configuration conf;
188 conf[DHCPConfig::kConfigurationKeyIPAddress].writer().append_uint32(
189 0x01020304);
190 conf[DHCPConfig::kConfigurationKeySubnetCIDR].writer().append_byte(
191 16);
192 conf[DHCPConfig::kConfigurationKeyBroadcastAddress].writer().append_uint32(
193 0x10203040);
194 {
Darin Petkove7cb7f82011-06-03 13:21:51 -0700195 vector<unsigned int> routers;
196 routers.push_back(0x02040608);
197 routers.push_back(0x03050709);
Darin Petkovf7897bc2011-06-08 17:13:36 -0700198 DBus::MessageIter writer =
199 conf[DHCPConfig::kConfigurationKeyRouters].writer();
Darin Petkove7cb7f82011-06-03 13:21:51 -0700200 writer << routers;
Darin Petkove7cb7f82011-06-03 13:21:51 -0700201 }
202 {
Darin Petkove7cb7f82011-06-03 13:21:51 -0700203 vector<unsigned int> dns;
204 dns.push_back(0x09070503);
205 dns.push_back(0x08060402);
Darin Petkovf7897bc2011-06-08 17:13:36 -0700206 DBus::MessageIter writer = conf[DHCPConfig::kConfigurationKeyDNS].writer();
Darin Petkove7cb7f82011-06-03 13:21:51 -0700207 writer << dns;
Darin Petkove7cb7f82011-06-03 13:21:51 -0700208 }
209 conf[DHCPConfig::kConfigurationKeyDomainName].writer().append_string(
210 "domain-name");
211 {
Darin Petkove7cb7f82011-06-03 13:21:51 -0700212 vector<string> search;
213 search.push_back("foo.com");
214 search.push_back("bar.com");
Darin Petkovf7897bc2011-06-08 17:13:36 -0700215 DBus::MessageIter writer =
216 conf[DHCPConfig::kConfigurationKeyDomainSearch].writer();
Darin Petkove7cb7f82011-06-03 13:21:51 -0700217 writer << search;
Darin Petkove7cb7f82011-06-03 13:21:51 -0700218 }
219 conf[DHCPConfig::kConfigurationKeyMTU].writer().append_uint16(600);
220 conf["UnknownKey"] = DBus::Variant();
221
Darin Petkove7cb7f82011-06-03 13:21:51 -0700222 IPConfig::Properties properties;
Darin Petkovf7897bc2011-06-08 17:13:36 -0700223 ASSERT_TRUE(config_->ParseConfiguration(conf, &properties));
Darin Petkove7cb7f82011-06-03 13:21:51 -0700224 EXPECT_EQ("4.3.2.1", properties.address);
Paul Stewart48100b02012-03-19 07:53:52 -0700225 EXPECT_EQ(16, properties.subnet_prefix);
Darin Petkove7cb7f82011-06-03 13:21:51 -0700226 EXPECT_EQ("64.48.32.16", properties.broadcast_address);
227 EXPECT_EQ("8.6.4.2", properties.gateway);
228 ASSERT_EQ(2, properties.dns_servers.size());
229 EXPECT_EQ("3.5.7.9", properties.dns_servers[0]);
230 EXPECT_EQ("2.4.6.8", properties.dns_servers[1]);
231 EXPECT_EQ("domain-name", properties.domain_name);
232 ASSERT_EQ(2, properties.domain_search.size());
233 EXPECT_EQ("foo.com", properties.domain_search[0]);
234 EXPECT_EQ("bar.com", properties.domain_search[1]);
235 EXPECT_EQ(600, properties.mtu);
236}
237
Darin Petkov92c43902011-06-09 20:46:06 -0700238TEST_F(DHCPConfigTest, StartFail) {
Jorge Lucangeli Obesad43cc62012-04-11 16:25:43 -0700239 EXPECT_CALL(*minijail_, RunAndDestroy(_, _, _)).WillOnce(Return(false));
Chris Masone2176a882011-09-14 22:29:15 -0700240 EXPECT_CALL(*glib(), ChildWatchAdd(_, _, _)).Times(0);
Darin Petkovf7897bc2011-06-08 17:13:36 -0700241 EXPECT_FALSE(config_->Start());
242 EXPECT_EQ(0, config_->pid_);
Darin Petkov92c43902011-06-09 20:46:06 -0700243}
Darin Petkovf7897bc2011-06-08 17:13:36 -0700244
Paul Stewartd408fdf2012-05-07 17:15:57 -0700245MATCHER_P3(IsDHCPCDArgs, has_hostname, has_arp_gateway, has_lease_suffix, "") {
Paul Stewartd32f4842012-01-11 16:08:13 -0800246 if (string(arg[0]) != "/sbin/dhcpcd" ||
Paul Stewartd408fdf2012-05-07 17:15:57 -0700247 string(arg[1]) != "-B") {
Paul Stewartd32f4842012-01-11 16:08:13 -0800248 return false;
249 }
250
Paul Stewartd408fdf2012-05-07 17:15:57 -0700251 int end_offset = 2;
Paul Stewartd32f4842012-01-11 16:08:13 -0800252 if (has_hostname) {
Paul Stewartd408fdf2012-05-07 17:15:57 -0700253 if (string(arg[end_offset]) != "-h" ||
254 string(arg[end_offset + 1]) != kHostName) {
Paul Stewartd32f4842012-01-11 16:08:13 -0800255 return false;
256 }
Paul Stewartd408fdf2012-05-07 17:15:57 -0700257 end_offset += 2;
Paul Stewartd32f4842012-01-11 16:08:13 -0800258 }
259
Paul Stewartd408fdf2012-05-07 17:15:57 -0700260 if (has_arp_gateway) {
261 if (string(arg[end_offset]) != "-R")
262 return false;
263 ++end_offset;
264 }
265
266 string device_arg = has_lease_suffix ?
267 string(kDeviceName) + "=" + string(kLeaseFileSuffix) : kDeviceName;
268 return string(arg[end_offset]) == device_arg && arg[end_offset + 1] == NULL;
Paul Stewartd32f4842012-01-11 16:08:13 -0800269}
270
271TEST_F(DHCPConfigTest, StartWithHostname) {
Jorge Lucangeli Obesad43cc62012-04-11 16:25:43 -0700272 EXPECT_CALL(*minijail_, RunAndDestroy(_, _, _)).WillOnce(Return(false));
Paul Stewartd32f4842012-01-11 16:08:13 -0800273 EXPECT_FALSE(config_->Start());
274}
275
276TEST_F(DHCPConfigTest, StartWithoutHostname) {
Jorge Lucangeli Obesad43cc62012-04-11 16:25:43 -0700277 DHCPConfigRefPtr config = CreateMockMinijailConfig("",
278 kLeaseFileSuffix,
279 kArpGateway);
Paul Stewartd408fdf2012-05-07 17:15:57 -0700280 EXPECT_FALSE(config->Start());
281}
282
283TEST_F(DHCPConfigTest, StartWithoutArpGateway) {
Jorge Lucangeli Obesad43cc62012-04-11 16:25:43 -0700284 DHCPConfigRefPtr config = CreateMockMinijailConfig(kHostName,
285 kLeaseFileSuffix,
286 false);
Paul Stewartd408fdf2012-05-07 17:15:57 -0700287 EXPECT_FALSE(config->Start());
288}
289
290TEST_F(DHCPConfigTest, StartWithoutLeaseSuffix) {
Jorge Lucangeli Obesad43cc62012-04-11 16:25:43 -0700291 DHCPConfigRefPtr config = CreateMockMinijailConfig(kHostName,
292 kDeviceName,
293 kArpGateway);
Paul Stewartd32f4842012-01-11 16:08:13 -0800294 EXPECT_FALSE(config->Start());
295}
296
Darin Petkovf9b0ca82011-06-20 12:10:23 -0700297namespace {
298
299class UpdateCallbackTest {
300 public:
301 UpdateCallbackTest(const string &message,
Chris Masone2b105542011-06-22 10:58:09 -0700302 const IPConfigRefPtr &ipconfig,
Darin Petkovf9b0ca82011-06-20 12:10:23 -0700303 bool success)
304 : message_(message),
305 ipconfig_(ipconfig),
306 success_(success),
307 called_(false) {}
308
Chris Masone2b105542011-06-22 10:58:09 -0700309 void Callback(const IPConfigRefPtr &ipconfig, bool success) {
Darin Petkovf9b0ca82011-06-20 12:10:23 -0700310 called_ = true;
311 EXPECT_EQ(ipconfig_.get(), ipconfig.get()) << message_;
312 EXPECT_EQ(success_, success) << message_;
313 }
314
315 bool called() const { return called_; }
316
317 private:
318 const string message_;
319 IPConfigRefPtr ipconfig_;
320 bool success_;
321 bool called_;
322};
323
mukesh agrawalcc0fded2012-05-09 13:40:58 -0700324void DoNothing() {}
325
Darin Petkovf9b0ca82011-06-20 12:10:23 -0700326} // namespace {}
327
328TEST_F(DHCPConfigTest, ProcessEventSignalFail) {
329 DHCPConfig::Configuration conf;
330 conf[DHCPConfig::kConfigurationKeyIPAddress].writer().append_uint32(
331 0x01020304);
332 UpdateCallbackTest callback_test(DHCPConfig::kReasonFail, config_, false);
333 config_->RegisterUpdateCallback(
Eric Shienbrood3e20a232012-02-16 11:35:56 -0500334 Bind(&UpdateCallbackTest::Callback, Unretained(&callback_test)));
mukesh agrawalcc0fded2012-05-09 13:40:58 -0700335 config_->lease_acquisition_timeout_callback_.Reset(base::Bind(&DoNothing));
Darin Petkovf9b0ca82011-06-20 12:10:23 -0700336 config_->ProcessEventSignal(DHCPConfig::kReasonFail, conf);
337 EXPECT_TRUE(callback_test.called());
338 EXPECT_TRUE(config_->properties().address.empty());
mukesh agrawalcc0fded2012-05-09 13:40:58 -0700339 EXPECT_TRUE(config_->lease_acquisition_timeout_callback_.IsCancelled());
Darin Petkovf9b0ca82011-06-20 12:10:23 -0700340}
341
342TEST_F(DHCPConfigTest, ProcessEventSignalSuccess) {
343 static const char * const kReasons[] = {
344 DHCPConfig::kReasonBound,
345 DHCPConfig::kReasonRebind,
346 DHCPConfig::kReasonReboot,
347 DHCPConfig::kReasonRenew
348 };
349 for (size_t r = 0; r < arraysize(kReasons); r++) {
350 DHCPConfig::Configuration conf;
351 string message = string(kReasons[r]) + " failed";
352 conf[DHCPConfig::kConfigurationKeyIPAddress].writer().append_uint32(r);
353 UpdateCallbackTest callback_test(message, config_, true);
354 config_->RegisterUpdateCallback(
Eric Shienbrood3e20a232012-02-16 11:35:56 -0500355 Bind(&UpdateCallbackTest::Callback, Unretained(&callback_test)));
mukesh agrawalcc0fded2012-05-09 13:40:58 -0700356 config_->lease_acquisition_timeout_callback_.Reset(base::Bind(&DoNothing));
Darin Petkovf9b0ca82011-06-20 12:10:23 -0700357 config_->ProcessEventSignal(kReasons[r], conf);
358 EXPECT_TRUE(callback_test.called()) << message;
Paul Stewart83224402011-11-30 14:52:30 -0800359 EXPECT_EQ(base::StringPrintf("%zu.0.0.0", r), config_->properties().address)
Darin Petkovf9b0ca82011-06-20 12:10:23 -0700360 << message;
mukesh agrawalcc0fded2012-05-09 13:40:58 -0700361 EXPECT_TRUE(config_->lease_acquisition_timeout_callback_.IsCancelled());
Darin Petkovf9b0ca82011-06-20 12:10:23 -0700362 }
363}
364
365TEST_F(DHCPConfigTest, ProcessEventSignalUnknown) {
366 DHCPConfig::Configuration conf;
367 conf[DHCPConfig::kConfigurationKeyIPAddress].writer().append_uint32(
368 0x01020304);
369 static const char kReasonUnknown[] = "UNKNOWN_REASON";
370 UpdateCallbackTest callback_test(kReasonUnknown, config_, false);
371 config_->RegisterUpdateCallback(
Eric Shienbrood3e20a232012-02-16 11:35:56 -0500372 Bind(&UpdateCallbackTest::Callback, Unretained(&callback_test)));
mukesh agrawalcc0fded2012-05-09 13:40:58 -0700373 config_->lease_acquisition_timeout_callback_.Reset(base::Bind(&DoNothing));
Darin Petkovf9b0ca82011-06-20 12:10:23 -0700374 config_->ProcessEventSignal(kReasonUnknown, conf);
375 EXPECT_FALSE(callback_test.called());
376 EXPECT_TRUE(config_->properties().address.empty());
mukesh agrawalcc0fded2012-05-09 13:40:58 -0700377 EXPECT_FALSE(config_->lease_acquisition_timeout_callback_.IsCancelled());
Darin Petkovf9b0ca82011-06-20 12:10:23 -0700378}
379
380
Darin Petkov98dd6a02011-06-10 15:12:57 -0700381TEST_F(DHCPConfigTest, ReleaseIP) {
382 config_->pid_ = 1 << 18; // Ensure unknown positive PID.
Darin Petkovaceede32011-07-18 15:32:38 -0700383 EXPECT_CALL(*proxy_, Release(kDeviceName)).Times(1);
Darin Petkova7b89492011-07-27 12:48:17 -0700384 config_->proxy_.reset(proxy_.release());
Darin Petkov98dd6a02011-06-10 15:12:57 -0700385 EXPECT_TRUE(config_->ReleaseIP());
386 config_->pid_ = 0;
387}
388
389TEST_F(DHCPConfigTest, RenewIP) {
mukesh agrawalcc0fded2012-05-09 13:40:58 -0700390 EXPECT_TRUE(config_->lease_acquisition_timeout_callback_.IsCancelled());
Darin Petkov98dd6a02011-06-10 15:12:57 -0700391 config_->pid_ = 456;
Darin Petkovaceede32011-07-18 15:32:38 -0700392 EXPECT_CALL(*proxy_, Rebind(kDeviceName)).Times(1);
Darin Petkova7b89492011-07-27 12:48:17 -0700393 config_->proxy_.reset(proxy_.release());
Darin Petkov98dd6a02011-06-10 15:12:57 -0700394 EXPECT_TRUE(config_->RenewIP());
mukesh agrawalcc0fded2012-05-09 13:40:58 -0700395 EXPECT_FALSE(config_->lease_acquisition_timeout_callback_.IsCancelled());
Darin Petkov98dd6a02011-06-10 15:12:57 -0700396 config_->pid_ = 0;
397}
398
399TEST_F(DHCPConfigTest, RequestIP) {
mukesh agrawalcc0fded2012-05-09 13:40:58 -0700400 EXPECT_TRUE(config_->lease_acquisition_timeout_callback_.IsCancelled());
Darin Petkov98dd6a02011-06-10 15:12:57 -0700401 config_->pid_ = 567;
Darin Petkovaceede32011-07-18 15:32:38 -0700402 EXPECT_CALL(*proxy_, Rebind(kDeviceName)).Times(1);
Darin Petkova7b89492011-07-27 12:48:17 -0700403 config_->proxy_.reset(proxy_.release());
Darin Petkov98dd6a02011-06-10 15:12:57 -0700404 EXPECT_TRUE(config_->RenewIP());
mukesh agrawalcc0fded2012-05-09 13:40:58 -0700405 EXPECT_FALSE(config_->lease_acquisition_timeout_callback_.IsCancelled());
406 config_->pid_ = 0;
407}
408
409TEST_F(DHCPConfigTest, RequestIPTimeout) {
410 UpdateCallbackTest callback_test(DHCPConfig::kReasonFail, config_, false);
411 config_->RegisterUpdateCallback(
412 Bind(&UpdateCallbackTest::Callback, Unretained(&callback_test)));
413 config_->lease_acquisition_timeout_seconds_ = 0;
414 config_->pid_ = 567;
415 EXPECT_CALL(*proxy_, Rebind(kDeviceName)).Times(1);
416 config_->proxy_.reset(proxy_.release());
417 config_->RenewIP();
418 config_->dispatcher_->DispatchPendingEvents();
419 EXPECT_TRUE(callback_test.called());
Darin Petkov98dd6a02011-06-10 15:12:57 -0700420 config_->pid_ = 0;
421}
422
423TEST_F(DHCPConfigTest, Restart) {
424 const int kPID1 = 1 << 17; // Ensure unknown positive PID.
425 const int kPID2 = 987;
426 const unsigned int kTag1 = 11;
427 const unsigned int kTag2 = 22;
428 config_->pid_ = kPID1;
429 config_->child_watch_tag_ = kTag1;
430 DHCPProvider::GetInstance()->BindPID(kPID1, config_);
Chris Masone2176a882011-09-14 22:29:15 -0700431 EXPECT_CALL(*glib(), SourceRemove(kTag1)).WillOnce(Return(true));
Jorge Lucangeli Obesad43cc62012-04-11 16:25:43 -0700432 EXPECT_CALL(*minijail_, RunAndDestroy(_, _, _)).WillOnce(
433 DoAll(SetArgumentPointee<2>(kPID2), Return(true)));
Chris Masone2176a882011-09-14 22:29:15 -0700434 EXPECT_CALL(*glib(), ChildWatchAdd(kPID2, _, _)).WillOnce(Return(kTag2));
Darin Petkov98dd6a02011-06-10 15:12:57 -0700435 EXPECT_TRUE(config_->Restart());
436 EXPECT_EQ(kPID2, config_->pid_);
437 EXPECT_EQ(config_.get(), DHCPProvider::GetInstance()->GetConfig(kPID2).get());
438 EXPECT_EQ(kTag2, config_->child_watch_tag_);
439 DHCPProvider::GetInstance()->UnbindPID(kPID2);
440 config_->pid_ = 0;
441 config_->child_watch_tag_ = 0;
442}
443
444TEST_F(DHCPConfigTest, RestartNoClient) {
445 const int kPID = 777;
446 const unsigned int kTag = 66;
Chris Masone2176a882011-09-14 22:29:15 -0700447 EXPECT_CALL(*glib(), SourceRemove(_)).Times(0);
Jorge Lucangeli Obesad43cc62012-04-11 16:25:43 -0700448 EXPECT_CALL(*minijail_, RunAndDestroy(_, _, _)).WillOnce(
449 DoAll(SetArgumentPointee<2>(kPID), Return(true)));
Chris Masone2176a882011-09-14 22:29:15 -0700450 EXPECT_CALL(*glib(), ChildWatchAdd(kPID, _, _)).WillOnce(Return(kTag));
Darin Petkov98dd6a02011-06-10 15:12:57 -0700451 EXPECT_TRUE(config_->Restart());
452 EXPECT_EQ(kPID, config_->pid_);
453 EXPECT_EQ(config_.get(), DHCPProvider::GetInstance()->GetConfig(kPID).get());
454 EXPECT_EQ(kTag, config_->child_watch_tag_);
455 DHCPProvider::GetInstance()->UnbindPID(kPID);
456 config_->pid_ = 0;
457 config_->child_watch_tag_ = 0;
458}
459
Paul Stewartd408fdf2012-05-07 17:15:57 -0700460TEST_F(DHCPConfigTest, StartSuccessEphemeral) {
461 DHCPConfigRefPtr config =
462 CreateRunningConfig(kHostName, kDeviceName, kArpGateway);
463 StopRunningConfigAndExpect(config, false);
464}
Darin Petkov92c43902011-06-09 20:46:06 -0700465
Paul Stewartd408fdf2012-05-07 17:15:57 -0700466TEST_F(DHCPConfigTest, StartSuccessPersistent) {
467 DHCPConfigRefPtr config =
468 CreateRunningConfig(kHostName, kLeaseFileSuffix, kArpGateway);
469 StopRunningConfigAndExpect(config, true);
Darin Petkovf7897bc2011-06-08 17:13:36 -0700470}
471
mukesh agrawalcc0fded2012-05-09 13:40:58 -0700472TEST_F(DHCPConfigTest, StartTimeout) {
473 UpdateCallbackTest callback_test(DHCPConfig::kReasonFail, config_, false);
474 config_->RegisterUpdateCallback(
475 Bind(&UpdateCallbackTest::Callback, Unretained(&callback_test)));
476 config_->lease_acquisition_timeout_seconds_ = 0;
477 config_->proxy_.reset(proxy_.release());
Jorge Lucangeli Obesad43cc62012-04-11 16:25:43 -0700478 EXPECT_CALL(*minijail_, RunAndDestroy(_, _, _)).WillOnce(Return(true));
mukesh agrawalcc0fded2012-05-09 13:40:58 -0700479 config_->Start();
480 config_->dispatcher_->DispatchPendingEvents();
481 EXPECT_TRUE(callback_test.called());
482}
483
Darin Petkov98dd6a02011-06-10 15:12:57 -0700484TEST_F(DHCPConfigTest, Stop) {
485 // Ensure no crashes.
486 const int kPID = 1 << 17; // Ensure unknown positive PID.
Darin Petkov98dd6a02011-06-10 15:12:57 -0700487 config_->pid_ = kPID;
488 config_->Stop();
mukesh agrawalcc0fded2012-05-09 13:40:58 -0700489 EXPECT_TRUE(config_->lease_acquisition_timeout_callback_.IsCancelled());
490}
491
492TEST_F(DHCPConfigTest, StopDuringRequestIP) {
493 config_->pid_ = 567;
494 EXPECT_CALL(*proxy_, Rebind(kDeviceName)).Times(1);
495 config_->proxy_.reset(proxy_.release());
496 EXPECT_TRUE(config_->RenewIP());
497 EXPECT_FALSE(config_->lease_acquisition_timeout_callback_.IsCancelled());
498 config_->pid_ = 0; // Keep Stop from killing a real process.
499 config_->Stop();
500 EXPECT_TRUE(config_->lease_acquisition_timeout_callback_.IsCancelled());
Darin Petkov98dd6a02011-06-10 15:12:57 -0700501}
502
mukesh agrawal6bb9e7c2012-01-30 14:57:54 -0800503TEST_F(DHCPConfigTest, SetProperty) {
Chris Masone43b48a12011-07-01 13:37:07 -0700504 ::DBus::Error error;
505 // Ensure that an attempt to write a R/O property returns InvalidArgs error.
mukesh agrawal6bb9e7c2012-01-30 14:57:54 -0800506 EXPECT_FALSE(DBusAdaptor::SetProperty(config_->mutable_store(),
507 flimflam::kAddressProperty,
508 PropertyStoreTest::kStringV,
509 &error));
Chris Masone9d779932011-08-25 16:33:41 -0700510 EXPECT_EQ(invalid_args(), error.name());
Chris Masone43b48a12011-07-01 13:37:07 -0700511}
512
Darin Petkove7cb7f82011-06-03 13:21:51 -0700513} // namespace shill