blob: 5c506d4320837152d0d2f2a61a32d770594f10c7 [file] [log] [blame]
joi@chromium.org026cee42012-10-23 05:01:52 +09001// Copyright (c) 2012 The Chromium Authors. All rights reserved.
2// Use of this source code is governed by a BSD-style license that can be
3// found in the LICENSE file.
4
tfarina@chromium.orgfe046b52012-11-28 04:09:17 +09005#include "base/prefs/json_pref_store.h"
6
joi@chromium.org026cee42012-10-23 05:01:52 +09007#include "base/file_util.h"
brettw@chromium.org091db522012-11-17 05:34:23 +09008#include "base/files/scoped_temp_dir.h"
joi@chromium.org026cee42012-10-23 05:01:52 +09009#include "base/memory/ref_counted.h"
10#include "base/memory/scoped_ptr.h"
joi@chromium.org026cee42012-10-23 05:01:52 +090011#include "base/path_service.h"
tfarina@chromium.orge9e11f52013-02-05 17:14:15 +090012#include "base/run_loop.h"
brettw@chromium.orgabcde5c2013-02-07 11:57:22 +090013#include "base/strings/string_number_conversions.h"
avi@chromium.org67d593d2013-06-11 04:06:57 +090014#include "base/strings/string_util.h"
avi@chromium.org17f60622013-06-08 03:37:07 +090015#include "base/strings/utf_string_conversions.h"
zelidrag@chromium.org011ad402012-11-08 13:40:59 +090016#include "base/threading/sequenced_worker_pool.h"
joi@chromium.org026cee42012-10-23 05:01:52 +090017#include "base/threading/thread.h"
joi@chromium.org026cee42012-10-23 05:01:52 +090018#include "base/values.h"
joi@chromium.org026cee42012-10-23 05:01:52 +090019#include "testing/gmock/include/gmock/gmock.h"
20#include "testing/gtest/include/gtest/gtest.h"
21
tfarina@chromium.org50be5762012-12-16 14:13:21 +090022namespace base {
joi@chromium.org026cee42012-10-23 05:01:52 +090023namespace {
24
tfarina@chromium.orgad1d8ce2012-11-23 07:08:24 +090025const char kHomePage[] = "homepage";
26
joi@chromium.org026cee42012-10-23 05:01:52 +090027class MockPrefStoreObserver : public PrefStore::Observer {
28 public:
29 MOCK_METHOD1(OnPrefValueChanged, void (const std::string&));
30 MOCK_METHOD1(OnInitializationCompleted, void (bool));
31};
32
33class MockReadErrorDelegate : public PersistentPrefStore::ReadErrorDelegate {
34 public:
35 MOCK_METHOD1(OnError, void(PersistentPrefStore::PrefReadError));
36};
37
38} // namespace
39
40class JsonPrefStoreTest : public testing::Test {
41 protected:
zelidrag@chromium.org011ad402012-11-08 13:40:59 +090042 virtual void SetUp() OVERRIDE {
joi@chromium.org026cee42012-10-23 05:01:52 +090043 ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
44
tfarina@chromium.orgd05540d2013-04-08 01:27:46 +090045 ASSERT_TRUE(PathService::Get(base::DIR_TEST_DATA, &data_dir_));
tfarina@chromium.orgfe046b52012-11-28 04:09:17 +090046 data_dir_ = data_dir_.AppendASCII("prefs");
brettw@chromium.org10b64122013-07-12 02:36:07 +090047 ASSERT_TRUE(PathExists(data_dir_));
joi@chromium.org026cee42012-10-23 05:01:52 +090048 }
49
50 // The path to temporary directory used to contain the test operations.
brettw@chromium.org091db522012-11-17 05:34:23 +090051 base::ScopedTempDir temp_dir_;
joi@chromium.org026cee42012-10-23 05:01:52 +090052 // The path to the directory where the test data is stored.
brettw@chromium.org82bcf512013-02-17 14:07:23 +090053 base::FilePath data_dir_;
joi@chromium.org026cee42012-10-23 05:01:52 +090054 // A message loop that we can use as the file thread message loop.
55 MessageLoop message_loop_;
joi@chromium.org026cee42012-10-23 05:01:52 +090056};
57
58// Test fallback behavior for a nonexistent file.
59TEST_F(JsonPrefStoreTest, NonExistentFile) {
brettw@chromium.org82bcf512013-02-17 14:07:23 +090060 base::FilePath bogus_input_file = data_dir_.AppendASCII("read.txt");
brettw@chromium.org10b64122013-07-12 02:36:07 +090061 ASSERT_FALSE(PathExists(bogus_input_file));
rsleevi@chromium.org64293f12013-06-12 01:46:36 +090062 scoped_refptr<JsonPrefStore> pref_store = new JsonPrefStore(
63 bogus_input_file, message_loop_.message_loop_proxy().get());
joi@chromium.org026cee42012-10-23 05:01:52 +090064 EXPECT_EQ(PersistentPrefStore::PREF_READ_ERROR_NO_FILE,
65 pref_store->ReadPrefs());
66 EXPECT_FALSE(pref_store->ReadOnly());
67}
68
69// Test fallback behavior for an invalid file.
70TEST_F(JsonPrefStoreTest, InvalidFile) {
brettw@chromium.org82bcf512013-02-17 14:07:23 +090071 base::FilePath invalid_file_original = data_dir_.AppendASCII("invalid.json");
72 base::FilePath invalid_file = temp_dir_.path().AppendASCII("invalid.json");
brettw@chromium.orgaecf7a32013-07-10 02:42:26 +090073 ASSERT_TRUE(base::CopyFile(invalid_file_original, invalid_file));
joi@chromium.org026cee42012-10-23 05:01:52 +090074 scoped_refptr<JsonPrefStore> pref_store =
rsleevi@chromium.org64293f12013-06-12 01:46:36 +090075 new JsonPrefStore(invalid_file, message_loop_.message_loop_proxy().get());
joi@chromium.org026cee42012-10-23 05:01:52 +090076 EXPECT_EQ(PersistentPrefStore::PREF_READ_ERROR_JSON_PARSE,
77 pref_store->ReadPrefs());
78 EXPECT_FALSE(pref_store->ReadOnly());
79
80 // The file should have been moved aside.
brettw@chromium.org10b64122013-07-12 02:36:07 +090081 EXPECT_FALSE(PathExists(invalid_file));
brettw@chromium.org82bcf512013-02-17 14:07:23 +090082 base::FilePath moved_aside = temp_dir_.path().AppendASCII("invalid.bad");
brettw@chromium.org10b64122013-07-12 02:36:07 +090083 EXPECT_TRUE(PathExists(moved_aside));
joi@chromium.org026cee42012-10-23 05:01:52 +090084 EXPECT_TRUE(file_util::TextContentsEqual(invalid_file_original,
85 moved_aside));
86}
87
88// This function is used to avoid code duplication while testing synchronous and
89// asynchronous version of the JsonPrefStore loading.
zelidrag@chromium.org011ad402012-11-08 13:40:59 +090090void RunBasicJsonPrefStoreTest(JsonPrefStore* pref_store,
brettw@chromium.org82bcf512013-02-17 14:07:23 +090091 const base::FilePath& output_file,
92 const base::FilePath& golden_output_file) {
joi@chromium.org026cee42012-10-23 05:01:52 +090093 const char kNewWindowsInTabs[] = "tabs.new_windows_in_tabs";
94 const char kMaxTabs[] = "tabs.max_tabs";
95 const char kLongIntPref[] = "long_int.pref";
96
97 std::string cnn("http://www.cnn.com");
98
99 const Value* actual;
tfarina@chromium.orgad1d8ce2012-11-23 07:08:24 +0900100 EXPECT_TRUE(pref_store->GetValue(kHomePage, &actual));
joi@chromium.org026cee42012-10-23 05:01:52 +0900101 std::string string_value;
102 EXPECT_TRUE(actual->GetAsString(&string_value));
103 EXPECT_EQ(cnn, string_value);
104
105 const char kSomeDirectory[] = "some_directory";
106
bauerb@chromium.org17fcbc82012-11-09 03:24:34 +0900107 EXPECT_TRUE(pref_store->GetValue(kSomeDirectory, &actual));
brettw@chromium.org82bcf512013-02-17 14:07:23 +0900108 base::FilePath::StringType path;
joi@chromium.org026cee42012-10-23 05:01:52 +0900109 EXPECT_TRUE(actual->GetAsString(&path));
brettw@chromium.org82bcf512013-02-17 14:07:23 +0900110 EXPECT_EQ(base::FilePath::StringType(FILE_PATH_LITERAL("/usr/local/")), path);
111 base::FilePath some_path(FILE_PATH_LITERAL("/usr/sbin/"));
joi@chromium.org026cee42012-10-23 05:01:52 +0900112
tfarina@chromium.org50be5762012-12-16 14:13:21 +0900113 pref_store->SetValue(kSomeDirectory, new StringValue(some_path.value()));
bauerb@chromium.org17fcbc82012-11-09 03:24:34 +0900114 EXPECT_TRUE(pref_store->GetValue(kSomeDirectory, &actual));
joi@chromium.org026cee42012-10-23 05:01:52 +0900115 EXPECT_TRUE(actual->GetAsString(&path));
116 EXPECT_EQ(some_path.value(), path);
117
118 // Test reading some other data types from sub-dictionaries.
bauerb@chromium.org17fcbc82012-11-09 03:24:34 +0900119 EXPECT_TRUE(pref_store->GetValue(kNewWindowsInTabs, &actual));
joi@chromium.org026cee42012-10-23 05:01:52 +0900120 bool boolean = false;
121 EXPECT_TRUE(actual->GetAsBoolean(&boolean));
122 EXPECT_TRUE(boolean);
123
tfarina@chromium.org50be5762012-12-16 14:13:21 +0900124 pref_store->SetValue(kNewWindowsInTabs, new FundamentalValue(false));
bauerb@chromium.org17fcbc82012-11-09 03:24:34 +0900125 EXPECT_TRUE(pref_store->GetValue(kNewWindowsInTabs, &actual));
joi@chromium.org026cee42012-10-23 05:01:52 +0900126 EXPECT_TRUE(actual->GetAsBoolean(&boolean));
127 EXPECT_FALSE(boolean);
128
bauerb@chromium.org17fcbc82012-11-09 03:24:34 +0900129 EXPECT_TRUE(pref_store->GetValue(kMaxTabs, &actual));
joi@chromium.org026cee42012-10-23 05:01:52 +0900130 int integer = 0;
131 EXPECT_TRUE(actual->GetAsInteger(&integer));
132 EXPECT_EQ(20, integer);
tfarina@chromium.org50be5762012-12-16 14:13:21 +0900133 pref_store->SetValue(kMaxTabs, new FundamentalValue(10));
bauerb@chromium.org17fcbc82012-11-09 03:24:34 +0900134 EXPECT_TRUE(pref_store->GetValue(kMaxTabs, &actual));
joi@chromium.org026cee42012-10-23 05:01:52 +0900135 EXPECT_TRUE(actual->GetAsInteger(&integer));
136 EXPECT_EQ(10, integer);
137
138 pref_store->SetValue(kLongIntPref,
tfarina@chromium.org50be5762012-12-16 14:13:21 +0900139 new StringValue(base::Int64ToString(214748364842LL)));
bauerb@chromium.org17fcbc82012-11-09 03:24:34 +0900140 EXPECT_TRUE(pref_store->GetValue(kLongIntPref, &actual));
joi@chromium.org026cee42012-10-23 05:01:52 +0900141 EXPECT_TRUE(actual->GetAsString(&string_value));
142 int64 value;
143 base::StringToInt64(string_value, &value);
144 EXPECT_EQ(214748364842LL, value);
145
146 // Serialize and compare to expected output.
brettw@chromium.org10b64122013-07-12 02:36:07 +0900147 ASSERT_TRUE(PathExists(golden_output_file));
joi@chromium.org026cee42012-10-23 05:01:52 +0900148 pref_store->CommitPendingWrite();
tfarina@chromium.orge9e11f52013-02-05 17:14:15 +0900149 RunLoop().RunUntilIdle();
joi@chromium.org026cee42012-10-23 05:01:52 +0900150 EXPECT_TRUE(file_util::TextContentsEqual(golden_output_file, output_file));
brettw@chromium.orge9f99482013-07-02 04:41:02 +0900151 ASSERT_TRUE(base::Delete(output_file, false));
joi@chromium.org026cee42012-10-23 05:01:52 +0900152}
153
154TEST_F(JsonPrefStoreTest, Basic) {
brettw@chromium.orgaecf7a32013-07-10 02:42:26 +0900155 ASSERT_TRUE(base::CopyFile(data_dir_.AppendASCII("read.json"),
joi@chromium.org026cee42012-10-23 05:01:52 +0900156 temp_dir_.path().AppendASCII("write.json")));
157
158 // Test that the persistent value can be loaded.
brettw@chromium.org82bcf512013-02-17 14:07:23 +0900159 base::FilePath input_file = temp_dir_.path().AppendASCII("write.json");
brettw@chromium.org10b64122013-07-12 02:36:07 +0900160 ASSERT_TRUE(PathExists(input_file));
joi@chromium.org026cee42012-10-23 05:01:52 +0900161 scoped_refptr<JsonPrefStore> pref_store =
rsleevi@chromium.org64293f12013-06-12 01:46:36 +0900162 new JsonPrefStore(input_file, message_loop_.message_loop_proxy().get());
joi@chromium.org026cee42012-10-23 05:01:52 +0900163 ASSERT_EQ(PersistentPrefStore::PREF_READ_ERROR_NONE, pref_store->ReadPrefs());
164 ASSERT_FALSE(pref_store->ReadOnly());
165
166 // The JSON file looks like this:
167 // {
168 // "homepage": "http://www.cnn.com",
169 // "some_directory": "/usr/local/",
170 // "tabs": {
171 // "new_windows_in_tabs": true,
172 // "max_tabs": 20
173 // }
174 // }
175
rsleevi@chromium.org7501b032013-06-01 06:37:53 +0900176 RunBasicJsonPrefStoreTest(
177 pref_store.get(), input_file, data_dir_.AppendASCII("write.golden.json"));
joi@chromium.org026cee42012-10-23 05:01:52 +0900178}
179
180TEST_F(JsonPrefStoreTest, BasicAsync) {
brettw@chromium.orgaecf7a32013-07-10 02:42:26 +0900181 ASSERT_TRUE(base::CopyFile(data_dir_.AppendASCII("read.json"),
joi@chromium.org026cee42012-10-23 05:01:52 +0900182 temp_dir_.path().AppendASCII("write.json")));
183
184 // Test that the persistent value can be loaded.
brettw@chromium.org82bcf512013-02-17 14:07:23 +0900185 base::FilePath input_file = temp_dir_.path().AppendASCII("write.json");
brettw@chromium.org10b64122013-07-12 02:36:07 +0900186 ASSERT_TRUE(PathExists(input_file));
joi@chromium.org026cee42012-10-23 05:01:52 +0900187 scoped_refptr<JsonPrefStore> pref_store =
rsleevi@chromium.org64293f12013-06-12 01:46:36 +0900188 new JsonPrefStore(input_file, message_loop_.message_loop_proxy().get());
joi@chromium.org026cee42012-10-23 05:01:52 +0900189
zelidrag@chromium.org011ad402012-11-08 13:40:59 +0900190 {
191 MockPrefStoreObserver mock_observer;
192 pref_store->AddObserver(&mock_observer);
joi@chromium.org026cee42012-10-23 05:01:52 +0900193
zelidrag@chromium.org011ad402012-11-08 13:40:59 +0900194 MockReadErrorDelegate* mock_error_delegate = new MockReadErrorDelegate;
195 pref_store->ReadPrefsAsync(mock_error_delegate);
joi@chromium.org026cee42012-10-23 05:01:52 +0900196
zelidrag@chromium.org011ad402012-11-08 13:40:59 +0900197 EXPECT_CALL(mock_observer, OnInitializationCompleted(true)).Times(1);
198 EXPECT_CALL(*mock_error_delegate,
199 OnError(PersistentPrefStore::PREF_READ_ERROR_NONE)).Times(0);
tfarina@chromium.org7d4a0ec2013-02-07 01:56:19 +0900200 RunLoop().RunUntilIdle();
zelidrag@chromium.org011ad402012-11-08 13:40:59 +0900201 pref_store->RemoveObserver(&mock_observer);
joi@chromium.org026cee42012-10-23 05:01:52 +0900202
zelidrag@chromium.org011ad402012-11-08 13:40:59 +0900203 ASSERT_FALSE(pref_store->ReadOnly());
204 }
joi@chromium.org026cee42012-10-23 05:01:52 +0900205
206 // The JSON file looks like this:
207 // {
208 // "homepage": "http://www.cnn.com",
209 // "some_directory": "/usr/local/",
210 // "tabs": {
211 // "new_windows_in_tabs": true,
212 // "max_tabs": 20
213 // }
214 // }
215
rsleevi@chromium.org7501b032013-06-01 06:37:53 +0900216 RunBasicJsonPrefStoreTest(
217 pref_store.get(), input_file, data_dir_.AppendASCII("write.golden.json"));
joi@chromium.org026cee42012-10-23 05:01:52 +0900218}
219
220// Tests asynchronous reading of the file when there is no file.
221TEST_F(JsonPrefStoreTest, AsyncNonExistingFile) {
brettw@chromium.org82bcf512013-02-17 14:07:23 +0900222 base::FilePath bogus_input_file = data_dir_.AppendASCII("read.txt");
brettw@chromium.org10b64122013-07-12 02:36:07 +0900223 ASSERT_FALSE(PathExists(bogus_input_file));
rsleevi@chromium.org64293f12013-06-12 01:46:36 +0900224 scoped_refptr<JsonPrefStore> pref_store = new JsonPrefStore(
225 bogus_input_file, message_loop_.message_loop_proxy().get());
joi@chromium.org026cee42012-10-23 05:01:52 +0900226 MockPrefStoreObserver mock_observer;
227 pref_store->AddObserver(&mock_observer);
228
229 MockReadErrorDelegate *mock_error_delegate = new MockReadErrorDelegate;
230 pref_store->ReadPrefsAsync(mock_error_delegate);
231
232 EXPECT_CALL(mock_observer, OnInitializationCompleted(true)).Times(1);
233 EXPECT_CALL(*mock_error_delegate,
234 OnError(PersistentPrefStore::PREF_READ_ERROR_NO_FILE)).Times(1);
tfarina@chromium.org7d4a0ec2013-02-07 01:56:19 +0900235 RunLoop().RunUntilIdle();
joi@chromium.org026cee42012-10-23 05:01:52 +0900236 pref_store->RemoveObserver(&mock_observer);
237
238 EXPECT_FALSE(pref_store->ReadOnly());
239}
240
241TEST_F(JsonPrefStoreTest, NeedsEmptyValue) {
brettw@chromium.org82bcf512013-02-17 14:07:23 +0900242 base::FilePath pref_file = temp_dir_.path().AppendASCII("write.json");
joi@chromium.org026cee42012-10-23 05:01:52 +0900243
brettw@chromium.orgaecf7a32013-07-10 02:42:26 +0900244 ASSERT_TRUE(base::CopyFile(
joi@chromium.org026cee42012-10-23 05:01:52 +0900245 data_dir_.AppendASCII("read.need_empty_value.json"),
246 pref_file));
247
248 // Test that the persistent value can be loaded.
brettw@chromium.org10b64122013-07-12 02:36:07 +0900249 ASSERT_TRUE(PathExists(pref_file));
joi@chromium.org026cee42012-10-23 05:01:52 +0900250 scoped_refptr<JsonPrefStore> pref_store =
rsleevi@chromium.org64293f12013-06-12 01:46:36 +0900251 new JsonPrefStore(pref_file, message_loop_.message_loop_proxy().get());
joi@chromium.org026cee42012-10-23 05:01:52 +0900252 ASSERT_EQ(PersistentPrefStore::PREF_READ_ERROR_NONE, pref_store->ReadPrefs());
253 ASSERT_FALSE(pref_store->ReadOnly());
254
255 // The JSON file looks like this:
256 // {
257 // "list": [ 1 ],
258 // "list_needs_empty_value": [ 2 ],
259 // "dict": {
260 // "dummy": true,
261 // },
262 // "dict_needs_empty_value": {
263 // "dummy": true,
264 // },
265 // }
266
267 // Set flag to preserve empty values for the following keys.
268 pref_store->MarkNeedsEmptyValue("list_needs_empty_value");
269 pref_store->MarkNeedsEmptyValue("dict_needs_empty_value");
270
271 // Set all keys to empty values.
272 pref_store->SetValue("list", new base::ListValue);
273 pref_store->SetValue("list_needs_empty_value", new base::ListValue);
274 pref_store->SetValue("dict", new base::DictionaryValue);
275 pref_store->SetValue("dict_needs_empty_value", new base::DictionaryValue);
276
277 // Write to file.
278 pref_store->CommitPendingWrite();
tfarina@chromium.orge9e11f52013-02-05 17:14:15 +0900279 RunLoop().RunUntilIdle();
joi@chromium.org026cee42012-10-23 05:01:52 +0900280
281 // Compare to expected output.
brettw@chromium.org82bcf512013-02-17 14:07:23 +0900282 base::FilePath golden_output_file =
joi@chromium.org026cee42012-10-23 05:01:52 +0900283 data_dir_.AppendASCII("write.golden.need_empty_value.json");
brettw@chromium.org10b64122013-07-12 02:36:07 +0900284 ASSERT_TRUE(PathExists(golden_output_file));
joi@chromium.org026cee42012-10-23 05:01:52 +0900285 EXPECT_TRUE(file_util::TextContentsEqual(golden_output_file, pref_file));
286}
tfarina@chromium.org50be5762012-12-16 14:13:21 +0900287
288} // namespace base