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

Unified Diff: tests/language/conditional_method_invocation_test.dart

Issue 2765893003: Fix warnings_checker.dart handling of multitests (Closed)
Patch Set: Created 3 years, 9 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/conditional_method_invocation_test.dart
diff --git a/tests/language/conditional_method_invocation_test.dart b/tests/language/conditional_method_invocation_test.dart
index 0dd219deea717962cecca65d4c22ff91b6505aef..5131d250aaa03b29850ee7f21939cc11144bb8a6 100644
--- a/tests/language/conditional_method_invocation_test.dart
+++ b/tests/language/conditional_method_invocation_test.dart
@@ -30,35 +30,35 @@ main() {
nullC()?.f(null);
// o?.m(...) is equivalent to ((x) => x == null ? null : x.m(...))(o).
- Expect.equals(null, nullC()?.f(bad())); /// 01: ok
- Expect.equals(1, new C()?.f(() => 1)); /// 02: ok
+ Expect.equals(null, nullC()?.f(bad())); //# 01: ok
+ Expect.equals(1, new C()?.f(() => 1)); //# 02: ok
// C?.m(...) is equivalent to C.m(...).
- Expect.equals(1, C?.staticF(() => 1)); /// 14: ok
- Expect.equals(1, h.C?.staticF(() => 1)); /// 15: ok
+ Expect.equals(1, C?.staticF(() => 1)); //# 14: ok
+ Expect.equals(1, h.C?.staticF(() => 1)); //# 15: ok
// The static type of o?.m(...) is the same as the static type of
// o.m(...).
- { int i = nullC()?.g(bad()); Expect.equals(null, i); } /// 03: ok
- { int i = new C()?.g(() => 1); Expect.equals(1, i); } /// 04: ok
- { String s = nullC()?.g(bad()); Expect.equals(null, s); } /// 05: static type warning
- { String s = new C()?.g(() => null); Expect.equals(null, s); } /// 06: static type warning
- { int i = C?.staticG(() => 1); Expect.equals(1, i); } /// 16: ok
- { int i = h.C?.staticG(() => 1); Expect.equals(1, i); } /// 17: ok
- { String s = C?.staticG(() => null); Expect.equals(null, s); } /// 18: static type warning
- { String s = h.C?.staticG(() => null); Expect.equals(null, s); } /// 19: static type warning
+ { int i = nullC()?.g(bad()); Expect.equals(null, i); } //# 03: ok
+ { int i = new C()?.g(() => 1); Expect.equals(1, i); } //# 04: ok
+ { String s = nullC()?.g(bad()); Expect.equals(null, s); } //# 05: static type warning
+ { String s = new C()?.g(() => null); Expect.equals(null, s); } //# 06: static type warning
+ { int i = C?.staticG(() => 1); Expect.equals(1, i); } //# 16: ok
+ { int i = h.C?.staticG(() => 1); Expect.equals(1, i); } //# 17: ok
+ { String s = C?.staticG(() => null); Expect.equals(null, s); } //# 18: static type warning
+ { String s = h.C?.staticG(() => null); Expect.equals(null, s); } //# 19: static type warning
// Let T be the static type of o and let y be a fresh variable of type T.
// Exactly the same static warnings that would be caused by y.m(...) are also
// generated in the case of o?.m(...).
- { B b = new C(); Expect.equals(1, b?.f(() => 1)); } /// 07: static type warning
- { int i = 1; Expect.equals(null, nullC()?.f(i)); } /// 08: static type warning
+ { B b = new C(); Expect.equals(1, b?.f(() => 1)); } //# 07: static type warning
+ { int i = 1; Expect.equals(null, nullC()?.f(i)); } //# 08: static type warning
// '?.' can't be used to access toplevel functions in libraries imported via
// prefix.
- h?.topLevelFunction(); /// 11: compile-time error
+ h?.topLevelFunction(); //# 11: compile-time error
// Nor can it be used to access the toString method on the class Type.
- Expect.throws(() => C?.toString(), noMethod); /// 12: static type warning
- Expect.throws(() => h.C?.toString(), noMethod); /// 13: static type warning
+ Expect.throws(() => C?.toString(), noMethod); //# 12: static type warning
+ Expect.throws(() => h.C?.toString(), noMethod); //# 13: static type warning
}
« no previous file with comments | « tests/language/compile_time_constant_test.dart ('k') | tests/language/conditional_property_access_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698