| Index: tools/json_schema_compiler/test/error_generation_unittest.cc
|
| diff --git a/tools/json_schema_compiler/test/error_generation_unittest.cc b/tools/json_schema_compiler/test/error_generation_unittest.cc
|
| index 9c22682bbf1409b6c9b832c417282602d6ff3055..e033ca45ce08012b9d909da11dc28bb54f3d3ecb 100644
|
| --- a/tools/json_schema_compiler/test/error_generation_unittest.cc
|
| +++ b/tools/json_schema_compiler/test/error_generation_unittest.cc
|
| @@ -36,7 +36,7 @@ testing::AssertionResult EqualsUtf16(const std::string& expected,
|
| TEST(JsonSchemaCompilerErrorTest, RequiredPropertyPopulate) {
|
| {
|
| std::unique_ptr<base::DictionaryValue> value =
|
| - Dictionary("string", new base::Value("bling"));
|
| + Dictionary("string", base::MakeUnique<base::Value>("bling"));
|
| EXPECT_TRUE(EqualsUtf16("", GetPopulateError<TestType>(*value)));
|
| }
|
| {
|
| @@ -64,7 +64,7 @@ TEST(JsonSchemaCompilerErrorTest, UnexpectedTypePopulation) {
|
| TEST(JsonSchemaCompilerErrorTest, TypeIsRequired) {
|
| {
|
| std::unique_ptr<base::DictionaryValue> value =
|
| - Dictionary("integers", new Value(5));
|
| + Dictionary("integers", base::MakeUnique<Value>(5));
|
| EXPECT_TRUE(EqualsUtf16("", GetPopulateError<ChoiceType>(*value)));
|
| }
|
| {
|
| @@ -112,12 +112,12 @@ TEST(JsonSchemaCompilerErrorTest, ParamIsRequired) {
|
| TEST(JsonSchemaCompilerErrorTest, WrongPropertyValueType) {
|
| {
|
| std::unique_ptr<base::DictionaryValue> value =
|
| - Dictionary("string", new base::Value("yes"));
|
| + Dictionary("string", base::MakeUnique<base::Value>("yes"));
|
| EXPECT_TRUE(EqualsUtf16("", GetPopulateError<TestType>(*value)));
|
| }
|
| {
|
| std::unique_ptr<base::DictionaryValue> value =
|
| - Dictionary("string", new Value(1.1));
|
| + Dictionary("string", base::MakeUnique<Value>(1.1));
|
| EXPECT_TRUE(EqualsUtf16("'string': expected string, got double",
|
| GetPopulateError<TestType>(*value)));
|
| }
|
| @@ -146,7 +146,7 @@ TEST(JsonSchemaCompilerErrorTest, WrongTypeValueType) {
|
| }
|
| {
|
| std::unique_ptr<base::DictionaryValue> value =
|
| - Dictionary("otherType", new Value(1.1));
|
| + Dictionary("otherType", base::MakeUnique<Value>(1.1));
|
| ObjectType out;
|
| base::string16 error;
|
| EXPECT_TRUE(ObjectType::Populate(*value, &out, &error));
|
| @@ -173,13 +173,13 @@ TEST(JsonSchemaCompilerErrorTest, UnableToPopulateArray) {
|
|
|
| TEST(JsonSchemaCompilerErrorTest, BinaryTypeExpected) {
|
| {
|
| - std::unique_ptr<base::DictionaryValue> value =
|
| - Dictionary("data", new base::Value(base::Value::Type::BINARY));
|
| + std::unique_ptr<base::DictionaryValue> value = Dictionary(
|
| + "data", base::MakeUnique<base::Value>(base::Value::Type::BINARY));
|
| EXPECT_TRUE(EqualsUtf16("", GetPopulateError<BinaryData>(*value)));
|
| }
|
| {
|
| std::unique_ptr<base::DictionaryValue> value =
|
| - Dictionary("data", new Value(1.1));
|
| + Dictionary("data", base::MakeUnique<Value>(1.1));
|
| EXPECT_TRUE(EqualsUtf16("'data': expected binary, got double",
|
| GetPopulateError<BinaryData>(*value)));
|
| }
|
| @@ -188,12 +188,12 @@ TEST(JsonSchemaCompilerErrorTest, BinaryTypeExpected) {
|
| TEST(JsonSchemaCompilerErrorTest, ListExpected) {
|
| {
|
| std::unique_ptr<base::DictionaryValue> value =
|
| - Dictionary("TheArray", new base::ListValue());
|
| + Dictionary("TheArray", base::MakeUnique<base::ListValue>());
|
| EXPECT_TRUE(EqualsUtf16("", GetPopulateError<ArrayObject>(*value)));
|
| }
|
| {
|
| std::unique_ptr<base::DictionaryValue> value =
|
| - Dictionary("TheArray", new Value(5));
|
| + Dictionary("TheArray", base::MakeUnique<Value>(5));
|
| EXPECT_TRUE(EqualsUtf16("'TheArray': expected list, got integer",
|
| GetPopulateError<ArrayObject>(*value)));
|
| }
|
| @@ -204,12 +204,12 @@ TEST(JsonSchemaCompilerErrorTest, ListExpected) {
|
| TEST(JsonSchemaCompilerErrorTest, BadEnumValue) {
|
| {
|
| std::unique_ptr<base::DictionaryValue> value =
|
| - Dictionary("enumeration", new base::Value("one"));
|
| + Dictionary("enumeration", base::MakeUnique<base::Value>("one"));
|
| EXPECT_TRUE(EqualsUtf16("", GetPopulateError<HasEnumeration>(*value)));
|
| }
|
| {
|
| std::unique_ptr<base::DictionaryValue> value =
|
| - Dictionary("enumeration", new base::Value("bad sauce"));
|
| + Dictionary("enumeration", base::MakeUnique<base::Value>("bad sauce"));
|
| EXPECT_TRUE(EqualsUtf16("'Enumeration': expected \"one\" or \"two\" "
|
| "or \"three\", got \"bad sauce\"",
|
| GetPopulateError<HasEnumeration>(*value)));
|
| @@ -221,12 +221,12 @@ TEST(JsonSchemaCompilerErrorTest, BadEnumValue) {
|
| TEST(JsonSchemaCompilerErrorTest, WarnOnOptionalFailure) {
|
| {
|
| std::unique_ptr<base::DictionaryValue> value =
|
| - Dictionary("string", new base::Value("bling"));
|
| + Dictionary("string", base::MakeUnique<base::Value>("bling"));
|
| EXPECT_TRUE(EqualsUtf16("", GetPopulateError<OptionalTestType>(*value)));
|
| }
|
| {
|
| std::unique_ptr<base::DictionaryValue> value =
|
| - Dictionary("string", new base::Value(1));
|
| + Dictionary("string", base::MakeUnique<base::Value>(1));
|
|
|
| OptionalTestType out;
|
| base::string16 error;
|
| @@ -239,14 +239,14 @@ TEST(JsonSchemaCompilerErrorTest, WarnOnOptionalFailure) {
|
|
|
| TEST(JsonSchemaCompilerErrorTest, OptionalBinaryTypeFailure) {
|
| {
|
| - std::unique_ptr<base::DictionaryValue> value =
|
| - Dictionary("data", new base::Value(base::Value::Type::BINARY));
|
| + std::unique_ptr<base::DictionaryValue> value = Dictionary(
|
| + "data", base::MakeUnique<base::Value>(base::Value::Type::BINARY));
|
| EXPECT_TRUE(EqualsUtf16("", GetPopulateError<OptionalBinaryData>(*value)));
|
| }
|
| {
|
| // There's a bug with silent failures if the key doesn't exist.
|
| std::unique_ptr<base::DictionaryValue> value =
|
| - Dictionary("data", new base::Value(1));
|
| + Dictionary("data", base::MakeUnique<base::Value>(1));
|
|
|
| OptionalBinaryData out;
|
| base::string16 error;
|
| @@ -260,12 +260,12 @@ TEST(JsonSchemaCompilerErrorTest, OptionalBinaryTypeFailure) {
|
| TEST(JsonSchemaCompilerErrorTest, OptionalArrayTypeFailure) {
|
| {
|
| std::unique_ptr<base::DictionaryValue> value =
|
| - Dictionary("TheArray", new base::ListValue());
|
| + Dictionary("TheArray", base::MakeUnique<base::ListValue>());
|
| EXPECT_TRUE(EqualsUtf16("", GetPopulateError<ArrayObject>(*value)));
|
| }
|
| {
|
| std::unique_ptr<base::DictionaryValue> value =
|
| - Dictionary("TheArray", new Value(5));
|
| + Dictionary("TheArray", base::MakeUnique<Value>(5));
|
| ArrayObject out;
|
| base::string16 error;
|
| EXPECT_TRUE(ArrayObject::Populate(*value, &out, &error));
|
| @@ -298,7 +298,7 @@ TEST(JsonSchemaCompilerErrorTest, OptionalUnableToPopulateArray) {
|
| TEST(JsonSchemaCompilerErrorTest, MultiplePopulationErrors) {
|
| {
|
| std::unique_ptr<base::DictionaryValue> value =
|
| - Dictionary("TheArray", new Value(5));
|
| + Dictionary("TheArray", base::MakeUnique<Value>(5));
|
| ArrayObject out;
|
| base::string16 error;
|
| EXPECT_TRUE(ArrayObject::Populate(*value, &out, &error));
|
| @@ -317,13 +317,13 @@ TEST(JsonSchemaCompilerErrorTest, MultiplePopulationErrors) {
|
| TEST(JsonSchemaCompilerErrorTest, TooManyKeys) {
|
| {
|
| std::unique_ptr<base::DictionaryValue> value =
|
| - Dictionary("string", new base::Value("yes"));
|
| + Dictionary("string", base::MakeUnique<base::Value>("yes"));
|
| EXPECT_TRUE(EqualsUtf16("", GetPopulateError<TestType>(*value)));
|
| }
|
| {
|
| std::unique_ptr<base::DictionaryValue> value =
|
| - Dictionary("string", new base::Value("yes"), "ohno",
|
| - new base::Value("many values"));
|
| + Dictionary("string", base::MakeUnique<base::Value>("yes"), "ohno",
|
| + base::MakeUnique<base::Value>("many values"));
|
| EXPECT_TRUE(EqualsUtf16("found unexpected key 'ohno'",
|
| GetPopulateError<TestType>(*value)));
|
| }
|
|
|