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

Unified Diff: tests/language/not_enough_positional_arguments_test.dart

Issue 23460006: Update language status and rewrite a test (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Addressed review comments Created 7 years, 4 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 | « tests/language/language_dart2js.status ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/language/not_enough_positional_arguments_test.dart
diff --git a/tests/language/not_enough_positional_arguments_test.dart b/tests/language/not_enough_positional_arguments_test.dart
index a49f864cfe7d5853dd5e9aea8e0b3689d0ae57d7..87c50afb04dccc37d86c4e6117add2cb91aa7913 100644
--- a/tests/language/not_enough_positional_arguments_test.dart
+++ b/tests/language/not_enough_positional_arguments_test.dart
@@ -5,17 +5,50 @@
foo(a, [b]) {
}
+bar(a, {b}) {
+}
+
class A {
- A(a, [b]);
+ A();
+ A.test(a, [b]);
}
class B {
B()
- : super(b: 1) /// 01: runtime error
+ : super.test(b: 1) /// 01: runtime error
+ ;
+}
+
+class C extends A {
+ C()
+ : super.test(b: 1) /// 02: runtime error
+ ;
+}
+
+class D {
+ D();
+ D.test(a, {b});
+}
+
+class E {
+ E()
+ : super.test(b: 1) /// 04: runtime error
+ ;
+}
kustermann 2013/08/28 08:28:19 Is this "/// 04" (class E) not testing the same th
Søren Gjesse 2013/08/28 08:42:48 True. Removed E.
+
+class F extends D {
+ F()
+ : super.test(b: 1) /// 05: runtime error
;
}
main() {
- new B(); /// 01: continued
- foo(b: 1); /// 02: runtime error
+ new A.test(b: 1); /// 00: runtime error
+ new B();
+ new C();
+ new D.test(b: 1); /// 03: runtime error
+ new E();
+ new F();
+ foo(b: 1); /// 06: runtime error
+ bar(b: 1); /// 07: runtime error
}
« no previous file with comments | « tests/language/language_dart2js.status ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698