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

Unified Diff: tests/language/named_parameters_aggregated_test.dart

Issue 2901853002: Improve various old tests. (Closed)
Patch Set: Created 3 years, 7 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: tests/language/named_parameters_aggregated_test.dart
diff --git a/tests/language/named_parameters_aggregated_test.dart b/tests/language/named_parameters_aggregated_test.dart
index 149f2eb7efa5f37aa41b8b24a03db40073bb211d..52e2189d8fc0bd4fe91607fbcfa8ec7b7178b186 100644
--- a/tests/language/named_parameters_aggregated_test.dart
+++ b/tests/language/named_parameters_aggregated_test.dart
@@ -10,7 +10,7 @@ class TypeTester<T> {}
// Expect compile-time error as no default values are allowed
// in closure type definitions.
typedef void Callback([String msg
- = "" // //# 01: compile-time error
+ = "" //# 01: compile-time error
]);
class NamedParametersAggregatedTests {
@@ -38,16 +38,17 @@ class NamedParametersAggregatedTests {
main() {
// Expect compile-time error due to missing comma in function definition.
NamedParametersAggregatedTests.f_missing_comma(10
- , 25 // //# 02: continued
+ , 25 //# 02: continued
);
// Expect compile-time erorr due to duplicate named argument.
NamedParametersAggregatedTests.F31(10, b: 25
- , b:35 // //# 04: compile-time error
+ , b:35 //# 04: compile-time error
+ , b:35, b:45 //# 06: compile-time error
);
// Expect compile-time error due to missing positional argument.
- Expect.throws(() => NamedParametersAggregatedTests.F31(b:25, c:35), (e) => e is NoSuchMethodError); // //# 05: static type warning
+ Expect.throws(() => NamedParametersAggregatedTests.F31(b:25, c:35), (e) => e is NoSuchMethodError); //# 05: static type warning
new TypeTester<Callback>();
« no previous file with comments | « tests/language/const_error_multiply_initialized_test.dart ('k') | tests/language/ref_before_declaration_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698