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

Unified Diff: tools/json_schema_compiler/test/arrays_unittest.cc

Issue 10701012: JSON Schema Compiler: Added event compilation. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Tested GetAllPossibleParameterLists better. Created 8 years, 5 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 side-by-side diff with in-line comments
Download patch
Index: tools/json_schema_compiler/test/arrays_unittest.cc
diff --git a/tools/json_schema_compiler/test/arrays_unittest.cc b/tools/json_schema_compiler/test/arrays_unittest.cc
index 070967779a8a52936ba514220ec6743380a44006..b065e4e5d7d65e5c6b423c5c428a811079fbcf2a 100644
--- a/tools/json_schema_compiler/test/arrays_unittest.cc
+++ b/tools/json_schema_compiler/test/arrays_unittest.cc
@@ -4,40 +4,26 @@
#include "tools/json_schema_compiler/test/arrays.h"
+#include "chrome/common/extensions/value_builder.h"
#include "testing/gtest/include/gtest/gtest.h"
using namespace test::api::arrays;
+using namespace extensions;
namespace {
-// TODO(calamity): Change to AppendString etc once kalman's patch goes through
static scoped_ptr<DictionaryValue> CreateBasicArrayTypeDictionary() {
- DictionaryValue* value = new DictionaryValue();
- ListValue* strings_value = new ListValue();
- strings_value->Append(Value::CreateStringValue("a"));
- strings_value->Append(Value::CreateStringValue("b"));
- strings_value->Append(Value::CreateStringValue("c"));
- strings_value->Append(Value::CreateStringValue("it's easy as"));
- ListValue* integers_value = new ListValue();
- integers_value->Append(Value::CreateIntegerValue(1));
- integers_value->Append(Value::CreateIntegerValue(2));
- integers_value->Append(Value::CreateIntegerValue(3));
- ListValue* booleans_value = new ListValue();
- booleans_value->Append(Value::CreateBooleanValue(false));
- booleans_value->Append(Value::CreateBooleanValue(true));
- ListValue* numbers_value = new ListValue();
- numbers_value->Append(Value::CreateDoubleValue(6.1));
- value->Set("numbers", numbers_value);
- value->Set("booleans", booleans_value);
- value->Set("strings", strings_value);
- value->Set("integers", integers_value);
- return scoped_ptr<DictionaryValue>(value);
+ return DictionaryBuilder()
+ .Set("numbers", ListBuilder().Append(6.1))
+ .Set("booleans", ListBuilder().AppendBoolean(false).AppendBoolean(true))
+ .Set("strings", ListBuilder().Append("a").Append("b").Append("c")
+ .Append("it's easy as"))
+ .Set("integers", ListBuilder().Append(1).Append(2).Append(3))
+ .Build().Pass();
}
static Value* CreateItemValue(int val) {
- DictionaryValue* value(new DictionaryValue());
- value->Set("val", Value::CreateIntegerValue(val));
- return value;
+ return DictionaryBuilder().Set("val", val).Build().release();
}
} // namespace
@@ -194,15 +180,14 @@ TEST(JsonSchemaCompilerArrayTest, ReturnIntegerArrayResultCreate) {
std::vector<int> integers;
integers.push_back(1);
integers.push_back(2);
- scoped_ptr<Value> result(ReturnIntegerArray::Result::Create(integers));
- ListValue* list = NULL;
- EXPECT_TRUE(result->GetAsList(&list));
- int temp;
- ASSERT_EQ(2u, list->GetSize());
- EXPECT_TRUE(list->GetInteger(0, &temp));
- EXPECT_EQ(1, temp);
- EXPECT_TRUE(list->GetInteger(1, &temp));
- EXPECT_EQ(2, temp);
+ scoped_ptr<Value> results(ReturnIntegerArray::Result::Create(integers));
+
+ ListValue expected;
+ ListValue* expected_argument = new ListValue();
+ expected_argument->Append(Value::CreateIntegerValue(1));
+ expected_argument->Append(Value::CreateIntegerValue(2));
+ expected.Append(expected_argument);
+ EXPECT_TRUE(results->Equals(&expected));
}
TEST(JsonSchemaCompilerArrayTest, ReturnRefArrayResultCreate) {
@@ -211,16 +196,16 @@ TEST(JsonSchemaCompilerArrayTest, ReturnRefArrayResultCreate) {
items.push_back(linked_ptr<Item>(new Item()));
items[0]->val = 1;
items[1]->val = 2;
- scoped_ptr<Value> result(ReturnRefArray::Result::Create(items));
- ListValue* list = NULL;
- EXPECT_TRUE(result->GetAsList(&list));
- ASSERT_EQ(2u, list->GetSize());
- DictionaryValue* item_value = NULL;
- int temp;
- EXPECT_TRUE(list->GetDictionary(0, &item_value));
- EXPECT_TRUE(item_value->GetInteger("val", &temp));
- EXPECT_EQ(1, temp);
- EXPECT_TRUE(list->GetDictionary(1, &item_value));
- EXPECT_TRUE(item_value->GetInteger("val", &temp));
- EXPECT_EQ(2, temp);
+ scoped_ptr<Value> results(ReturnRefArray::Result::Create(items));
+
+ ListValue expected;
+ ListValue* expected_argument = new ListValue();
+ DictionaryValue* first = new DictionaryValue();
+ first->SetInteger("val", 1);
+ expected_argument->Append(first);
+ DictionaryValue* second = new DictionaryValue();
+ second->SetInteger("val", 2);
+ expected_argument->Append(second);
+ expected.Append(expected_argument);
+ EXPECT_TRUE(results->Equals(&expected));
}

Powered by Google App Engine
This is Rietveld 408576698