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

Unified Diff: runtime/vm/object_test.cc

Issue 547703002: Rework how types work in the VM Service. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: gen js Created 6 years, 3 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
« no previous file with comments | « runtime/vm/object.cc ('k') | runtime/vm/service/protocol.md » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/object_test.cc
diff --git a/runtime/vm/object_test.cc b/runtime/vm/object_test.cc
index e9e2dfc2809b319bdb588509fdfe4799d0f70cca..b85cb60e68c2d1b8b998537e35c8960527eebd3a 100644
--- a/runtime/vm/object_test.cc
+++ b/runtime/vm/object_test.cc
@@ -4293,10 +4293,11 @@ TEST_CASE(PrintJSONPrimitives) {
JSONStream js;
Bool::True().PrintJSON(&js, true);
elideSubstring("classes", js.ToCString(), buffer);
- EXPECT_STREQ("{\"type\":\"@Bool\",\"id\":\"objects\\/bool-true\","
- "\"class\":{\"type\":\"@Class\",\"id\":\"\","
- "\"name\":\"bool\"},\"valueAsString\":\"true\"}",
- buffer);
+ EXPECT_STREQ(
+ "{\"type\":\"@bool\","
+ "\"class\":{\"type\":\"@Class\",\"id\":\"\",\"name\":\"bool\"},"
+ "\"id\":\"objects\\/bool-true\",\"valueAsString\":\"true\"}",
+ buffer);
}
// Smi reference
{
@@ -4306,9 +4307,9 @@ TEST_CASE(PrintJSONPrimitives) {
elideSubstring("classes", js.ToCString(), buffer);
elideSubstring("_Smi@", buffer, buffer);
EXPECT_STREQ(
- "{\"type\":\"@Smi\","
+ "{\"type\":\"@int\",\"_vmType\":\"@Smi\","
"\"class\":{\"type\":\"@Class\",\"id\":\"\",\"name\":\"_Smi\","
- "\"vmName\":\"\"},"
+ "\"_vmName\":\"\"},"
"\"id\":\"objects\\/int-7\",\"valueAsString\":\"7\"}",
buffer);
}
@@ -4321,9 +4322,9 @@ TEST_CASE(PrintJSONPrimitives) {
elideSubstring("objects", buffer, buffer);
elideSubstring("_Mint@", buffer, buffer);
EXPECT_STREQ(
- "{\"type\":\"@Mint\","
+ "{\"type\":\"@int\",\"_vmType\":\"@Mint\","
"\"class\":{\"type\":\"@Class\",\"id\":\"\",\"name\":\"_Mint\","
- "\"vmName\":\"\"},"
+ "\"_vmName\":\"\"},"
"\"id\":\"\",\"valueAsString\":\"-9223372036854775808\"}",
buffer);
}
@@ -4338,9 +4339,9 @@ TEST_CASE(PrintJSONPrimitives) {
elideSubstring("objects", buffer, buffer);
elideSubstring("_Bigint@", buffer, buffer);
EXPECT_STREQ(
- "{\"type\":\"@Bigint\","
+ "{\"type\":\"@int\",\"_vmType\":\"@Bigint\","
"\"class\":{\"type\":\"@Class\",\"id\":\"\",\"name\":\"_Bigint\","
- "\"vmName\":\"\"},"
+ "\"_vmName\":\"\"},"
"\"id\":\"\",\"valueAsString\":\"44444444444444444444444444444444\"}",
buffer);
}
@@ -4353,9 +4354,9 @@ TEST_CASE(PrintJSONPrimitives) {
elideSubstring("objects", buffer, buffer);
elideSubstring("_Double@", buffer, buffer);
EXPECT_STREQ(
- "{\"type\":\"@Double\","
+ "{\"type\":\"@double\","
"\"class\":{\"type\":\"@Class\",\"id\":\"\",\"name\":\"_Double\","
- "\"vmName\":\"\"},"
+ "\"_vmName\":\"\"},"
"\"id\":\"\",\"valueAsString\":\"0.1234\"}",
buffer);
}
@@ -4370,7 +4371,7 @@ TEST_CASE(PrintJSONPrimitives) {
EXPECT_STREQ(
"{\"type\":\"@String\","
"\"class\":{\"type\":\"@Class\",\"id\":\"\","
- "\"name\":\"_OneByteString\",\"vmName\":\"\"},"
+ "\"name\":\"_OneByteString\",\"_vmName\":\"\"},"
"\"id\":\"\",\"valueAsString\":\"\\\"dw\\\"\"}",
buffer);
}
@@ -4383,9 +4384,9 @@ TEST_CASE(PrintJSONPrimitives) {
elideSubstring("objects", buffer, buffer);
elideSubstring("_List@", buffer, buffer);
EXPECT_STREQ(
- "{\"type\":\"@Array\","
+ "{\"type\":\"@List\",\"_vmType\":\"@Array\","
"\"class\":{\"type\":\"@Class\",\"id\":\"\",\"name\":\"_List\","
- "\"vmName\":\"\"},"
+ "\"_vmName\":\"\"},"
"\"id\":\"\",\"length\":0}",
buffer);
}
@@ -4399,9 +4400,9 @@ TEST_CASE(PrintJSONPrimitives) {
elideSubstring("objects", buffer, buffer);
elideSubstring("_GrowableList@", buffer, buffer);
EXPECT_STREQ(
- "{\"type\":\"@GrowableObjectArray\","
+ "{\"type\":\"@List\",\"_vmType\":\"@GrowableObjectArray\","
"\"class\":{\"type\":\"@Class\",\"id\":\"\",\"name\":\"_GrowableList\","
- "\"vmName\":\"\"},\"id\":\"\",\"length\":0}",
+ "\"_vmName\":\"\"},\"id\":\"\",\"length\":0}",
buffer);
}
// LinkedHashMap reference
@@ -4413,9 +4414,9 @@ TEST_CASE(PrintJSONPrimitives) {
elideSubstring("objects", buffer, buffer);
elideSubstring("_InternalLinkedHashMap@", buffer, buffer);
EXPECT_STREQ(
- "{\"type\":\"@LinkedHashMap\","
+ "{\"type\":\"@Instance\",\"_vmType\":\"@LinkedHashMap\","
"\"class\":{\"type\":\"@Class\",\"id\":\"\","
- "\"name\":\"_InternalLinkedHashMap\",\"vmName\":\"\"},\"id\":\"\"}",
+ "\"name\":\"_InternalLinkedHashMap\",\"_vmName\":\"\"},\"id\":\"\"}",
buffer);
}
// UserTag reference
@@ -4427,9 +4428,9 @@ TEST_CASE(PrintJSONPrimitives) {
elideSubstring("objects", buffer, buffer);
elideSubstring("_UserTag@", buffer, buffer);
EXPECT_STREQ(
- "{\"type\":\"@UserTag\","
+ "{\"type\":\"@Instance\",\"_vmType\":\"@UserTag\","
"\"class\":{\"type\":\"@Class\",\"id\":\"\",\"name\":\"_UserTag\","
- "\"vmName\":\"\"},"
+ "\"_vmName\":\"\"},"
"\"id\":\"\"}",
buffer);
}
@@ -4445,7 +4446,7 @@ TEST_CASE(PrintJSONPrimitives) {
EXPECT_STREQ(
"{\"type\":\"@Type\","
"\"class\":{\"type\":\"@Class\",\"id\":\"\",\"name\":\"_Type\","
- "\"vmName\":\"\"},\"id\":\"\","
+ "\"_vmName\":\"\"},\"id\":\"\","
"\"typeClass\":{\"type\":\"@Class\",\"id\":\"\",\"name\":\"bool\"},"
"\"name\":\"bool\"}",
buffer);
@@ -4455,7 +4456,7 @@ TEST_CASE(PrintJSONPrimitives) {
JSONStream js;
Object::null_object().PrintJSON(&js, true);
EXPECT_STREQ(
- "{\"type\":\"@Null\",\"id\":\"objects\\/null\","
+ "{\"type\":\"@null\",\"id\":\"objects\\/null\","
"\"valueAsString\":\"null\"}",
js.ToCString());
}
@@ -4486,7 +4487,7 @@ TEST_CASE(PrintJSONPrimitives) {
tok.PrintJSON(&js, true);
elideSubstring("objects", js.ToCString(), buffer);
EXPECT_STREQ(
- "{\"type\":\"@LiteralToken\",\"id\":\"\"}",
+ "{\"type\":\"@Object\",\"_vmType\":\"@LiteralToken\",\"id\":\"\"}",
buffer);
}
}
« no previous file with comments | « runtime/vm/object.cc ('k') | runtime/vm/service/protocol.md » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698