| Alex Deymo | aea4c1c | 2015-08-19 20:24:43 -0700 | [diff] [blame] | 1 | // | 
|  | 2 | // Copyright (C) 2012 The Android Open Source Project | 
|  | 3 | // | 
|  | 4 | // Licensed under the Apache License, Version 2.0 (the "License"); | 
|  | 5 | // you may not use this file except in compliance with the License. | 
|  | 6 | // You may obtain a copy of the License at | 
|  | 7 | // | 
|  | 8 | //      http://www.apache.org/licenses/LICENSE-2.0 | 
|  | 9 | // | 
|  | 10 | // Unless required by applicable law or agreed to in writing, software | 
|  | 11 | // distributed under the License is distributed on an "AS IS" BASIS, | 
|  | 12 | // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | 
|  | 13 | // See the License for the specific language governing permissions and | 
|  | 14 | // limitations under the License. | 
|  | 15 | // | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 16 |  | 
| Alex Deymo | 39910dc | 2015-11-09 17:04:30 -0800 | [diff] [blame] | 17 | #include "update_engine/common/prefs.h" | 
| Alex Deymo | aab50e3 | 2014-11-10 19:55:35 -0800 | [diff] [blame] | 18 |  | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 19 | #include <inttypes.h> | 
|  | 20 |  | 
|  | 21 | #include <string> | 
|  | 22 |  | 
| Ben Chan | 06c76a4 | 2014-09-05 08:21:06 -0700 | [diff] [blame] | 23 | #include <base/files/file_util.h> | 
| Ben Chan | 05735a1 | 2014-09-03 07:48:22 -0700 | [diff] [blame] | 24 | #include <base/macros.h> | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 25 | #include <base/strings/string_util.h> | 
|  | 26 | #include <base/strings/stringprintf.h> | 
| Alex Deymo | d6f6007 | 2015-10-12 12:22:27 -0700 | [diff] [blame] | 27 | #include <gmock/gmock.h> | 
| Ben Chan | 05735a1 | 2014-09-03 07:48:22 -0700 | [diff] [blame] | 28 | #include <gtest/gtest.h> | 
|  | 29 |  | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 30 | using std::string; | 
| Alex Deymo | d6f6007 | 2015-10-12 12:22:27 -0700 | [diff] [blame] | 31 | using testing::Eq; | 
|  | 32 | using testing::_; | 
|  | 33 |  | 
|  | 34 | namespace { | 
|  | 35 | // Test key used along the tests. | 
|  | 36 | const char kKey[] = "test-key"; | 
|  | 37 | } | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 38 |  | 
|  | 39 | namespace chromeos_update_engine { | 
|  | 40 |  | 
|  | 41 | class PrefsTest : public ::testing::Test { | 
|  | 42 | protected: | 
| Alex Deymo | 610277e | 2014-11-11 21:18:11 -0800 | [diff] [blame] | 43 | void SetUp() override { | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 44 | ASSERT_TRUE(base::CreateNewTempDirectory("auprefs", &prefs_dir_)); | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 45 | ASSERT_TRUE(prefs_.Init(prefs_dir_)); | 
|  | 46 | } | 
|  | 47 |  | 
| Alex Deymo | 610277e | 2014-11-11 21:18:11 -0800 | [diff] [blame] | 48 | void TearDown() override { | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 49 | base::DeleteFile(prefs_dir_, true);  // recursive | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 50 | } | 
|  | 51 |  | 
|  | 52 | bool SetValue(const string& key, const string& value) { | 
| Ben Chan | 736fcb5 | 2014-05-21 18:28:22 -0700 | [diff] [blame] | 53 | return base::WriteFile(prefs_dir_.Append(key), value.data(), | 
|  | 54 | value.length()) == static_cast<int>(value.length()); | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 55 | } | 
|  | 56 |  | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 57 | base::FilePath prefs_dir_; | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 58 | Prefs prefs_; | 
|  | 59 | }; | 
|  | 60 |  | 
|  | 61 | TEST_F(PrefsTest, GetFileNameForKey) { | 
| Alex Deymo | d6f6007 | 2015-10-12 12:22:27 -0700 | [diff] [blame] | 62 | const char kAllvalidCharsKey[] = | 
|  | 63 | "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz_-"; | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 64 | base::FilePath path; | 
| Alex Deymo | d6f6007 | 2015-10-12 12:22:27 -0700 | [diff] [blame] | 65 | EXPECT_TRUE(prefs_.GetFileNameForKey(kAllvalidCharsKey, &path)); | 
|  | 66 | EXPECT_EQ(prefs_dir_.Append(kAllvalidCharsKey).value(), path.value()); | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 67 | } | 
|  | 68 |  | 
|  | 69 | TEST_F(PrefsTest, GetFileNameForKeyBadCharacter) { | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 70 | base::FilePath path; | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 71 | EXPECT_FALSE(prefs_.GetFileNameForKey("ABC abc", &path)); | 
|  | 72 | } | 
|  | 73 |  | 
|  | 74 | TEST_F(PrefsTest, GetFileNameForKeyEmpty) { | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 75 | base::FilePath path; | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 76 | EXPECT_FALSE(prefs_.GetFileNameForKey("", &path)); | 
|  | 77 | } | 
|  | 78 |  | 
|  | 79 | TEST_F(PrefsTest, GetString) { | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 80 | const string test_data = "test data"; | 
|  | 81 | ASSERT_TRUE(SetValue(kKey, test_data)); | 
|  | 82 | string value; | 
|  | 83 | EXPECT_TRUE(prefs_.GetString(kKey, &value)); | 
|  | 84 | EXPECT_EQ(test_data, value); | 
|  | 85 | } | 
|  | 86 |  | 
|  | 87 | TEST_F(PrefsTest, GetStringBadKey) { | 
|  | 88 | string value; | 
|  | 89 | EXPECT_FALSE(prefs_.GetString(",bad", &value)); | 
|  | 90 | } | 
|  | 91 |  | 
|  | 92 | TEST_F(PrefsTest, GetStringNonExistentKey) { | 
|  | 93 | string value; | 
|  | 94 | EXPECT_FALSE(prefs_.GetString("non-existent-key", &value)); | 
|  | 95 | } | 
|  | 96 |  | 
|  | 97 | TEST_F(PrefsTest, SetString) { | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 98 | const char kValue[] = "some test value\non 2 lines"; | 
|  | 99 | EXPECT_TRUE(prefs_.SetString(kKey, kValue)); | 
|  | 100 | string value; | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 101 | EXPECT_TRUE(base::ReadFileToString(prefs_dir_.Append(kKey), &value)); | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 102 | EXPECT_EQ(kValue, value); | 
|  | 103 | } | 
|  | 104 |  | 
|  | 105 | TEST_F(PrefsTest, SetStringBadKey) { | 
| Alex Deymo | d6f6007 | 2015-10-12 12:22:27 -0700 | [diff] [blame] | 106 | const char kKeyWithDots[] = ".no-dots"; | 
|  | 107 | EXPECT_FALSE(prefs_.SetString(kKeyWithDots, "some value")); | 
|  | 108 | EXPECT_FALSE(base::PathExists(prefs_dir_.Append(kKeyWithDots))); | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 109 | } | 
|  | 110 |  | 
|  | 111 | TEST_F(PrefsTest, SetStringCreateDir) { | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 112 | const char kValue[] = "test value"; | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 113 | base::FilePath subdir = prefs_dir_.Append("subdir1").Append("subdir2"); | 
| Darin Petkov | 1cbd78f | 2010-07-29 12:38:34 -0700 | [diff] [blame] | 114 | EXPECT_TRUE(prefs_.Init(subdir)); | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 115 | EXPECT_TRUE(prefs_.SetString(kKey, kValue)); | 
|  | 116 | string value; | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 117 | EXPECT_TRUE(base::ReadFileToString(subdir.Append(kKey), &value)); | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 118 | EXPECT_EQ(kValue, value); | 
|  | 119 | } | 
|  | 120 |  | 
|  | 121 | TEST_F(PrefsTest, SetStringDirCreationFailure) { | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 122 | EXPECT_TRUE(prefs_.Init(base::FilePath("/dev/null"))); | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 123 | EXPECT_FALSE(prefs_.SetString(kKey, "test value")); | 
|  | 124 | } | 
|  | 125 |  | 
|  | 126 | TEST_F(PrefsTest, SetStringFileCreationFailure) { | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 127 | base::CreateDirectory(prefs_dir_.Append(kKey)); | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 128 | EXPECT_FALSE(prefs_.SetString(kKey, "test value")); | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 129 | EXPECT_TRUE(base::DirectoryExists(prefs_dir_.Append(kKey))); | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 130 | } | 
|  | 131 |  | 
|  | 132 | TEST_F(PrefsTest, GetInt64) { | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 133 | ASSERT_TRUE(SetValue(kKey, " \n 25 \t ")); | 
|  | 134 | int64_t value; | 
|  | 135 | EXPECT_TRUE(prefs_.GetInt64(kKey, &value)); | 
|  | 136 | EXPECT_EQ(25, value); | 
|  | 137 | } | 
|  | 138 |  | 
|  | 139 | TEST_F(PrefsTest, GetInt64BadValue) { | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 140 | ASSERT_TRUE(SetValue(kKey, "30a")); | 
|  | 141 | int64_t value; | 
|  | 142 | EXPECT_FALSE(prefs_.GetInt64(kKey, &value)); | 
|  | 143 | } | 
|  | 144 |  | 
|  | 145 | TEST_F(PrefsTest, GetInt64Max) { | 
| Alex Vakulenko | 0103c36 | 2016-01-20 07:56:15 -0800 | [diff] [blame] | 146 | ASSERT_TRUE(SetValue(kKey, base::StringPrintf( | 
| Alex Deymo | 80f70ff | 2016-02-10 16:08:11 -0800 | [diff] [blame] | 147 | "%" PRIi64, std::numeric_limits<int64_t>::max()))); | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 148 | int64_t value; | 
|  | 149 | EXPECT_TRUE(prefs_.GetInt64(kKey, &value)); | 
| Alex Deymo | 80f70ff | 2016-02-10 16:08:11 -0800 | [diff] [blame] | 150 | EXPECT_EQ(std::numeric_limits<int64_t>::max(), value); | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 151 | } | 
|  | 152 |  | 
|  | 153 | TEST_F(PrefsTest, GetInt64Min) { | 
| Alex Vakulenko | 0103c36 | 2016-01-20 07:56:15 -0800 | [diff] [blame] | 154 | ASSERT_TRUE(SetValue(kKey, base::StringPrintf( | 
| Alex Deymo | 80f70ff | 2016-02-10 16:08:11 -0800 | [diff] [blame] | 155 | "%" PRIi64, std::numeric_limits<int64_t>::min()))); | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 156 | int64_t value; | 
|  | 157 | EXPECT_TRUE(prefs_.GetInt64(kKey, &value)); | 
| Alex Deymo | 80f70ff | 2016-02-10 16:08:11 -0800 | [diff] [blame] | 158 | EXPECT_EQ(std::numeric_limits<int64_t>::min(), value); | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 159 | } | 
|  | 160 |  | 
|  | 161 | TEST_F(PrefsTest, GetInt64Negative) { | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 162 | ASSERT_TRUE(SetValue(kKey, " \t -100 \n ")); | 
|  | 163 | int64_t value; | 
|  | 164 | EXPECT_TRUE(prefs_.GetInt64(kKey, &value)); | 
|  | 165 | EXPECT_EQ(-100, value); | 
|  | 166 | } | 
|  | 167 |  | 
|  | 168 | TEST_F(PrefsTest, GetInt64NonExistentKey) { | 
|  | 169 | int64_t value; | 
|  | 170 | EXPECT_FALSE(prefs_.GetInt64("random-key", &value)); | 
|  | 171 | } | 
|  | 172 |  | 
|  | 173 | TEST_F(PrefsTest, SetInt64) { | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 174 | EXPECT_TRUE(prefs_.SetInt64(kKey, -123)); | 
|  | 175 | string value; | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 176 | EXPECT_TRUE(base::ReadFileToString(prefs_dir_.Append(kKey), &value)); | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 177 | EXPECT_EQ("-123", value); | 
|  | 178 | } | 
|  | 179 |  | 
|  | 180 | TEST_F(PrefsTest, SetInt64BadKey) { | 
| Alex Deymo | d6f6007 | 2015-10-12 12:22:27 -0700 | [diff] [blame] | 181 | const char kKeyWithSpaces[] = "s p a c e s"; | 
|  | 182 | EXPECT_FALSE(prefs_.SetInt64(kKeyWithSpaces, 20)); | 
|  | 183 | EXPECT_FALSE(base::PathExists(prefs_dir_.Append(kKeyWithSpaces))); | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 184 | } | 
|  | 185 |  | 
|  | 186 | TEST_F(PrefsTest, SetInt64Max) { | 
| Alex Vakulenko | 0103c36 | 2016-01-20 07:56:15 -0800 | [diff] [blame] | 187 | EXPECT_TRUE(prefs_.SetInt64(kKey, std::numeric_limits<int64_t>::max())); | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 188 | string value; | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 189 | EXPECT_TRUE(base::ReadFileToString(prefs_dir_.Append(kKey), &value)); | 
| Alex Vakulenko | 0103c36 | 2016-01-20 07:56:15 -0800 | [diff] [blame] | 190 | EXPECT_EQ(base::StringPrintf("%" PRIi64, std::numeric_limits<int64_t>::max()), | 
|  | 191 | value); | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 192 | } | 
|  | 193 |  | 
|  | 194 | TEST_F(PrefsTest, SetInt64Min) { | 
| Alex Vakulenko | 0103c36 | 2016-01-20 07:56:15 -0800 | [diff] [blame] | 195 | EXPECT_TRUE(prefs_.SetInt64(kKey, std::numeric_limits<int64_t>::min())); | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 196 | string value; | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 197 | EXPECT_TRUE(base::ReadFileToString(prefs_dir_.Append(kKey), &value)); | 
| Alex Vakulenko | 0103c36 | 2016-01-20 07:56:15 -0800 | [diff] [blame] | 198 | EXPECT_EQ(base::StringPrintf("%" PRIi64, std::numeric_limits<int64_t>::min()), | 
|  | 199 | value); | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 200 | } | 
|  | 201 |  | 
| Alex Deymo | efb7c4c | 2013-07-09 14:34:00 -0700 | [diff] [blame] | 202 | TEST_F(PrefsTest, GetBooleanFalse) { | 
| Alex Deymo | efb7c4c | 2013-07-09 14:34:00 -0700 | [diff] [blame] | 203 | ASSERT_TRUE(SetValue(kKey, " \n false \t ")); | 
|  | 204 | bool value; | 
|  | 205 | EXPECT_TRUE(prefs_.GetBoolean(kKey, &value)); | 
|  | 206 | EXPECT_FALSE(value); | 
|  | 207 | } | 
|  | 208 |  | 
|  | 209 | TEST_F(PrefsTest, GetBooleanTrue) { | 
|  | 210 | const char kKey[] = "test-key"; | 
|  | 211 | ASSERT_TRUE(SetValue(kKey, " \t true \n ")); | 
|  | 212 | bool value; | 
|  | 213 | EXPECT_TRUE(prefs_.GetBoolean(kKey, &value)); | 
|  | 214 | EXPECT_TRUE(value); | 
|  | 215 | } | 
|  | 216 |  | 
|  | 217 | TEST_F(PrefsTest, GetBooleanBadValue) { | 
|  | 218 | const char kKey[] = "test-key"; | 
|  | 219 | ASSERT_TRUE(SetValue(kKey, "1")); | 
|  | 220 | bool value; | 
|  | 221 | EXPECT_FALSE(prefs_.GetBoolean(kKey, &value)); | 
|  | 222 | } | 
|  | 223 |  | 
|  | 224 | TEST_F(PrefsTest, GetBooleanBadEmptyValue) { | 
|  | 225 | const char kKey[] = "test-key"; | 
|  | 226 | ASSERT_TRUE(SetValue(kKey, "")); | 
|  | 227 | bool value; | 
|  | 228 | EXPECT_FALSE(prefs_.GetBoolean(kKey, &value)); | 
|  | 229 | } | 
|  | 230 |  | 
|  | 231 | TEST_F(PrefsTest, GetBooleanNonExistentKey) { | 
|  | 232 | bool value; | 
|  | 233 | EXPECT_FALSE(prefs_.GetBoolean("random-key", &value)); | 
|  | 234 | } | 
|  | 235 |  | 
|  | 236 | TEST_F(PrefsTest, SetBooleanTrue) { | 
|  | 237 | const char kKey[] = "test-bool"; | 
|  | 238 | EXPECT_TRUE(prefs_.SetBoolean(kKey, true)); | 
|  | 239 | string value; | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 240 | EXPECT_TRUE(base::ReadFileToString(prefs_dir_.Append(kKey), &value)); | 
| Alex Deymo | efb7c4c | 2013-07-09 14:34:00 -0700 | [diff] [blame] | 241 | EXPECT_EQ("true", value); | 
|  | 242 | } | 
|  | 243 |  | 
|  | 244 | TEST_F(PrefsTest, SetBooleanFalse) { | 
|  | 245 | const char kKey[] = "test-bool"; | 
|  | 246 | EXPECT_TRUE(prefs_.SetBoolean(kKey, false)); | 
|  | 247 | string value; | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 248 | EXPECT_TRUE(base::ReadFileToString(prefs_dir_.Append(kKey), &value)); | 
| Alex Deymo | efb7c4c | 2013-07-09 14:34:00 -0700 | [diff] [blame] | 249 | EXPECT_EQ("false", value); | 
|  | 250 | } | 
|  | 251 |  | 
|  | 252 | TEST_F(PrefsTest, SetBooleanBadKey) { | 
|  | 253 | const char kKey[] = "s p a c e s"; | 
|  | 254 | EXPECT_FALSE(prefs_.SetBoolean(kKey, true)); | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 255 | EXPECT_FALSE(base::PathExists(prefs_dir_.Append(kKey))); | 
| Alex Deymo | efb7c4c | 2013-07-09 14:34:00 -0700 | [diff] [blame] | 256 | } | 
|  | 257 |  | 
| Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 258 | TEST_F(PrefsTest, ExistsWorks) { | 
| Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 259 | // test that the key doesn't exist before we set it. | 
|  | 260 | EXPECT_FALSE(prefs_.Exists(kKey)); | 
|  | 261 |  | 
|  | 262 | // test that the key exists after we set it. | 
|  | 263 | ASSERT_TRUE(prefs_.SetInt64(kKey, 8)); | 
|  | 264 | EXPECT_TRUE(prefs_.Exists(kKey)); | 
|  | 265 | } | 
|  | 266 |  | 
|  | 267 | TEST_F(PrefsTest, DeleteWorks) { | 
| Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 268 | // test that it's alright to delete a non-existent key. | 
|  | 269 | EXPECT_TRUE(prefs_.Delete(kKey)); | 
|  | 270 |  | 
|  | 271 | // delete the key after we set it. | 
|  | 272 | ASSERT_TRUE(prefs_.SetInt64(kKey, 0)); | 
|  | 273 | EXPECT_TRUE(prefs_.Delete(kKey)); | 
|  | 274 |  | 
|  | 275 | // make sure it doesn't exist anymore. | 
|  | 276 | EXPECT_FALSE(prefs_.Exists(kKey)); | 
|  | 277 | } | 
|  | 278 |  | 
| Alex Deymo | d6f6007 | 2015-10-12 12:22:27 -0700 | [diff] [blame] | 279 | class MockPrefsObserver : public PrefsInterface::ObserverInterface { | 
|  | 280 | public: | 
|  | 281 | MOCK_METHOD1(OnPrefSet, void(const string&)); | 
|  | 282 | MOCK_METHOD1(OnPrefDeleted, void(const string& key)); | 
|  | 283 | }; | 
|  | 284 |  | 
|  | 285 | TEST_F(PrefsTest, ObserversCalled) { | 
|  | 286 | MockPrefsObserver mock_obserser; | 
|  | 287 | prefs_.AddObserver(kKey, &mock_obserser); | 
|  | 288 |  | 
|  | 289 | EXPECT_CALL(mock_obserser, OnPrefSet(Eq(kKey))); | 
|  | 290 | EXPECT_CALL(mock_obserser, OnPrefDeleted(_)).Times(0); | 
|  | 291 | prefs_.SetString(kKey, "value"); | 
|  | 292 | testing::Mock::VerifyAndClearExpectations(&mock_obserser); | 
|  | 293 |  | 
|  | 294 | EXPECT_CALL(mock_obserser, OnPrefSet(_)).Times(0); | 
|  | 295 | EXPECT_CALL(mock_obserser, OnPrefDeleted(Eq(kKey))); | 
|  | 296 | prefs_.Delete(kKey); | 
|  | 297 | testing::Mock::VerifyAndClearExpectations(&mock_obserser); | 
|  | 298 |  | 
|  | 299 | prefs_.RemoveObserver(kKey, &mock_obserser); | 
|  | 300 | } | 
|  | 301 |  | 
|  | 302 | TEST_F(PrefsTest, OnlyCalledOnObservedKeys) { | 
|  | 303 | MockPrefsObserver mock_obserser; | 
|  | 304 | const char kUnusedKey[] = "unused-key"; | 
|  | 305 | prefs_.AddObserver(kUnusedKey, &mock_obserser); | 
|  | 306 |  | 
|  | 307 | EXPECT_CALL(mock_obserser, OnPrefSet(_)).Times(0); | 
|  | 308 | EXPECT_CALL(mock_obserser, OnPrefDeleted(_)).Times(0); | 
|  | 309 | prefs_.SetString(kKey, "value"); | 
|  | 310 | prefs_.Delete(kKey); | 
|  | 311 |  | 
|  | 312 | prefs_.RemoveObserver(kUnusedKey, &mock_obserser); | 
|  | 313 | } | 
|  | 314 |  | 
|  | 315 | TEST_F(PrefsTest, RemovedObserversNotCalled) { | 
|  | 316 | MockPrefsObserver mock_obserser_a, mock_obserser_b; | 
|  | 317 | prefs_.AddObserver(kKey, &mock_obserser_a); | 
|  | 318 | prefs_.AddObserver(kKey, &mock_obserser_b); | 
|  | 319 | EXPECT_CALL(mock_obserser_a, OnPrefSet(_)).Times(2); | 
|  | 320 | EXPECT_CALL(mock_obserser_b, OnPrefSet(_)).Times(1); | 
|  | 321 | EXPECT_TRUE(prefs_.SetString(kKey, "value")); | 
|  | 322 | prefs_.RemoveObserver(kKey, &mock_obserser_b); | 
|  | 323 | EXPECT_TRUE(prefs_.SetString(kKey, "other value")); | 
|  | 324 | prefs_.RemoveObserver(kKey, &mock_obserser_a); | 
|  | 325 | EXPECT_TRUE(prefs_.SetString(kKey, "yet another value")); | 
|  | 326 | } | 
|  | 327 |  | 
|  | 328 | TEST_F(PrefsTest, UnsuccessfulCallsNotObserved) { | 
|  | 329 | MockPrefsObserver mock_obserser; | 
|  | 330 | const char kInvalidKey[] = "no spaces or ."; | 
|  | 331 | prefs_.AddObserver(kInvalidKey, &mock_obserser); | 
|  | 332 |  | 
|  | 333 | EXPECT_CALL(mock_obserser, OnPrefSet(_)).Times(0); | 
|  | 334 | EXPECT_CALL(mock_obserser, OnPrefDeleted(_)).Times(0); | 
|  | 335 | EXPECT_FALSE(prefs_.SetString(kInvalidKey, "value")); | 
|  | 336 | EXPECT_FALSE(prefs_.Delete(kInvalidKey)); | 
|  | 337 |  | 
|  | 338 | prefs_.RemoveObserver(kInvalidKey, &mock_obserser); | 
|  | 339 | } | 
|  | 340 |  | 
| Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 341 | }  // namespace chromeos_update_engine |