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

Unified Diff: test/cctest/test-parsing.cc

Issue 1817093002: Add parsing for object types (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@types-1810943002-tuple
Patch Set: Minor changes, add tests and bug fix Created 4 years, 8 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 | « src/parsing/preparser.h ('k') | test/mjsunit/harmony/typesystem/object-types.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/cctest/test-parsing.cc
diff --git a/test/cctest/test-parsing.cc b/test/cctest/test-parsing.cc
index 37c60d0d61eb40bd82d0dcda790d21e109502a3e..a40057fac25c38eb01d749d859186d11e025461d 100644
--- a/test/cctest/test-parsing.cc
+++ b/test/cctest/test-parsing.cc
@@ -7549,3 +7549,103 @@ TEST(TypedModeTupleTypes) {
RunParserSyncTest(typed_context_data, error_data, kError, NULL, 0,
always_flags, arraysize(always_flags));
}
+
+TEST(TypedModeObjectTypes) {
+ const char* untyped_context_data[][2] = {{"", ""}, {NULL, NULL}};
+ const char* typed_context_data[][2] = {{"'use types'; ", ""}, {NULL, NULL}};
+
+ const char* correct_data[] = {
+ "var x: {}",
+ "var x: {a}",
+ "var x: {a, b}",
+ "var x: {a, b?, c}",
+ "var x: {a: number}",
+ "var x: {a: number, b: string}",
+ "var x: {a?: number; b: {c: any}}",
+ "var x: {f()}",
+ "var x: {f() : number}",
+ "var x: {f(a, b)}",
+ "var x: {f(a: number, b: string)}",
+ "var x: {f(a: number, b: string) : boolean}",
+ "var x: {f?(a: number, b: string) : boolean}",
+ "var x: {f<A>(a: A, b: string) : boolean}",
+ "var x: {f<A, B>(a: A, b: B) : boolean}",
+ "var x: {f<A extends number>(a: A, b: string) : boolean}",
+ "var x: {f<A extends number, B extends string>(a: A, b: B) : boolean}",
+ "var x: {f?<A>(a: A, b: string) : boolean}",
+ "var x: {f?<A, B>(a: A, b: B) : boolean}",
+ "var x: {f?<A extends number>(a: A, b: string) : boolean}",
+ "var x: {f?<A extends number, B extends string>(a: A, b: B) : boolean}",
+ "var x: {()}",
+ "var x: {() : number}",
+ "var x: {(a, b)}",
+ "var x: {(a: number, b: string)}",
+ "var x: {(a: number, b: string) : boolean}",
+ "var x: {<A>(a: A, b: string) : boolean}",
+ "var x: {<A, B>(a: A, b: B) : boolean}",
+ "var x: {<A extends number>(a: A, b: string) : boolean}",
+ "var x: {<A extends number, B extends string>(a: A, b: B) : boolean}",
+ "var x: {new (a, b)}",
+ "var x: {new (a: number, b: string)}",
+ "var x: {new (a: number, b: string) : boolean}",
+ "var x: {new <A>(a: A, b: string) : boolean}",
+ "var x: {new <A, B>(a: A, b: B) : boolean}",
+ "var x: {new <A extends number>(a: A, b: string) : boolean}",
+ "var x: {new <A extends number, B extends string>(a: A, b: B) : boolean}",
+ NULL
+ };
+
+ const char* error_data[] = {
+ "var z: {a: ()}", // Not valid in general.
+ "var z: {a: []}", // Valid as a binding pattern, not as a type.
+ "var z: {f<>()}", // Invalid type parameters
+ "var z: {f<A>}",
+ "var z: {f<A>: A}",
+ NULL
+ };
+
+ static const ParserFlag always_flags[] = {kAllowTypes};
+ RunParserSyncTest(untyped_context_data, correct_data, kError, NULL, 0,
+ always_flags, arraysize(always_flags));
+ RunParserSyncTest(typed_context_data, correct_data, kSuccess, NULL, 0,
+ always_flags, arraysize(always_flags));
+ RunParserSyncTest(untyped_context_data, error_data, kError, NULL, 0,
+ always_flags, arraysize(always_flags));
+ RunParserSyncTest(typed_context_data, error_data, kError, NULL, 0,
+ always_flags, arraysize(always_flags));
+}
+
+TEST(TypedModeBindingPatterns) {
+ const char* untyped_context_data[][2] = {{"", ""}, {NULL, NULL}};
+ const char* typed_context_data[][2] = {{"'use types'; ", ""}, {NULL, NULL}};
+
+ const char* correct_data[] = {
+ "var f: (x: number, cmd?: 'two', ...rest) => string",
+ "var f: ([]: number[]) => boolean",
+ "var f: ([x, y]: number[]) => boolean",
+ "var f: ([x, ...rest]: number[]) => boolean",
+ "var f: ([,x, y]: number[]) => boolean",
+ "var f: ([x, y,]: number[]) => boolean",
+ "var f: ([x, y,,]: number[]) => boolean",
+ "var f: ([x, [a, b, c], z]: [number, any[], string]) => boolean",
+ "var f: ({x: a, y: b}: {x: number, y: string}) => boolean",
+ "var f: ({x: a, y: [b, c, ...rest]}: {x: number, y: string[]}) => boolean",
+ "var f: ({x: {a, b}, y: c}: {x: {a, b}, y: string}) => boolean",
+ NULL
+ };
+
+ const char* error_data[] = {
+ "var f: ({x: {a; b}, y: c}: {x: {a; b}, y: string}) => boolean",
+ NULL
+ };
+
+ static const ParserFlag always_flags[] = {kAllowTypes};
+ RunParserSyncTest(untyped_context_data, correct_data, kError, NULL, 0,
+ always_flags, arraysize(always_flags));
+ RunParserSyncTest(typed_context_data, correct_data, kSuccess, NULL, 0,
+ always_flags, arraysize(always_flags));
+ RunParserSyncTest(untyped_context_data, error_data, kError, NULL, 0,
+ always_flags, arraysize(always_flags));
+ RunParserSyncTest(typed_context_data, error_data, kError, NULL, 0,
+ always_flags, arraysize(always_flags));
+}
« no previous file with comments | « src/parsing/preparser.h ('k') | test/mjsunit/harmony/typesystem/object-types.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698