blob: eb0c9b6575f7b1ca0c89ebc8d18b659ad5eec4cc [file] [log] [blame]
Paul Stewartd8ad3c42012-01-09 12:39:38 -08001// Copyright (c) 2012 The Chromium OS Authors. All rights reserved.
Chris Masone3bd3c8c2011-06-13 08:20:26 -07002// 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/service.h"
6
7#include <map>
8#include <string>
9#include <vector>
10
Chris Masone3bd3c8c2011-06-13 08:20:26 -070011#include <chromeos/dbus/service_constants.h>
mukesh agrawal1830fa12011-09-26 14:31:40 -070012#include <dbus-c++/dbus.h>
Chris Masone3bd3c8c2011-06-13 08:20:26 -070013#include <gtest/gtest.h>
14#include <gmock/gmock.h>
15
16#include "shill/dbus_adaptor.h"
17#include "shill/ethernet_service.h"
Paul Stewart26b327e2011-10-19 11:38:09 -070018#include "shill/event_dispatcher.h"
Chris Masone3bd3c8c2011-06-13 08:20:26 -070019#include "shill/manager.h"
Chris Masone95207da2011-06-29 16:50:49 -070020#include "shill/mock_adaptors.h"
Chris Masone3bd3c8c2011-06-13 08:20:26 -070021#include "shill/mock_control.h"
Paul Stewart03dba0b2011-08-22 16:32:45 -070022#include "shill/mock_manager.h"
Darin Petkovba40dd32011-07-11 20:06:39 -070023#include "shill/mock_store.h"
Chris Masone3bd3c8c2011-06-13 08:20:26 -070024#include "shill/property_store_unittest.h"
Chris Masone6515aab2011-10-12 16:19:09 -070025#include "shill/service_under_test.h"
Chris Masone3bd3c8c2011-06-13 08:20:26 -070026
27using std::map;
28using std::string;
29using std::vector;
Darin Petkovba40dd32011-07-11 20:06:39 -070030using testing::_;
31using testing::AtLeast;
Paul Stewartd8ad3c42012-01-09 12:39:38 -080032using testing::DoAll;
Darin Petkovba40dd32011-07-11 20:06:39 -070033using testing::NiceMock;
34using testing::Return;
35using testing::StrictMock;
Paul Stewartd8ad3c42012-01-09 12:39:38 -080036using testing::SetArgumentPointee;
Darin Petkovba40dd32011-07-11 20:06:39 -070037using testing::Test;
Chris Masone3bd3c8c2011-06-13 08:20:26 -070038
39namespace shill {
40
41class ServiceTest : public PropertyStoreTest {
42 public:
Chris Masoneb925cc82011-06-22 15:39:57 -070043 ServiceTest()
Thieu Le3426c8f2012-01-11 17:35:11 -080044 : mock_manager_(control_interface(), dispatcher(), metrics(), glib()),
Chris Masone2176a882011-09-14 22:29:15 -070045 service_(new ServiceUnderTest(control_interface(),
46 dispatcher(),
Thieu Le3426c8f2012-01-11 17:35:11 -080047 metrics(),
Paul Stewart03dba0b2011-08-22 16:32:45 -070048 &mock_manager_)),
Chris Masone9d779932011-08-25 16:33:41 -070049 storage_id_(ServiceUnderTest::kStorageId) {
50 }
Chris Masoneb925cc82011-06-22 15:39:57 -070051
Chris Masone3bd3c8c2011-06-13 08:20:26 -070052 virtual ~ServiceTest() {}
Chris Masoneb925cc82011-06-22 15:39:57 -070053
54 protected:
Paul Stewart03dba0b2011-08-22 16:32:45 -070055 MockManager mock_manager_;
56 scoped_refptr<ServiceUnderTest> service_;
Chris Masone34af2182011-08-22 11:59:36 -070057 string storage_id_;
Chris Masone3bd3c8c2011-06-13 08:20:26 -070058};
59
Darin Petkovba40dd32011-07-11 20:06:39 -070060TEST_F(ServiceTest, Constructor) {
61 EXPECT_TRUE(service_->save_credentials_);
62 EXPECT_EQ(Service::kCheckPortalAuto, service_->check_portal_);
63}
64
Chris Masonea8a2c252011-06-27 22:16:30 -070065TEST_F(ServiceTest, GetProperties) {
66 map<string, ::DBus::Variant> props;
67 Error error(Error::kInvalidProperty, "");
68 {
69 ::DBus::Error dbus_error;
70 string expected("portal_list");
mukesh agrawalde29fa82011-09-16 16:16:36 -070071 service_->mutable_store()->SetStringProperty(flimflam::kCheckPortalProperty,
72 expected,
73 &error);
Chris Masone27c4aa52011-07-02 13:10:14 -070074 DBusAdaptor::GetProperties(service_->store(), &props, &dbus_error);
Chris Masonea8a2c252011-06-27 22:16:30 -070075 ASSERT_FALSE(props.find(flimflam::kCheckPortalProperty) == props.end());
76 EXPECT_EQ(props[flimflam::kCheckPortalProperty].reader().get_string(),
77 expected);
78 }
79 {
80 ::DBus::Error dbus_error;
81 bool expected = true;
mukesh agrawalde29fa82011-09-16 16:16:36 -070082 service_->mutable_store()->SetBoolProperty(flimflam::kAutoConnectProperty,
83 expected,
84 &error);
Chris Masone27c4aa52011-07-02 13:10:14 -070085 DBusAdaptor::GetProperties(service_->store(), &props, &dbus_error);
Chris Masonea8a2c252011-06-27 22:16:30 -070086 ASSERT_FALSE(props.find(flimflam::kAutoConnectProperty) == props.end());
87 EXPECT_EQ(props[flimflam::kAutoConnectProperty].reader().get_bool(),
88 expected);
89 }
90 {
91 ::DBus::Error dbus_error;
Chris Masone27c4aa52011-07-02 13:10:14 -070092 DBusAdaptor::GetProperties(service_->store(), &props, &dbus_error);
Chris Masonea8a2c252011-06-27 22:16:30 -070093 ASSERT_FALSE(props.find(flimflam::kConnectableProperty) == props.end());
94 EXPECT_EQ(props[flimflam::kConnectableProperty].reader().get_bool(), false);
95 }
96 {
97 ::DBus::Error dbus_error;
98 int32 expected = 127;
mukesh agrawalde29fa82011-09-16 16:16:36 -070099 service_->mutable_store()->SetInt32Property(flimflam::kPriorityProperty,
100 expected,
101 &error);
Chris Masone27c4aa52011-07-02 13:10:14 -0700102 DBusAdaptor::GetProperties(service_->store(), &props, &dbus_error);
Chris Masonea8a2c252011-06-27 22:16:30 -0700103 ASSERT_FALSE(props.find(flimflam::kPriorityProperty) == props.end());
104 EXPECT_EQ(props[flimflam::kPriorityProperty].reader().get_int32(),
105 expected);
106 }
Chris Masone95207da2011-06-29 16:50:49 -0700107 {
108 ::DBus::Error dbus_error;
Chris Masone27c4aa52011-07-02 13:10:14 -0700109 DBusAdaptor::GetProperties(service_->store(), &props, &dbus_error);
Chris Masone95207da2011-06-29 16:50:49 -0700110 ASSERT_FALSE(props.find(flimflam::kDeviceProperty) == props.end());
111 EXPECT_EQ(props[flimflam::kDeviceProperty].reader().get_string(),
Paul Stewart03dba0b2011-08-22 16:32:45 -0700112 string(ServiceUnderTest::kRpcId));
Chris Masone95207da2011-06-29 16:50:49 -0700113 }
Chris Masonea8a2c252011-06-27 22:16:30 -0700114}
Chris Masone3bd3c8c2011-06-13 08:20:26 -0700115
Chris Masonea8a2c252011-06-27 22:16:30 -0700116TEST_F(ServiceTest, Dispatch) {
117 {
118 ::DBus::Error error;
mukesh agrawalde29fa82011-09-16 16:16:36 -0700119 EXPECT_TRUE(DBusAdaptor::DispatchOnType(service_->mutable_store(),
Chris Masonea8a2c252011-06-27 22:16:30 -0700120 flimflam::kSaveCredentialsProperty,
121 PropertyStoreTest::kBoolV,
122 &error));
123 }
124 {
125 ::DBus::Error error;
mukesh agrawalde29fa82011-09-16 16:16:36 -0700126 EXPECT_TRUE(DBusAdaptor::DispatchOnType(service_->mutable_store(),
Chris Masonea8a2c252011-06-27 22:16:30 -0700127 flimflam::kPriorityProperty,
128 PropertyStoreTest::kInt32V,
129 &error));
130 }
131 {
132 ::DBus::Error error;
mukesh agrawalde29fa82011-09-16 16:16:36 -0700133 EXPECT_TRUE(DBusAdaptor::DispatchOnType(service_->mutable_store(),
Chris Masonea8a2c252011-06-27 22:16:30 -0700134 flimflam::kEAPEAPProperty,
135 PropertyStoreTest::kStringV,
136 &error));
137 }
Chris Masoneb925cc82011-06-22 15:39:57 -0700138 // Ensure that an attempt to write a R/O property returns InvalidArgs error.
Chris Masonea8a2c252011-06-27 22:16:30 -0700139 {
140 ::DBus::Error error;
mukesh agrawalde29fa82011-09-16 16:16:36 -0700141 EXPECT_FALSE(DBusAdaptor::DispatchOnType(service_->mutable_store(),
Chris Masonea8a2c252011-06-27 22:16:30 -0700142 flimflam::kFavoriteProperty,
143 PropertyStoreTest::kBoolV,
144 &error));
Chris Masone9d779932011-08-25 16:33:41 -0700145 EXPECT_EQ(invalid_args(), error.name());
Chris Masonea8a2c252011-06-27 22:16:30 -0700146 }
Chris Masoneb925cc82011-06-22 15:39:57 -0700147}
148
Darin Petkovba40dd32011-07-11 20:06:39 -0700149TEST_F(ServiceTest, Load) {
150 NiceMock<MockStore> storage;
Chris Masone34af2182011-08-22 11:59:36 -0700151 EXPECT_CALL(storage, ContainsGroup(storage_id_)).WillOnce(Return(true));
152 EXPECT_CALL(storage, GetString(storage_id_, _, _))
Darin Petkovba40dd32011-07-11 20:06:39 -0700153 .Times(AtLeast(1))
154 .WillRepeatedly(Return(true));
Chris Masone9d779932011-08-25 16:33:41 -0700155 EXPECT_TRUE(service_->Load(&storage));
Darin Petkovba40dd32011-07-11 20:06:39 -0700156}
157
158TEST_F(ServiceTest, LoadFail) {
159 StrictMock<MockStore> storage;
Chris Masone34af2182011-08-22 11:59:36 -0700160 EXPECT_CALL(storage, ContainsGroup(storage_id_)).WillOnce(Return(false));
Chris Masone9d779932011-08-25 16:33:41 -0700161 EXPECT_FALSE(service_->Load(&storage));
Darin Petkovba40dd32011-07-11 20:06:39 -0700162}
163
164TEST_F(ServiceTest, SaveString) {
165 MockStore storage;
166 static const char kKey[] = "test-key";
167 static const char kData[] = "test-data";
Chris Masone34af2182011-08-22 11:59:36 -0700168 EXPECT_CALL(storage, SetString(storage_id_, kKey, kData))
Darin Petkovba40dd32011-07-11 20:06:39 -0700169 .WillOnce(Return(true));
Chris Masone34af2182011-08-22 11:59:36 -0700170 service_->SaveString(&storage, storage_id_, kKey, kData, false, true);
Darin Petkovba40dd32011-07-11 20:06:39 -0700171}
172
173TEST_F(ServiceTest, SaveStringCrypted) {
174 MockStore storage;
175 static const char kKey[] = "test-key";
176 static const char kData[] = "test-data";
Chris Masone34af2182011-08-22 11:59:36 -0700177 EXPECT_CALL(storage, SetCryptedString(storage_id_, kKey, kData))
Darin Petkovba40dd32011-07-11 20:06:39 -0700178 .WillOnce(Return(true));
Chris Masone34af2182011-08-22 11:59:36 -0700179 service_->SaveString(&storage, storage_id_, kKey, kData, true, true);
Darin Petkovba40dd32011-07-11 20:06:39 -0700180}
181
182TEST_F(ServiceTest, SaveStringDontSave) {
183 MockStore storage;
184 static const char kKey[] = "test-key";
Chris Masone34af2182011-08-22 11:59:36 -0700185 EXPECT_CALL(storage, DeleteKey(storage_id_, kKey))
Darin Petkovba40dd32011-07-11 20:06:39 -0700186 .WillOnce(Return(true));
Chris Masone34af2182011-08-22 11:59:36 -0700187 service_->SaveString(&storage, storage_id_, kKey, "data", false, false);
Darin Petkovba40dd32011-07-11 20:06:39 -0700188}
189
190TEST_F(ServiceTest, SaveStringEmpty) {
191 MockStore storage;
192 static const char kKey[] = "test-key";
Chris Masone34af2182011-08-22 11:59:36 -0700193 EXPECT_CALL(storage, DeleteKey(storage_id_, kKey))
Darin Petkovba40dd32011-07-11 20:06:39 -0700194 .WillOnce(Return(true));
Chris Masone34af2182011-08-22 11:59:36 -0700195 service_->SaveString(&storage, storage_id_, kKey, "", true, true);
Darin Petkovba40dd32011-07-11 20:06:39 -0700196}
197
198TEST_F(ServiceTest, Save) {
199 NiceMock<MockStore> storage;
Chris Masone34af2182011-08-22 11:59:36 -0700200 EXPECT_CALL(storage, SetString(storage_id_, _, _))
Darin Petkovba40dd32011-07-11 20:06:39 -0700201 .Times(AtLeast(1))
202 .WillRepeatedly(Return(true));
Chris Masone34af2182011-08-22 11:59:36 -0700203 EXPECT_CALL(storage, DeleteKey(storage_id_, _))
Darin Petkovba40dd32011-07-11 20:06:39 -0700204 .Times(AtLeast(1))
205 .WillRepeatedly(Return(true));
Chris Masone9d779932011-08-25 16:33:41 -0700206 EXPECT_TRUE(service_->Save(&storage));
Darin Petkovba40dd32011-07-11 20:06:39 -0700207}
208
Paul Stewartd8ad3c42012-01-09 12:39:38 -0800209TEST_F(ServiceTest, Unload) {
210 NiceMock<MockStore> storage;
211 EXPECT_CALL(storage, ContainsGroup(storage_id_)).WillOnce(Return(true));
212 static const string string_value("value");
213 EXPECT_CALL(storage, GetString(storage_id_, _, _))
214 .Times(AtLeast(1))
215 .WillRepeatedly(DoAll(SetArgumentPointee<2>(string_value), Return(true)));
216 ASSERT_TRUE(service_->Load(&storage));
217 // TODO(pstew): A single string property in the service is tested as
218 // a sentinel that properties are being set and reset at the rit times.
219 // However, since property load/store is essentially a manual process,
220 // it is error prone and should either be exhaustively unit-tested or
221 // a generic framework for registering loaded/stored properties should
222 // be created. crosbug.com/24859
223 EXPECT_EQ(string_value, service_->ui_data_);
224 service_->Unload();
225 EXPECT_EQ(string(""), service_->ui_data_);
226}
227
Paul Stewart03dba0b2011-08-22 16:32:45 -0700228TEST_F(ServiceTest, State) {
229 EXPECT_EQ(Service::kStateUnknown, service_->state());
230 EXPECT_EQ(Service::kFailureUnknown, service_->failure());
231
mukesh agrawal00917ce2011-11-22 23:56:55 +0000232 ServiceRefPtr service_ref(service_);
mukesh agrawalf2f68a52011-09-01 12:15:48 -0700233
234 // TODO(quiche): make this EXPECT_CALL work (crosbug.com/20154)
235 // EXPECT_CALL(*dynamic_cast<ServiceMockAdaptor *>(service_->adaptor_.get()),
236 // EmitStringChanged(flimflam::kStateProperty, _));
Paul Stewart03dba0b2011-08-22 16:32:45 -0700237 EXPECT_CALL(mock_manager_, UpdateService(service_ref));
238 service_->SetState(Service::kStateConnected);
239 // A second state change shouldn't cause another update
240 service_->SetState(Service::kStateConnected);
241
242 EXPECT_EQ(Service::kStateConnected, service_->state());
243 EXPECT_EQ(Service::kFailureUnknown, service_->failure());
244 EXPECT_CALL(mock_manager_, UpdateService(service_ref));
245 service_->SetState(Service::kStateDisconnected);
246
247 EXPECT_CALL(mock_manager_, UpdateService(service_ref));
248 service_->SetFailure(Service::kFailureOutOfRange);
249
250 EXPECT_EQ(Service::kStateFailure, service_->state());
251 EXPECT_EQ(Service::kFailureOutOfRange, service_->failure());
252}
253
Darin Petkovb100ae72011-08-24 16:19:45 -0700254TEST_F(ServiceTest, ActivateCellularModem) {
255 Error error;
256 service_->ActivateCellularModem("Carrier", &error);
257 EXPECT_EQ(Error::kInvalidArguments, error.type());
258}
259
mukesh agrawal00917ce2011-11-22 23:56:55 +0000260TEST_F(ServiceTest, MakeFavorite) {
261 EXPECT_FALSE(service_->favorite());
262 EXPECT_FALSE(service_->auto_connect());
263
264 service_->MakeFavorite();
265 EXPECT_TRUE(service_->favorite());
266 EXPECT_TRUE(service_->auto_connect());
267}
268
269TEST_F(ServiceTest, ReMakeFavorite) {
270 service_->MakeFavorite();
271 EXPECT_TRUE(service_->favorite());
272 EXPECT_TRUE(service_->auto_connect());
273
274 service_->set_auto_connect(false);
275 service_->MakeFavorite();
276 EXPECT_TRUE(service_->favorite());
277 EXPECT_FALSE(service_->auto_connect());
278}
279
mukesh agrawal76d13882012-01-12 15:23:11 -0800280TEST_F(ServiceTest, IsAutoConnectable) {
281 service_->set_connectable(true);
282 EXPECT_TRUE(service_->IsAutoConnectable());
283
284 service_->SetState(Service::kStateConnected);
285 EXPECT_FALSE(service_->IsAutoConnectable());
286
287 service_->SetState(Service::kStateAssociating);
288 EXPECT_FALSE(service_->IsAutoConnectable());
289}
290
Chris Masone3bd3c8c2011-06-13 08:20:26 -0700291} // namespace shill