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

Unified Diff: tests/language_strong/if_null_precedence_test.dart

Issue 2765693002: Update all tests (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_strong/if_null_precedence_test.dart
diff --git a/tests/language_strong/if_null_precedence_test.dart b/tests/language_strong/if_null_precedence_test.dart
index fcce3d4f1c5e36b1b9ab85df5a0665c5a0a9df27..11da6163997318ac3d1fc2e844ba3868fd23a01c 100644
--- a/tests/language_strong/if_null_precedence_test.dart
+++ b/tests/language_strong/if_null_precedence_test.dart
@@ -26,39 +26,39 @@ main() {
// "a ?? b ?? c" should be legal, and should evaluate to the first non-null
// value (or null if there are no non-null values).
- Expect.equals(1, 1 ?? 2 ?? 3); /// 01: ok
- Expect.equals(2, null ?? 2 ?? 3); /// 02: ok
- Expect.equals(3, null ?? null ?? 3); /// 03: ok
- Expect.equals(null, null ?? null ?? null); /// 04: ok
+ Expect.equals(1, 1 ?? 2 ?? 3); //# 01: ok
+ Expect.equals(2, null ?? 2 ?? 3); //# 02: ok
+ Expect.equals(3, null ?? null ?? 3); //# 03: ok
+ Expect.equals(null, null ?? null ?? null); //# 04: ok
// "a ?? b ? c : d" should parse as "(a ?? b) ? c : d", therefore provided
// that a is true, b need not be a bool. An incorrect parse of
// "a ?? (b ? c : d)" would require b to be a bool to avoid a static type
// warning.
- Expect.equals(2, true ?? 1 ? 2 : 3); /// 05: ok
+ Expect.equals(2, true ?? 1 ? 2 : 3); //# 05: ok
// "a ?? b || c" should parse as "a ?? (b || c)", therefore it's a static
// type warning if b doesn't have type bool. An incorrect parse of
// "(a ?? b) || c" would allow b to have any type provided that a is bool.
- Expect.equals(false, false ?? 1 || true); /// 06: static type warning
+ Expect.equals(false, false ?? 1 || true); //# 06: static type warning
// "a || b ?? c" should parse as "(a || b) ?? c", therefore it is a static
// type warning if b doesn't have type bool. An incorrect parse of
// "a || (b ?? c)" would allow b to have any type provided that c is bool.
if (checkedMode) {
- Expect.throws(() => false || 1 ?? true, assertionError); /// 07: static type warning
+ Expect.throws(() => false || 1 ?? true, assertionError); //# 07: static type warning
} else {
- Expect.equals(false, false || 1 ?? true); /// 07: continued
+ Expect.equals(false, false || 1 ?? true); //# 07: continued
}
if (checkedMode) {
// An incorrect parse of "a || (b ?? c)" would result in no checked-mode
// error.
- Expect.throws(() => false || null ?? true, assertionError); /// 08: ok
+ Expect.throws(() => false || null ?? true, assertionError); //# 08: ok
} else {
// An incorrect parse of "a || (b ?? c)" would result in c being evaluated.
- int i = 0; /// 08: continued
- Expect.equals(false, false || null ?? i++ == 0); /// 08: continued
- Expect.equals(0, i); /// 08: continued
+ int i = 0; //# 08: continued
+ Expect.equals(false, false || null ?? i++ == 0); //# 08: continued
+ Expect.equals(0, i); //# 08: continued
}
}

Powered by Google App Engine
This is Rietveld 408576698