Thieu Le | 3426c8f | 2012-01-11 17:35:11 -0800 | [diff] [blame] | 1 | // Copyright (c) 2012 The Chromium OS Authors. All rights reserved. |
Chris Masone | 88cbd5f | 2011-07-03 14:30:04 -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/default_profile.h" |
| 6 | |
| 7 | #include <map> |
| 8 | #include <string> |
Chris Masone | 877ff98 | 2011-09-21 16:18:24 -0700 | [diff] [blame] | 9 | #include <vector> |
Chris Masone | 88cbd5f | 2011-07-03 14:30:04 -0700 | [diff] [blame] | 10 | |
Chris Masone | 6515aab | 2011-10-12 16:19:09 -0700 | [diff] [blame] | 11 | #include <base/file_path.h> |
Chris Masone | 88cbd5f | 2011-07-03 14:30:04 -0700 | [diff] [blame] | 12 | #include <chromeos/dbus/service_constants.h> |
| 13 | #include <gtest/gtest.h> |
| 14 | #include <gmock/gmock.h> |
| 15 | |
Chris Masone | 6515aab | 2011-10-12 16:19:09 -0700 | [diff] [blame] | 16 | #include "shill/key_file_store.h" |
| 17 | #include "shill/glib.h" |
Chris Masone | 88cbd5f | 2011-07-03 14:30:04 -0700 | [diff] [blame] | 18 | #include "shill/manager.h" |
| 19 | #include "shill/mock_control.h" |
Chris Masone | 877ff98 | 2011-09-21 16:18:24 -0700 | [diff] [blame] | 20 | #include "shill/mock_device.h" |
Chris Masone | aa48237 | 2011-09-14 16:40:37 -0700 | [diff] [blame] | 21 | #include "shill/mock_store.h" |
Paul Stewart | e692740 | 2012-01-23 16:11:30 -0800 | [diff] [blame] | 22 | #include "shill/portal_detector.h" |
Chris Masone | 88cbd5f | 2011-07-03 14:30:04 -0700 | [diff] [blame] | 23 | #include "shill/property_store_unittest.h" |
| 24 | |
| 25 | using std::map; |
| 26 | using std::string; |
Chris Masone | 877ff98 | 2011-09-21 16:18:24 -0700 | [diff] [blame] | 27 | using std::vector; |
Chris Masone | 88cbd5f | 2011-07-03 14:30:04 -0700 | [diff] [blame] | 28 | using ::testing::_; |
Paul Stewart | 870523b | 2012-01-11 17:00:42 -0800 | [diff] [blame] | 29 | using ::testing::DoAll; |
Chris Masone | aa48237 | 2011-09-14 16:40:37 -0700 | [diff] [blame] | 30 | using ::testing::Return; |
Paul Stewart | 870523b | 2012-01-11 17:00:42 -0800 | [diff] [blame] | 31 | using ::testing::SetArgumentPointee; |
Chris Masone | 88cbd5f | 2011-07-03 14:30:04 -0700 | [diff] [blame] | 32 | |
| 33 | namespace shill { |
| 34 | |
| 35 | class DefaultProfileTest : public PropertyStoreTest { |
| 36 | public: |
Chris Masone | 7aa5f90 | 2011-07-11 11:13:35 -0700 | [diff] [blame] | 37 | DefaultProfileTest() |
Chris Masone | 2176a88 | 2011-09-14 22:29:15 -0700 | [diff] [blame] | 38 | : profile_(new DefaultProfile(control_interface(), |
Chris Masone | 9d77993 | 2011-08-25 16:33:41 -0700 | [diff] [blame] | 39 | manager(), |
Chris Masone | 6515aab | 2011-10-12 16:19:09 -0700 | [diff] [blame] | 40 | FilePath(storage_path()), |
Paul Stewart | d0a3b81 | 2012-03-28 22:48:22 -0700 | [diff] [blame] | 41 | DefaultProfile::kDefaultId, |
Chris Masone | 877ff98 | 2011-09-21 16:18:24 -0700 | [diff] [blame] | 42 | properties_)), |
| 43 | device_(new MockDevice(control_interface(), |
| 44 | dispatcher(), |
Thieu Le | 3426c8f | 2012-01-11 17:35:11 -0800 | [diff] [blame] | 45 | metrics(), |
Chris Masone | 877ff98 | 2011-09-21 16:18:24 -0700 | [diff] [blame] | 46 | manager(), |
| 47 | "null0", |
| 48 | "addr0", |
| 49 | 0)) { |
Chris Masone | 7aa5f90 | 2011-07-11 11:13:35 -0700 | [diff] [blame] | 50 | } |
Chris Masone | 88cbd5f | 2011-07-03 14:30:04 -0700 | [diff] [blame] | 51 | |
| 52 | virtual ~DefaultProfileTest() {} |
| 53 | |
Chris Masone | 6515aab | 2011-10-12 16:19:09 -0700 | [diff] [blame] | 54 | virtual void SetUp() { |
| 55 | PropertyStoreTest::SetUp(); |
| 56 | FilePath final_path; |
| 57 | ASSERT_TRUE(profile_->GetStoragePath(&final_path)); |
| 58 | scoped_ptr<KeyFileStore> storage(new KeyFileStore(&real_glib_)); |
| 59 | storage->set_path(final_path); |
| 60 | ASSERT_TRUE(storage->Open()); |
| 61 | profile_->set_storage(storage.release()); // Passes ownership. |
| 62 | } |
| 63 | |
Chris Masone | 88cbd5f | 2011-07-03 14:30:04 -0700 | [diff] [blame] | 64 | protected: |
Chris Masone | 2ae797d | 2011-08-23 20:41:00 -0700 | [diff] [blame] | 65 | static const char kTestStoragePath[]; |
| 66 | |
Chris Masone | 6515aab | 2011-10-12 16:19:09 -0700 | [diff] [blame] | 67 | GLib real_glib_; |
Paul Stewart | 870523b | 2012-01-11 17:00:42 -0800 | [diff] [blame] | 68 | scoped_refptr<DefaultProfile> profile_; |
Chris Masone | 877ff98 | 2011-09-21 16:18:24 -0700 | [diff] [blame] | 69 | scoped_refptr<MockDevice> device_; |
Chris Masone | 88cbd5f | 2011-07-03 14:30:04 -0700 | [diff] [blame] | 70 | Manager::Properties properties_; |
| 71 | }; |
| 72 | |
Chris Masone | 2ae797d | 2011-08-23 20:41:00 -0700 | [diff] [blame] | 73 | const char DefaultProfileTest::kTestStoragePath[] = "/no/where"; |
| 74 | |
Chris Masone | 88cbd5f | 2011-07-03 14:30:04 -0700 | [diff] [blame] | 75 | TEST_F(DefaultProfileTest, GetProperties) { |
| 76 | Error error(Error::kInvalidProperty, ""); |
| 77 | { |
| 78 | map<string, ::DBus::Variant> props; |
| 79 | ::DBus::Error dbus_error; |
Chris Masone | 7aa5f90 | 2011-07-11 11:13:35 -0700 | [diff] [blame] | 80 | DBusAdaptor::GetProperties(profile_->store(), &props, &dbus_error); |
Chris Masone | 88cbd5f | 2011-07-03 14:30:04 -0700 | [diff] [blame] | 81 | ASSERT_FALSE(props.find(flimflam::kOfflineModeProperty) == props.end()); |
| 82 | EXPECT_FALSE(props[flimflam::kOfflineModeProperty].reader().get_bool()); |
| 83 | } |
| 84 | properties_.offline_mode = true; |
| 85 | { |
| 86 | map<string, ::DBus::Variant> props; |
| 87 | ::DBus::Error dbus_error; |
Chris Masone | 7aa5f90 | 2011-07-11 11:13:35 -0700 | [diff] [blame] | 88 | DBusAdaptor::GetProperties(profile_->store(), &props, &dbus_error); |
Chris Masone | 88cbd5f | 2011-07-03 14:30:04 -0700 | [diff] [blame] | 89 | ASSERT_FALSE(props.find(flimflam::kOfflineModeProperty) == props.end()); |
| 90 | EXPECT_TRUE(props[flimflam::kOfflineModeProperty].reader().get_bool()); |
| 91 | } |
| 92 | { |
| 93 | Error error(Error::kInvalidProperty, ""); |
| 94 | EXPECT_FALSE( |
mukesh agrawal | de29fa8 | 2011-09-16 16:16:36 -0700 | [diff] [blame] | 95 | profile_->mutable_store()->SetBoolProperty( |
| 96 | flimflam::kOfflineModeProperty, |
| 97 | true, |
| 98 | &error)); |
Chris Masone | 88cbd5f | 2011-07-03 14:30:04 -0700 | [diff] [blame] | 99 | } |
| 100 | } |
| 101 | |
Chris Masone | aa48237 | 2011-09-14 16:40:37 -0700 | [diff] [blame] | 102 | TEST_F(DefaultProfileTest, Save) { |
Chris Masone | b9c0059 | 2011-10-06 13:10:39 -0700 | [diff] [blame] | 103 | scoped_ptr<MockStore> storage(new MockStore); |
| 104 | EXPECT_CALL(*storage.get(), SetString(DefaultProfile::kStorageId, |
| 105 | DefaultProfile::kStorageName, |
| 106 | DefaultProfile::kDefaultId)) |
Chris Masone | aa48237 | 2011-09-14 16:40:37 -0700 | [diff] [blame] | 107 | .WillOnce(Return(true)); |
Chris Masone | b9c0059 | 2011-10-06 13:10:39 -0700 | [diff] [blame] | 108 | EXPECT_CALL(*storage.get(), SetString(DefaultProfile::kStorageId, |
Paul Stewart | d32f484 | 2012-01-11 16:08:13 -0800 | [diff] [blame] | 109 | DefaultProfile::kStorageHostName, |
Chris Masone | b9c0059 | 2011-10-06 13:10:39 -0700 | [diff] [blame] | 110 | "")) |
Chris Masone | aa48237 | 2011-09-14 16:40:37 -0700 | [diff] [blame] | 111 | .WillOnce(Return(true)); |
Chris Masone | b9c0059 | 2011-10-06 13:10:39 -0700 | [diff] [blame] | 112 | EXPECT_CALL(*storage.get(), SetBool(DefaultProfile::kStorageId, |
| 113 | DefaultProfile::kStorageOfflineMode, |
| 114 | false)) |
Chris Masone | aa48237 | 2011-09-14 16:40:37 -0700 | [diff] [blame] | 115 | .WillOnce(Return(true)); |
Paul Stewart | d32f484 | 2012-01-11 16:08:13 -0800 | [diff] [blame] | 116 | EXPECT_CALL(*storage.get(), SetString(DefaultProfile::kStorageId, |
| 117 | DefaultProfile::kStorageCheckPortalList, |
| 118 | "")) |
| 119 | .WillOnce(Return(true)); |
Paul Stewart | e692740 | 2012-01-23 16:11:30 -0800 | [diff] [blame] | 120 | EXPECT_CALL(*storage.get(), SetString(DefaultProfile::kStorageId, |
| 121 | DefaultProfile::kStoragePortalURL, |
| 122 | "")) |
| 123 | .WillOnce(Return(true)); |
Paul Stewart | c681fa0 | 2012-03-02 19:40:04 -0800 | [diff] [blame] | 124 | EXPECT_CALL(*storage.get(), |
| 125 | SetString(DefaultProfile::kStorageId, |
| 126 | DefaultProfile::kStoragePortalCheckInterval, |
| 127 | "0")) |
| 128 | .WillOnce(Return(true)); |
Chris Masone | b9c0059 | 2011-10-06 13:10:39 -0700 | [diff] [blame] | 129 | EXPECT_CALL(*storage.get(), Flush()).WillOnce(Return(true)); |
Chris Masone | 877ff98 | 2011-09-21 16:18:24 -0700 | [diff] [blame] | 130 | |
Chris Masone | b9c0059 | 2011-10-06 13:10:39 -0700 | [diff] [blame] | 131 | EXPECT_CALL(*device_.get(), Save(storage.get())).WillOnce(Return(true)); |
| 132 | profile_->set_storage(storage.release()); |
Chris Masone | 877ff98 | 2011-09-21 16:18:24 -0700 | [diff] [blame] | 133 | |
| 134 | manager()->RegisterDevice(device_); |
Chris Masone | b9c0059 | 2011-10-06 13:10:39 -0700 | [diff] [blame] | 135 | ASSERT_TRUE(profile_->Save()); |
Chris Masone | 877ff98 | 2011-09-21 16:18:24 -0700 | [diff] [blame] | 136 | manager()->DeregisterDevice(device_); |
Chris Masone | aa48237 | 2011-09-14 16:40:37 -0700 | [diff] [blame] | 137 | } |
| 138 | |
Paul Stewart | e692740 | 2012-01-23 16:11:30 -0800 | [diff] [blame] | 139 | TEST_F(DefaultProfileTest, LoadManagerDefaultProperties) { |
| 140 | scoped_ptr<MockStore> storage(new MockStore); |
| 141 | EXPECT_CALL(*storage.get(), GetString(DefaultProfile::kStorageId, |
| 142 | DefaultProfile::kStorageHostName, |
| 143 | _)) |
| 144 | .WillOnce(Return(false)); |
| 145 | EXPECT_CALL(*storage.get(), GetBool(DefaultProfile::kStorageId, |
| 146 | DefaultProfile::kStorageOfflineMode, |
| 147 | _)) |
| 148 | .WillOnce(Return(false)); |
| 149 | EXPECT_CALL(*storage.get(), GetString(DefaultProfile::kStorageId, |
| 150 | DefaultProfile::kStorageCheckPortalList, |
| 151 | _)) |
| 152 | .WillOnce(Return(false)); |
| 153 | EXPECT_CALL(*storage.get(), GetString(DefaultProfile::kStorageId, |
| 154 | DefaultProfile::kStoragePortalURL, |
| 155 | _)) |
| 156 | .WillOnce(Return(false)); |
Paul Stewart | c681fa0 | 2012-03-02 19:40:04 -0800 | [diff] [blame] | 157 | EXPECT_CALL(*storage.get(), |
| 158 | GetString(DefaultProfile::kStorageId, |
| 159 | DefaultProfile::kStoragePortalCheckInterval, |
| 160 | _)) |
| 161 | .WillOnce(Return(false)); |
Paul Stewart | e692740 | 2012-01-23 16:11:30 -0800 | [diff] [blame] | 162 | profile_->set_storage(storage.release()); |
| 163 | |
| 164 | Manager::Properties manager_props; |
| 165 | ASSERT_TRUE(profile_->LoadManagerProperties(&manager_props)); |
| 166 | EXPECT_EQ("", manager_props.host_name); |
| 167 | EXPECT_FALSE(manager_props.offline_mode); |
Paul Stewart | f555cf8 | 2012-03-15 14:42:43 -0700 | [diff] [blame] | 168 | EXPECT_EQ(PortalDetector::kDefaultCheckPortalList, |
| 169 | manager_props.check_portal_list); |
Paul Stewart | e692740 | 2012-01-23 16:11:30 -0800 | [diff] [blame] | 170 | EXPECT_EQ(PortalDetector::kDefaultURL, manager_props.portal_url); |
Paul Stewart | c681fa0 | 2012-03-02 19:40:04 -0800 | [diff] [blame] | 171 | EXPECT_EQ(PortalDetector::kDefaultCheckIntervalSeconds, |
| 172 | manager_props.portal_check_interval_seconds); |
Paul Stewart | e692740 | 2012-01-23 16:11:30 -0800 | [diff] [blame] | 173 | } |
| 174 | |
Paul Stewart | 870523b | 2012-01-11 17:00:42 -0800 | [diff] [blame] | 175 | TEST_F(DefaultProfileTest, LoadManagerProperties) { |
| 176 | scoped_ptr<MockStore> storage(new MockStore); |
Paul Stewart | d32f484 | 2012-01-11 16:08:13 -0800 | [diff] [blame] | 177 | const string host_name("hostname"); |
| 178 | EXPECT_CALL(*storage.get(), GetString(DefaultProfile::kStorageId, |
| 179 | DefaultProfile::kStorageHostName, |
| 180 | _)) |
| 181 | .WillOnce(DoAll(SetArgumentPointee<2>(host_name), Return(true))); |
Paul Stewart | 870523b | 2012-01-11 17:00:42 -0800 | [diff] [blame] | 182 | EXPECT_CALL(*storage.get(), GetBool(DefaultProfile::kStorageId, |
| 183 | DefaultProfile::kStorageOfflineMode, |
| 184 | _)) |
| 185 | .WillOnce(DoAll(SetArgumentPointee<2>(true), Return(true))); |
| 186 | const string portal_list("technology1,technology2"); |
| 187 | EXPECT_CALL(*storage.get(), GetString(DefaultProfile::kStorageId, |
| 188 | DefaultProfile::kStorageCheckPortalList, |
| 189 | _)) |
| 190 | .WillOnce(DoAll(SetArgumentPointee<2>(portal_list), Return(true))); |
Paul Stewart | e692740 | 2012-01-23 16:11:30 -0800 | [diff] [blame] | 191 | const string portal_url("http://www.chromium.org"); |
| 192 | EXPECT_CALL(*storage.get(), GetString(DefaultProfile::kStorageId, |
| 193 | DefaultProfile::kStoragePortalURL, |
| 194 | _)) |
| 195 | .WillOnce(DoAll(SetArgumentPointee<2>(portal_url), Return(true))); |
Paul Stewart | c681fa0 | 2012-03-02 19:40:04 -0800 | [diff] [blame] | 196 | const string portal_check_interval_string("10"); |
| 197 | const int portal_check_interval_int = 10; |
| 198 | EXPECT_CALL(*storage.get(), |
| 199 | GetString(DefaultProfile::kStorageId, |
| 200 | DefaultProfile::kStoragePortalCheckInterval, |
| 201 | _)) |
| 202 | .WillOnce(DoAll(SetArgumentPointee<2>(portal_check_interval_string), |
| 203 | Return(true))); |
Paul Stewart | 870523b | 2012-01-11 17:00:42 -0800 | [diff] [blame] | 204 | profile_->set_storage(storage.release()); |
| 205 | |
| 206 | Manager::Properties manager_props; |
| 207 | ASSERT_TRUE(profile_->LoadManagerProperties(&manager_props)); |
Paul Stewart | d32f484 | 2012-01-11 16:08:13 -0800 | [diff] [blame] | 208 | EXPECT_EQ(host_name, manager_props.host_name); |
Paul Stewart | 870523b | 2012-01-11 17:00:42 -0800 | [diff] [blame] | 209 | EXPECT_TRUE(manager_props.offline_mode); |
| 210 | EXPECT_EQ(portal_list, manager_props.check_portal_list); |
Paul Stewart | e692740 | 2012-01-23 16:11:30 -0800 | [diff] [blame] | 211 | EXPECT_EQ(portal_url, manager_props.portal_url); |
Paul Stewart | c681fa0 | 2012-03-02 19:40:04 -0800 | [diff] [blame] | 212 | EXPECT_EQ(portal_check_interval_int, |
| 213 | manager_props.portal_check_interval_seconds); |
Paul Stewart | 870523b | 2012-01-11 17:00:42 -0800 | [diff] [blame] | 214 | } |
| 215 | |
Chris Masone | 2ae797d | 2011-08-23 20:41:00 -0700 | [diff] [blame] | 216 | TEST_F(DefaultProfileTest, GetStoragePath) { |
| 217 | FilePath path; |
| 218 | EXPECT_TRUE(profile_->GetStoragePath(&path)); |
Chris Masone | 6515aab | 2011-10-12 16:19:09 -0700 | [diff] [blame] | 219 | EXPECT_EQ(storage_path() + "/default.profile", path.value()); |
Chris Masone | 2ae797d | 2011-08-23 20:41:00 -0700 | [diff] [blame] | 220 | } |
| 221 | |
Chris Masone | 88cbd5f | 2011-07-03 14:30:04 -0700 | [diff] [blame] | 222 | } // namespace shill |