Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(993)

Side by Side Diff: base/values_unittest.cc

Issue 2516363005: Inline StringValue into base::Value (Closed)
Patch Set: Rebase and Nits. Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « base/values.cc ('k') | chrome/browser/chromeos/settings/system_settings_provider.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // 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 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/values.h" 5 #include "base/values.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <limits> 9 #include <limits>
10 #include <memory> 10 #include <memory>
(...skipping 22 matching lines...) Expand all
33 EXPECT_EQ(Value::Type::INTEGER, value.type()); 33 EXPECT_EQ(Value::Type::INTEGER, value.type());
34 EXPECT_EQ(-37, value.GetInt()); 34 EXPECT_EQ(-37, value.GetInt());
35 } 35 }
36 36
37 TEST(ValuesTest, ConstructDouble) { 37 TEST(ValuesTest, ConstructDouble) {
38 FundamentalValue value(-4.655); 38 FundamentalValue value(-4.655);
39 EXPECT_EQ(Value::Type::DOUBLE, value.type()); 39 EXPECT_EQ(Value::Type::DOUBLE, value.type());
40 EXPECT_EQ(-4.655, value.GetDouble()); 40 EXPECT_EQ(-4.655, value.GetDouble());
41 } 41 }
42 42
43 TEST(ValuesTest, ConstructStringFromConstCharPtr) {
44 const char* str = "foobar";
45 StringValue value(str);
46 EXPECT_EQ(Value::Type::STRING, value.type());
47 EXPECT_EQ("foobar", value.GetString());
48 }
49
50 TEST(ValuesTest, ConstructStringFromStdStringConstRef) {
51 std::string str = "foobar";
52 StringValue value(str);
53 EXPECT_EQ(Value::Type::STRING, value.type());
54 EXPECT_EQ("foobar", value.GetString());
55 }
56
57 TEST(ValuesTest, ConstructStringFromStdStringRefRef) {
58 std::string str = "foobar";
59 StringValue value(std::move(str));
60 EXPECT_EQ(Value::Type::STRING, value.type());
61 EXPECT_EQ("foobar", value.GetString());
62 }
63
64 TEST(ValuesTest, ConstructStringFromConstChar16Ptr) {
65 string16 str = ASCIIToUTF16("foobar");
66 StringValue value(str.c_str());
67 EXPECT_EQ(Value::Type::STRING, value.type());
68 EXPECT_EQ("foobar", value.GetString());
69 }
70
71 TEST(ValuesTest, ConstructStringFromString16) {
72 string16 str = ASCIIToUTF16("foobar");
73 StringValue value(str);
74 EXPECT_EQ(Value::Type::STRING, value.type());
75 EXPECT_EQ("foobar", value.GetString());
76 }
77
78 TEST(ValuesTest, ConstructStringFromStringPiece) {
79 StringPiece str = "foobar";
80 StringValue value(str);
81 EXPECT_EQ(Value::Type::STRING, value.type());
82 EXPECT_EQ("foobar", value.GetString());
83 }
84
43 // Group of tests for the copy constructors and copy-assigmnent. For equality 85 // Group of tests for the copy constructors and copy-assigmnent. For equality
44 // checks comparisons of the interesting fields are done instead of relying on 86 // checks comparisons of the interesting fields are done instead of relying on
45 // Equals being correct. 87 // Equals being correct.
46 TEST(ValuesTest, CopyBool) { 88 TEST(ValuesTest, CopyBool) {
47 FundamentalValue true_value(true); 89 FundamentalValue true_value(true);
48 FundamentalValue copied_true_value(true_value); 90 FundamentalValue copied_true_value(true_value);
49 EXPECT_EQ(true_value.type(), copied_true_value.type()); 91 EXPECT_EQ(true_value.type(), copied_true_value.type());
50 EXPECT_EQ(true_value.GetBool(), copied_true_value.GetBool()); 92 EXPECT_EQ(true_value.GetBool(), copied_true_value.GetBool());
51 93
52 FundamentalValue false_value(false); 94 FundamentalValue false_value(false);
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
84 EXPECT_EQ(value.type(), copied_value.type()); 126 EXPECT_EQ(value.type(), copied_value.type());
85 EXPECT_EQ(value.GetDouble(), copied_value.GetDouble()); 127 EXPECT_EQ(value.GetDouble(), copied_value.GetDouble());
86 128
87 Value blank; 129 Value blank;
88 130
89 blank = value; 131 blank = value;
90 EXPECT_EQ(value.type(), blank.type()); 132 EXPECT_EQ(value.type(), blank.type());
91 EXPECT_EQ(value.GetDouble(), blank.GetDouble()); 133 EXPECT_EQ(value.GetDouble(), blank.GetDouble());
92 } 134 }
93 135
136 TEST(ValuesTest, CopyString) {
137 StringValue value("foobar");
138 StringValue copied_value(value);
139 EXPECT_EQ(value.type(), copied_value.type());
140 EXPECT_EQ(value.GetString(), copied_value.GetString());
141
142 Value blank;
143
144 blank = value;
145 EXPECT_EQ(value.type(), blank.type());
146 EXPECT_EQ(value.GetString(), blank.GetString());
147 }
148
94 // Group of tests for the move constructors and move-assigmnent. 149 // Group of tests for the move constructors and move-assigmnent.
95 TEST(ValuesTest, MoveBool) { 150 TEST(ValuesTest, MoveBool) {
96 FundamentalValue true_value(true); 151 FundamentalValue true_value(true);
97 FundamentalValue moved_true_value(std::move(true_value)); 152 FundamentalValue moved_true_value(std::move(true_value));
98 EXPECT_EQ(Value::Type::BOOLEAN, moved_true_value.type()); 153 EXPECT_EQ(Value::Type::BOOLEAN, moved_true_value.type());
99 EXPECT_TRUE(moved_true_value.GetBool()); 154 EXPECT_TRUE(moved_true_value.GetBool());
100 155
101 FundamentalValue false_value(false); 156 FundamentalValue false_value(false);
102 FundamentalValue moved_false_value(std::move(false_value)); 157 FundamentalValue moved_false_value(std::move(false_value));
103 EXPECT_EQ(Value::Type::BOOLEAN, moved_false_value.type()); 158 EXPECT_EQ(Value::Type::BOOLEAN, moved_false_value.type());
(...skipping 29 matching lines...) Expand all
133 EXPECT_EQ(Value::Type::DOUBLE, moved_value.type()); 188 EXPECT_EQ(Value::Type::DOUBLE, moved_value.type());
134 EXPECT_EQ(74.896, moved_value.GetDouble()); 189 EXPECT_EQ(74.896, moved_value.GetDouble());
135 190
136 Value blank; 191 Value blank;
137 192
138 blank = FundamentalValue(654.38); 193 blank = FundamentalValue(654.38);
139 EXPECT_EQ(Value::Type::DOUBLE, blank.type()); 194 EXPECT_EQ(Value::Type::DOUBLE, blank.type());
140 EXPECT_EQ(654.38, blank.GetDouble()); 195 EXPECT_EQ(654.38, blank.GetDouble());
141 } 196 }
142 197
198 TEST(ValuesTest, MoveString) {
199 StringValue value("foobar");
200 StringValue moved_value(std::move(value));
201 EXPECT_EQ(Value::Type::STRING, moved_value.type());
202 EXPECT_EQ("foobar", moved_value.GetString());
203
204 Value blank;
205
206 blank = StringValue("foobar");
207 EXPECT_EQ(Value::Type::STRING, blank.type());
208 EXPECT_EQ("foobar", blank.GetString());
209 }
210
143 TEST(ValuesTest, Basic) { 211 TEST(ValuesTest, Basic) {
144 // Test basic dictionary getting/setting 212 // Test basic dictionary getting/setting
145 DictionaryValue settings; 213 DictionaryValue settings;
146 std::string homepage = "http://google.com"; 214 std::string homepage = "http://google.com";
147 ASSERT_FALSE(settings.GetString("global.homepage", &homepage)); 215 ASSERT_FALSE(settings.GetString("global.homepage", &homepage));
148 ASSERT_EQ(std::string("http://google.com"), homepage); 216 ASSERT_EQ(std::string("http://google.com"), homepage);
149 217
150 ASSERT_FALSE(settings.Get("global", NULL)); 218 ASSERT_FALSE(settings.Get("global", NULL));
151 settings.SetBoolean("global", true); 219 settings.SetBoolean("global", true);
152 ASSERT_TRUE(settings.Get("global", NULL)); 220 ASSERT_TRUE(settings.Get("global", NULL));
(...skipping 1117 matching lines...) Expand 10 before | Expand all | Expand 10 after
1270 EXPECT_FALSE(main_list.GetList(1, NULL)); 1338 EXPECT_FALSE(main_list.GetList(1, NULL));
1271 EXPECT_FALSE(main_list.GetList(2, NULL)); 1339 EXPECT_FALSE(main_list.GetList(2, NULL));
1272 EXPECT_FALSE(main_list.GetList(3, NULL)); 1340 EXPECT_FALSE(main_list.GetList(3, NULL));
1273 EXPECT_FALSE(main_list.GetList(4, NULL)); 1341 EXPECT_FALSE(main_list.GetList(4, NULL));
1274 EXPECT_FALSE(main_list.GetList(5, NULL)); 1342 EXPECT_FALSE(main_list.GetList(5, NULL));
1275 EXPECT_TRUE(main_list.GetList(6, NULL)); 1343 EXPECT_TRUE(main_list.GetList(6, NULL));
1276 EXPECT_FALSE(main_list.GetList(7, NULL)); 1344 EXPECT_FALSE(main_list.GetList(7, NULL));
1277 } 1345 }
1278 1346
1279 } // namespace base 1347 } // namespace base
OLDNEW
« no previous file with comments | « base/values.cc ('k') | chrome/browser/chromeos/settings/system_settings_provider.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698