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 | |
Alex Deymo | a0284ac | 2016-07-22 12:51:41 -0700 | [diff] [blame] | 21 | #include <limits> |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 22 | #include <string> |
| 23 | |
Ben Chan | 06c76a4 | 2014-09-05 08:21:06 -0700 | [diff] [blame] | 24 | #include <base/files/file_util.h> |
Sen Jiang | 371615b | 2016-04-13 15:54:29 -0700 | [diff] [blame] | 25 | #include <base/files/scoped_temp_dir.h> |
Ben Chan | 05735a1 | 2014-09-03 07:48:22 -0700 | [diff] [blame] | 26 | #include <base/macros.h> |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 27 | #include <base/strings/string_util.h> |
| 28 | #include <base/strings/stringprintf.h> |
Alex Deymo | d6f6007 | 2015-10-12 12:22:27 -0700 | [diff] [blame] | 29 | #include <gmock/gmock.h> |
Ben Chan | 05735a1 | 2014-09-03 07:48:22 -0700 | [diff] [blame] | 30 | #include <gtest/gtest.h> |
| 31 | |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 32 | using std::string; |
Alex Deymo | d6f6007 | 2015-10-12 12:22:27 -0700 | [diff] [blame] | 33 | using testing::_; |
Andrew | 065d78d | 2020-04-07 15:43:07 -0700 | [diff] [blame] | 34 | using testing::ElementsAre; |
Amin Hassani | b268959 | 2019-01-13 17:04:28 -0800 | [diff] [blame] | 35 | using testing::Eq; |
Alex Deymo | d6f6007 | 2015-10-12 12:22:27 -0700 | [diff] [blame] | 36 | |
| 37 | namespace { |
| 38 | // Test key used along the tests. |
| 39 | const char kKey[] = "test-key"; |
Amin Hassani | b268959 | 2019-01-13 17:04:28 -0800 | [diff] [blame] | 40 | } // namespace |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 41 | |
| 42 | namespace chromeos_update_engine { |
| 43 | |
| 44 | class PrefsTest : public ::testing::Test { |
| 45 | protected: |
Alex Deymo | 610277e | 2014-11-11 21:18:11 -0800 | [diff] [blame] | 46 | void SetUp() override { |
Sen Jiang | 371615b | 2016-04-13 15:54:29 -0700 | [diff] [blame] | 47 | ASSERT_TRUE(temp_dir_.CreateUniqueTempDir()); |
Eric Caruso | 7feac5b | 2018-01-23 16:20:19 -0800 | [diff] [blame] | 48 | prefs_dir_ = temp_dir_.GetPath(); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 49 | ASSERT_TRUE(prefs_.Init(prefs_dir_)); |
| 50 | } |
| 51 | |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 52 | bool SetValue(const string& key, const string& value) { |
Amin Hassani | b268959 | 2019-01-13 17:04:28 -0800 | [diff] [blame] | 53 | return base::WriteFile(prefs_dir_.Append(key), |
| 54 | value.data(), |
Ben Chan | 736fcb5 | 2014-05-21 18:28:22 -0700 | [diff] [blame] | 55 | value.length()) == static_cast<int>(value.length()); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 56 | } |
| 57 | |
Sen Jiang | 371615b | 2016-04-13 15:54:29 -0700 | [diff] [blame] | 58 | base::ScopedTempDir temp_dir_; |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 59 | base::FilePath prefs_dir_; |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 60 | Prefs prefs_; |
| 61 | }; |
| 62 | |
Andrew | 065d78d | 2020-04-07 15:43:07 -0700 | [diff] [blame] | 63 | TEST(Prefs, Init) { |
| 64 | Prefs prefs; |
Jae Hoon Kim | c1f3692 | 2020-05-11 18:20:18 -0700 | [diff] [blame^] | 65 | const string ns1 = "ns1"; |
| 66 | const string ns2A = "ns2A"; |
| 67 | const string ns2B = "ns2B"; |
Andrew | 065d78d | 2020-04-07 15:43:07 -0700 | [diff] [blame] | 68 | const string sub_pref = "sp"; |
| 69 | |
| 70 | base::ScopedTempDir temp_dir; |
| 71 | ASSERT_TRUE(temp_dir.CreateUniqueTempDir()); |
Jae Hoon Kim | c1f3692 | 2020-05-11 18:20:18 -0700 | [diff] [blame^] | 72 | auto ns1_path = temp_dir.GetPath().Append(ns1); |
| 73 | auto ns2A_path = ns1_path.Append(ns2A); |
| 74 | auto ns2B_path = ns1_path.Append(ns2B); |
| 75 | auto sub_pref_path = ns2A_path.Append(sub_pref); |
Andrew | 065d78d | 2020-04-07 15:43:07 -0700 | [diff] [blame] | 76 | |
Jae Hoon Kim | c1f3692 | 2020-05-11 18:20:18 -0700 | [diff] [blame^] | 77 | EXPECT_TRUE(base::CreateDirectory(ns2B_path)); |
| 78 | EXPECT_TRUE(base::PathExists(ns2B_path)); |
| 79 | |
| 80 | EXPECT_TRUE(base::CreateDirectory(sub_pref_path)); |
| 81 | EXPECT_TRUE(base::PathExists(sub_pref_path)); |
| 82 | |
| 83 | EXPECT_TRUE(base::PathExists(ns1_path)); |
Andrew | 065d78d | 2020-04-07 15:43:07 -0700 | [diff] [blame] | 84 | ASSERT_TRUE(prefs.Init(temp_dir.GetPath())); |
Jae Hoon Kim | c1f3692 | 2020-05-11 18:20:18 -0700 | [diff] [blame^] | 85 | EXPECT_FALSE(base::PathExists(ns1_path)); |
Andrew | 065d78d | 2020-04-07 15:43:07 -0700 | [diff] [blame] | 86 | } |
| 87 | |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 88 | TEST_F(PrefsTest, GetFileNameForKey) { |
Alex Deymo | d6f6007 | 2015-10-12 12:22:27 -0700 | [diff] [blame] | 89 | const char kAllvalidCharsKey[] = |
| 90 | "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz_-"; |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 91 | base::FilePath path; |
Alex Deymo | a0284ac | 2016-07-22 12:51:41 -0700 | [diff] [blame] | 92 | EXPECT_TRUE(prefs_.file_storage_.GetFileNameForKey(kAllvalidCharsKey, &path)); |
Alex Deymo | d6f6007 | 2015-10-12 12:22:27 -0700 | [diff] [blame] | 93 | EXPECT_EQ(prefs_dir_.Append(kAllvalidCharsKey).value(), path.value()); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 94 | } |
| 95 | |
| 96 | TEST_F(PrefsTest, GetFileNameForKeyBadCharacter) { |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 97 | base::FilePath path; |
Alex Deymo | a0284ac | 2016-07-22 12:51:41 -0700 | [diff] [blame] | 98 | EXPECT_FALSE(prefs_.file_storage_.GetFileNameForKey("ABC abc", &path)); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 99 | } |
| 100 | |
| 101 | TEST_F(PrefsTest, GetFileNameForKeyEmpty) { |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 102 | base::FilePath path; |
Alex Deymo | a0284ac | 2016-07-22 12:51:41 -0700 | [diff] [blame] | 103 | EXPECT_FALSE(prefs_.file_storage_.GetFileNameForKey("", &path)); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 104 | } |
| 105 | |
Andrew | 065d78d | 2020-04-07 15:43:07 -0700 | [diff] [blame] | 106 | TEST_F(PrefsTest, CreateSubKey) { |
| 107 | const string name_space = "ns"; |
| 108 | const string sub_pref1 = "sp1"; |
| 109 | const string sub_pref2 = "sp2"; |
| 110 | const string sub_key = "sk"; |
| 111 | |
Jae Hoon Kim | c1f3692 | 2020-05-11 18:20:18 -0700 | [diff] [blame^] | 112 | EXPECT_EQ(PrefsInterface::CreateSubKey({name_space, sub_pref1, sub_key}), |
Andrew | 065d78d | 2020-04-07 15:43:07 -0700 | [diff] [blame] | 113 | "ns/sp1/sk"); |
Jae Hoon Kim | c1f3692 | 2020-05-11 18:20:18 -0700 | [diff] [blame^] | 114 | EXPECT_EQ(PrefsInterface::CreateSubKey({name_space, sub_pref2, sub_key}), |
Andrew | 065d78d | 2020-04-07 15:43:07 -0700 | [diff] [blame] | 115 | "ns/sp2/sk"); |
| 116 | } |
| 117 | |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 118 | TEST_F(PrefsTest, GetString) { |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 119 | const string test_data = "test data"; |
| 120 | ASSERT_TRUE(SetValue(kKey, test_data)); |
| 121 | string value; |
| 122 | EXPECT_TRUE(prefs_.GetString(kKey, &value)); |
| 123 | EXPECT_EQ(test_data, value); |
| 124 | } |
| 125 | |
| 126 | TEST_F(PrefsTest, GetStringBadKey) { |
| 127 | string value; |
| 128 | EXPECT_FALSE(prefs_.GetString(",bad", &value)); |
| 129 | } |
| 130 | |
| 131 | TEST_F(PrefsTest, GetStringNonExistentKey) { |
| 132 | string value; |
| 133 | EXPECT_FALSE(prefs_.GetString("non-existent-key", &value)); |
| 134 | } |
| 135 | |
| 136 | TEST_F(PrefsTest, SetString) { |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 137 | const char kValue[] = "some test value\non 2 lines"; |
| 138 | EXPECT_TRUE(prefs_.SetString(kKey, kValue)); |
| 139 | string value; |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 140 | EXPECT_TRUE(base::ReadFileToString(prefs_dir_.Append(kKey), &value)); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 141 | EXPECT_EQ(kValue, value); |
| 142 | } |
| 143 | |
| 144 | TEST_F(PrefsTest, SetStringBadKey) { |
Alex Deymo | d6f6007 | 2015-10-12 12:22:27 -0700 | [diff] [blame] | 145 | const char kKeyWithDots[] = ".no-dots"; |
| 146 | EXPECT_FALSE(prefs_.SetString(kKeyWithDots, "some value")); |
| 147 | EXPECT_FALSE(base::PathExists(prefs_dir_.Append(kKeyWithDots))); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 148 | } |
| 149 | |
| 150 | TEST_F(PrefsTest, SetStringCreateDir) { |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 151 | const char kValue[] = "test value"; |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 152 | base::FilePath subdir = prefs_dir_.Append("subdir1").Append("subdir2"); |
Darin Petkov | 1cbd78f | 2010-07-29 12:38:34 -0700 | [diff] [blame] | 153 | EXPECT_TRUE(prefs_.Init(subdir)); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 154 | EXPECT_TRUE(prefs_.SetString(kKey, kValue)); |
| 155 | string value; |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 156 | EXPECT_TRUE(base::ReadFileToString(subdir.Append(kKey), &value)); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 157 | EXPECT_EQ(kValue, value); |
| 158 | } |
| 159 | |
| 160 | TEST_F(PrefsTest, SetStringDirCreationFailure) { |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 161 | EXPECT_TRUE(prefs_.Init(base::FilePath("/dev/null"))); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 162 | EXPECT_FALSE(prefs_.SetString(kKey, "test value")); |
| 163 | } |
| 164 | |
| 165 | TEST_F(PrefsTest, SetStringFileCreationFailure) { |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 166 | base::CreateDirectory(prefs_dir_.Append(kKey)); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 167 | EXPECT_FALSE(prefs_.SetString(kKey, "test value")); |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 168 | EXPECT_TRUE(base::DirectoryExists(prefs_dir_.Append(kKey))); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 169 | } |
| 170 | |
| 171 | TEST_F(PrefsTest, GetInt64) { |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 172 | ASSERT_TRUE(SetValue(kKey, " \n 25 \t ")); |
| 173 | int64_t value; |
| 174 | EXPECT_TRUE(prefs_.GetInt64(kKey, &value)); |
| 175 | EXPECT_EQ(25, value); |
| 176 | } |
| 177 | |
| 178 | TEST_F(PrefsTest, GetInt64BadValue) { |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 179 | ASSERT_TRUE(SetValue(kKey, "30a")); |
| 180 | int64_t value; |
| 181 | EXPECT_FALSE(prefs_.GetInt64(kKey, &value)); |
| 182 | } |
| 183 | |
| 184 | TEST_F(PrefsTest, GetInt64Max) { |
Amin Hassani | b268959 | 2019-01-13 17:04:28 -0800 | [diff] [blame] | 185 | ASSERT_TRUE(SetValue( |
| 186 | kKey, |
| 187 | base::StringPrintf("%" PRIi64, std::numeric_limits<int64_t>::max()))); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 188 | int64_t value; |
| 189 | EXPECT_TRUE(prefs_.GetInt64(kKey, &value)); |
Alex Deymo | 80f70ff | 2016-02-10 16:08:11 -0800 | [diff] [blame] | 190 | EXPECT_EQ(std::numeric_limits<int64_t>::max(), value); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 191 | } |
| 192 | |
| 193 | TEST_F(PrefsTest, GetInt64Min) { |
Amin Hassani | b268959 | 2019-01-13 17:04:28 -0800 | [diff] [blame] | 194 | ASSERT_TRUE(SetValue( |
| 195 | kKey, |
| 196 | base::StringPrintf("%" PRIi64, std::numeric_limits<int64_t>::min()))); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 197 | int64_t value; |
| 198 | EXPECT_TRUE(prefs_.GetInt64(kKey, &value)); |
Alex Deymo | 80f70ff | 2016-02-10 16:08:11 -0800 | [diff] [blame] | 199 | EXPECT_EQ(std::numeric_limits<int64_t>::min(), value); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 200 | } |
| 201 | |
| 202 | TEST_F(PrefsTest, GetInt64Negative) { |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 203 | ASSERT_TRUE(SetValue(kKey, " \t -100 \n ")); |
| 204 | int64_t value; |
| 205 | EXPECT_TRUE(prefs_.GetInt64(kKey, &value)); |
| 206 | EXPECT_EQ(-100, value); |
| 207 | } |
| 208 | |
| 209 | TEST_F(PrefsTest, GetInt64NonExistentKey) { |
| 210 | int64_t value; |
| 211 | EXPECT_FALSE(prefs_.GetInt64("random-key", &value)); |
| 212 | } |
| 213 | |
| 214 | TEST_F(PrefsTest, SetInt64) { |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 215 | EXPECT_TRUE(prefs_.SetInt64(kKey, -123)); |
| 216 | string value; |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 217 | EXPECT_TRUE(base::ReadFileToString(prefs_dir_.Append(kKey), &value)); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 218 | EXPECT_EQ("-123", value); |
| 219 | } |
| 220 | |
| 221 | TEST_F(PrefsTest, SetInt64BadKey) { |
Alex Deymo | d6f6007 | 2015-10-12 12:22:27 -0700 | [diff] [blame] | 222 | const char kKeyWithSpaces[] = "s p a c e s"; |
| 223 | EXPECT_FALSE(prefs_.SetInt64(kKeyWithSpaces, 20)); |
| 224 | EXPECT_FALSE(base::PathExists(prefs_dir_.Append(kKeyWithSpaces))); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 225 | } |
| 226 | |
| 227 | TEST_F(PrefsTest, SetInt64Max) { |
Alex Vakulenko | 0103c36 | 2016-01-20 07:56:15 -0800 | [diff] [blame] | 228 | EXPECT_TRUE(prefs_.SetInt64(kKey, std::numeric_limits<int64_t>::max())); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 229 | string value; |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 230 | EXPECT_TRUE(base::ReadFileToString(prefs_dir_.Append(kKey), &value)); |
Alex Vakulenko | 0103c36 | 2016-01-20 07:56:15 -0800 | [diff] [blame] | 231 | EXPECT_EQ(base::StringPrintf("%" PRIi64, std::numeric_limits<int64_t>::max()), |
| 232 | value); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 233 | } |
| 234 | |
| 235 | TEST_F(PrefsTest, SetInt64Min) { |
Alex Vakulenko | 0103c36 | 2016-01-20 07:56:15 -0800 | [diff] [blame] | 236 | EXPECT_TRUE(prefs_.SetInt64(kKey, std::numeric_limits<int64_t>::min())); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 237 | string value; |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 238 | EXPECT_TRUE(base::ReadFileToString(prefs_dir_.Append(kKey), &value)); |
Alex Vakulenko | 0103c36 | 2016-01-20 07:56:15 -0800 | [diff] [blame] | 239 | EXPECT_EQ(base::StringPrintf("%" PRIi64, std::numeric_limits<int64_t>::min()), |
| 240 | value); |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 241 | } |
| 242 | |
Alex Deymo | efb7c4c | 2013-07-09 14:34:00 -0700 | [diff] [blame] | 243 | TEST_F(PrefsTest, GetBooleanFalse) { |
Alex Deymo | efb7c4c | 2013-07-09 14:34:00 -0700 | [diff] [blame] | 244 | ASSERT_TRUE(SetValue(kKey, " \n false \t ")); |
| 245 | bool value; |
| 246 | EXPECT_TRUE(prefs_.GetBoolean(kKey, &value)); |
| 247 | EXPECT_FALSE(value); |
| 248 | } |
| 249 | |
| 250 | TEST_F(PrefsTest, GetBooleanTrue) { |
| 251 | const char kKey[] = "test-key"; |
| 252 | ASSERT_TRUE(SetValue(kKey, " \t true \n ")); |
| 253 | bool value; |
| 254 | EXPECT_TRUE(prefs_.GetBoolean(kKey, &value)); |
| 255 | EXPECT_TRUE(value); |
| 256 | } |
| 257 | |
| 258 | TEST_F(PrefsTest, GetBooleanBadValue) { |
| 259 | const char kKey[] = "test-key"; |
| 260 | ASSERT_TRUE(SetValue(kKey, "1")); |
| 261 | bool value; |
| 262 | EXPECT_FALSE(prefs_.GetBoolean(kKey, &value)); |
| 263 | } |
| 264 | |
| 265 | TEST_F(PrefsTest, GetBooleanBadEmptyValue) { |
| 266 | const char kKey[] = "test-key"; |
| 267 | ASSERT_TRUE(SetValue(kKey, "")); |
| 268 | bool value; |
| 269 | EXPECT_FALSE(prefs_.GetBoolean(kKey, &value)); |
| 270 | } |
| 271 | |
| 272 | TEST_F(PrefsTest, GetBooleanNonExistentKey) { |
| 273 | bool value; |
| 274 | EXPECT_FALSE(prefs_.GetBoolean("random-key", &value)); |
| 275 | } |
| 276 | |
| 277 | TEST_F(PrefsTest, SetBooleanTrue) { |
| 278 | const char kKey[] = "test-bool"; |
| 279 | EXPECT_TRUE(prefs_.SetBoolean(kKey, true)); |
| 280 | string value; |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 281 | EXPECT_TRUE(base::ReadFileToString(prefs_dir_.Append(kKey), &value)); |
Alex Deymo | efb7c4c | 2013-07-09 14:34:00 -0700 | [diff] [blame] | 282 | EXPECT_EQ("true", value); |
| 283 | } |
| 284 | |
| 285 | TEST_F(PrefsTest, SetBooleanFalse) { |
| 286 | const char kKey[] = "test-bool"; |
| 287 | EXPECT_TRUE(prefs_.SetBoolean(kKey, false)); |
| 288 | string value; |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 289 | EXPECT_TRUE(base::ReadFileToString(prefs_dir_.Append(kKey), &value)); |
Alex Deymo | efb7c4c | 2013-07-09 14:34:00 -0700 | [diff] [blame] | 290 | EXPECT_EQ("false", value); |
| 291 | } |
| 292 | |
| 293 | TEST_F(PrefsTest, SetBooleanBadKey) { |
| 294 | const char kKey[] = "s p a c e s"; |
| 295 | EXPECT_FALSE(prefs_.SetBoolean(kKey, true)); |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 296 | EXPECT_FALSE(base::PathExists(prefs_dir_.Append(kKey))); |
Alex Deymo | efb7c4c | 2013-07-09 14:34:00 -0700 | [diff] [blame] | 297 | } |
| 298 | |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 299 | TEST_F(PrefsTest, ExistsWorks) { |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 300 | // test that the key doesn't exist before we set it. |
| 301 | EXPECT_FALSE(prefs_.Exists(kKey)); |
| 302 | |
| 303 | // test that the key exists after we set it. |
| 304 | ASSERT_TRUE(prefs_.SetInt64(kKey, 8)); |
| 305 | EXPECT_TRUE(prefs_.Exists(kKey)); |
| 306 | } |
| 307 | |
| 308 | TEST_F(PrefsTest, DeleteWorks) { |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 309 | // test that it's alright to delete a non-existent key. |
| 310 | EXPECT_TRUE(prefs_.Delete(kKey)); |
| 311 | |
| 312 | // delete the key after we set it. |
| 313 | ASSERT_TRUE(prefs_.SetInt64(kKey, 0)); |
| 314 | EXPECT_TRUE(prefs_.Delete(kKey)); |
| 315 | |
| 316 | // make sure it doesn't exist anymore. |
| 317 | EXPECT_FALSE(prefs_.Exists(kKey)); |
| 318 | } |
| 319 | |
Andrew | 065d78d | 2020-04-07 15:43:07 -0700 | [diff] [blame] | 320 | TEST_F(PrefsTest, SetDeleteSubKey) { |
| 321 | const string name_space = "ns"; |
| 322 | const string sub_pref = "sp"; |
| 323 | const string sub_key1 = "sk1"; |
| 324 | const string sub_key2 = "sk2"; |
Jae Hoon Kim | c1f3692 | 2020-05-11 18:20:18 -0700 | [diff] [blame^] | 325 | auto key1 = prefs_.CreateSubKey({name_space, sub_pref, sub_key1}); |
| 326 | auto key2 = prefs_.CreateSubKey({name_space, sub_pref, sub_key2}); |
Andrew | 065d78d | 2020-04-07 15:43:07 -0700 | [diff] [blame] | 327 | base::FilePath sub_pref_path = prefs_dir_.Append(name_space).Append(sub_pref); |
| 328 | |
| 329 | ASSERT_TRUE(prefs_.SetInt64(key1, 0)); |
| 330 | ASSERT_TRUE(prefs_.SetInt64(key2, 0)); |
| 331 | EXPECT_TRUE(base::PathExists(sub_pref_path.Append(sub_key1))); |
| 332 | EXPECT_TRUE(base::PathExists(sub_pref_path.Append(sub_key2))); |
| 333 | |
| 334 | ASSERT_TRUE(prefs_.Delete(key1)); |
| 335 | EXPECT_FALSE(base::PathExists(sub_pref_path.Append(sub_key1))); |
| 336 | EXPECT_TRUE(base::PathExists(sub_pref_path.Append(sub_key2))); |
| 337 | ASSERT_TRUE(prefs_.Delete(key2)); |
| 338 | EXPECT_FALSE(base::PathExists(sub_pref_path.Append(sub_key2))); |
| 339 | prefs_.Init(prefs_dir_); |
| 340 | EXPECT_FALSE(base::PathExists(prefs_dir_.Append(name_space))); |
| 341 | } |
| 342 | |
Alex Deymo | d6f6007 | 2015-10-12 12:22:27 -0700 | [diff] [blame] | 343 | class MockPrefsObserver : public PrefsInterface::ObserverInterface { |
| 344 | public: |
| 345 | MOCK_METHOD1(OnPrefSet, void(const string&)); |
| 346 | MOCK_METHOD1(OnPrefDeleted, void(const string& key)); |
| 347 | }; |
| 348 | |
| 349 | TEST_F(PrefsTest, ObserversCalled) { |
| 350 | MockPrefsObserver mock_obserser; |
| 351 | prefs_.AddObserver(kKey, &mock_obserser); |
| 352 | |
| 353 | EXPECT_CALL(mock_obserser, OnPrefSet(Eq(kKey))); |
| 354 | EXPECT_CALL(mock_obserser, OnPrefDeleted(_)).Times(0); |
| 355 | prefs_.SetString(kKey, "value"); |
| 356 | testing::Mock::VerifyAndClearExpectations(&mock_obserser); |
| 357 | |
| 358 | EXPECT_CALL(mock_obserser, OnPrefSet(_)).Times(0); |
| 359 | EXPECT_CALL(mock_obserser, OnPrefDeleted(Eq(kKey))); |
| 360 | prefs_.Delete(kKey); |
| 361 | testing::Mock::VerifyAndClearExpectations(&mock_obserser); |
| 362 | |
Jae Hoon Kim | c1f3692 | 2020-05-11 18:20:18 -0700 | [diff] [blame^] | 363 | auto key1 = prefs_.CreateSubKey({"ns", "sp1", "key1"}); |
Andrew | 065d78d | 2020-04-07 15:43:07 -0700 | [diff] [blame] | 364 | prefs_.AddObserver(key1, &mock_obserser); |
| 365 | |
| 366 | EXPECT_CALL(mock_obserser, OnPrefSet(key1)); |
| 367 | EXPECT_CALL(mock_obserser, OnPrefDeleted(_)).Times(0); |
| 368 | prefs_.SetString(key1, "value"); |
| 369 | testing::Mock::VerifyAndClearExpectations(&mock_obserser); |
| 370 | |
| 371 | EXPECT_CALL(mock_obserser, OnPrefSet(_)).Times(0); |
| 372 | EXPECT_CALL(mock_obserser, OnPrefDeleted(Eq(key1))); |
| 373 | prefs_.Delete(key1); |
| 374 | testing::Mock::VerifyAndClearExpectations(&mock_obserser); |
| 375 | |
Alex Deymo | d6f6007 | 2015-10-12 12:22:27 -0700 | [diff] [blame] | 376 | prefs_.RemoveObserver(kKey, &mock_obserser); |
| 377 | } |
| 378 | |
| 379 | TEST_F(PrefsTest, OnlyCalledOnObservedKeys) { |
| 380 | MockPrefsObserver mock_obserser; |
| 381 | const char kUnusedKey[] = "unused-key"; |
| 382 | prefs_.AddObserver(kUnusedKey, &mock_obserser); |
| 383 | |
| 384 | EXPECT_CALL(mock_obserser, OnPrefSet(_)).Times(0); |
| 385 | EXPECT_CALL(mock_obserser, OnPrefDeleted(_)).Times(0); |
| 386 | prefs_.SetString(kKey, "value"); |
| 387 | prefs_.Delete(kKey); |
| 388 | |
| 389 | prefs_.RemoveObserver(kUnusedKey, &mock_obserser); |
| 390 | } |
| 391 | |
| 392 | TEST_F(PrefsTest, RemovedObserversNotCalled) { |
| 393 | MockPrefsObserver mock_obserser_a, mock_obserser_b; |
| 394 | prefs_.AddObserver(kKey, &mock_obserser_a); |
| 395 | prefs_.AddObserver(kKey, &mock_obserser_b); |
| 396 | EXPECT_CALL(mock_obserser_a, OnPrefSet(_)).Times(2); |
| 397 | EXPECT_CALL(mock_obserser_b, OnPrefSet(_)).Times(1); |
| 398 | EXPECT_TRUE(prefs_.SetString(kKey, "value")); |
| 399 | prefs_.RemoveObserver(kKey, &mock_obserser_b); |
| 400 | EXPECT_TRUE(prefs_.SetString(kKey, "other value")); |
| 401 | prefs_.RemoveObserver(kKey, &mock_obserser_a); |
| 402 | EXPECT_TRUE(prefs_.SetString(kKey, "yet another value")); |
| 403 | } |
| 404 | |
| 405 | TEST_F(PrefsTest, UnsuccessfulCallsNotObserved) { |
| 406 | MockPrefsObserver mock_obserser; |
| 407 | const char kInvalidKey[] = "no spaces or ."; |
| 408 | prefs_.AddObserver(kInvalidKey, &mock_obserser); |
| 409 | |
| 410 | EXPECT_CALL(mock_obserser, OnPrefSet(_)).Times(0); |
| 411 | EXPECT_CALL(mock_obserser, OnPrefDeleted(_)).Times(0); |
| 412 | EXPECT_FALSE(prefs_.SetString(kInvalidKey, "value")); |
| 413 | EXPECT_FALSE(prefs_.Delete(kInvalidKey)); |
| 414 | |
| 415 | prefs_.RemoveObserver(kInvalidKey, &mock_obserser); |
| 416 | } |
| 417 | |
Alex Deymo | a0284ac | 2016-07-22 12:51:41 -0700 | [diff] [blame] | 418 | class MemoryPrefsTest : public ::testing::Test { |
| 419 | protected: |
| 420 | MemoryPrefs prefs_; |
| 421 | }; |
| 422 | |
| 423 | TEST_F(MemoryPrefsTest, BasicTest) { |
| 424 | EXPECT_FALSE(prefs_.Exists(kKey)); |
| 425 | int64_t value = 0; |
| 426 | EXPECT_FALSE(prefs_.GetInt64(kKey, &value)); |
| 427 | |
| 428 | EXPECT_TRUE(prefs_.SetInt64(kKey, 1234)); |
| 429 | EXPECT_TRUE(prefs_.Exists(kKey)); |
| 430 | EXPECT_TRUE(prefs_.GetInt64(kKey, &value)); |
| 431 | EXPECT_EQ(1234, value); |
| 432 | |
| 433 | EXPECT_TRUE(prefs_.Delete(kKey)); |
| 434 | EXPECT_FALSE(prefs_.Exists(kKey)); |
Andrew | 914f554 | 2020-04-21 10:56:33 -0700 | [diff] [blame] | 435 | EXPECT_TRUE(prefs_.Delete(kKey)); |
Andrew | 065d78d | 2020-04-07 15:43:07 -0700 | [diff] [blame] | 436 | |
Jae Hoon Kim | c1f3692 | 2020-05-11 18:20:18 -0700 | [diff] [blame^] | 437 | auto key = prefs_.CreateSubKey({"ns", "sp", "sk"}); |
Andrew | 065d78d | 2020-04-07 15:43:07 -0700 | [diff] [blame] | 438 | ASSERT_TRUE(prefs_.SetInt64(key, 0)); |
| 439 | EXPECT_TRUE(prefs_.Exists(key)); |
| 440 | EXPECT_TRUE(prefs_.Delete(kKey)); |
Alex Deymo | a0284ac | 2016-07-22 12:51:41 -0700 | [diff] [blame] | 441 | } |
| 442 | |
Darin Petkov | 3003059 | 2010-07-27 13:53:20 -0700 | [diff] [blame] | 443 | } // namespace chromeos_update_engine |