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

Unified Diff: tests/lib/mirrors/symbol_validation_test.dart

Issue 2770063002: Revert "Format all 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
« no previous file with comments | « tests/lib/mirrors/static_metatarget_test.dart ('k') | tests/lib/mirrors/syntax_error_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/lib/mirrors/symbol_validation_test.dart
diff --git a/tests/lib/mirrors/symbol_validation_test.dart b/tests/lib/mirrors/symbol_validation_test.dart
index f36cb8f0ff7699a4f10410b5ba7c2715867faf7c..1eef33984009d99d6bbbd306e220a06d68a9f5b1 100644
--- a/tests/lib/mirrors/symbol_validation_test.dart
+++ b/tests/lib/mirrors/symbol_validation_test.dart
@@ -8,79 +8,48 @@ import 'dart:mirrors';
import 'package:expect/expect.dart';
validSymbol(String string) {
- Expect.equals(string, MirrorSystem.getName(new Symbol(string)),
- 'Valid symbol "$string" should be invertable');
- Expect.equals(string, MirrorSystem.getName(MirrorSystem.getSymbol(string)),
- 'Valid symbol "$string" should be invertable');
+ Expect.equals(string,
+ MirrorSystem.getName(new Symbol(string)),
+ 'Valid symbol "$string" should be invertable');
+ Expect.equals(string,
+ MirrorSystem.getName(MirrorSystem.getSymbol(string)),
+ 'Valid symbol "$string" should be invertable');
}
invalidSymbol(String string) {
- Expect.throws(() => new Symbol(string), (e) => e is ArgumentError,
- 'Invalid symbol "$string" should be rejected');
- Expect.throws(() => MirrorSystem.getSymbol(string), (e) => e is ArgumentError,
- 'Invalid symbol "$string" should be rejected');
+ Expect.throws(() => new Symbol(string),
+ (e) => e is ArgumentError,
+ 'Invalid symbol "$string" should be rejected');
+ Expect.throws(() => MirrorSystem.getSymbol(string),
+ (e) => e is ArgumentError,
+ 'Invalid symbol "$string" should be rejected');
}
validPrivateSymbol(String string) {
ClosureMirror closure = reflect(main);
LibraryMirror library = closure.function.owner;
- Expect.equals(
- string,
- MirrorSystem.getName(MirrorSystem.getSymbol(string, library)),
- 'Valid private symbol "$string" should be invertable');
+ Expect.equals(string,
+ MirrorSystem.getName(MirrorSystem.getSymbol(string, library)),
+ 'Valid private symbol "$string" should be invertable');
}
main() {
// Operators that can be declared as class member operators.
// These are all valid as symbols.
var operators = [
- '%',
- '&',
- '*',
- '+',
- '-',
- '/',
- '<',
- '<<',
- '<=',
- '==',
- '>',
- '>=',
- '>>',
- '[]',
- '[]=',
- '^',
- 'unary-',
- '|',
- '~',
- '~/'
+ '%', '&', '*', '+', '-', '/', '<', '<<', '<=', '==', '>',
+ '>=', '>>', '[]', '[]=', '^', 'unary-', '|', '~', '~/'
];
operators.expand((op) => [op, "x.$op"]).forEach(validSymbol);
- operators
- .expand((op) => [".$op", "$op.x", "x$op", "_x.$op"])
- .forEach(invalidSymbol);
- operators
- .expand((op) => operators.contains("$op=") ? [] : ["x.$op=", "$op="])
- .forEach(invalidSymbol);
+ operators.expand((op) => [".$op", "$op.x", "x$op", "_x.$op"])
+ .forEach(invalidSymbol);
+ operators.expand((op) => operators.contains("$op=") ? [] : ["x.$op=", "$op="])
+ .forEach(invalidSymbol);
var simpleSymbols = [
- 'foo',
- 'bar_',
- 'baz.quz',
- 'fisk1',
- 'hest2fisk',
- 'a.b.c.d.e',
- r'$',
- r'foo$',
- r'bar$bar',
- r'$.$',
- r'x6$_',
- r'$6_',
- r'x.$$6_',
- 'x_',
- 'x_.x_',
- 'unary',
- 'x.unary'
+ 'foo', 'bar_', 'baz.quz', 'fisk1', 'hest2fisk', 'a.b.c.d.e',
+ r'$', r'foo$', r'bar$bar', r'$.$', r'x6$_', r'$6_', r'x.$$6_',
+ 'x_', 'x_.x_', 'unary', 'x.unary'
];
simpleSymbols.expand((s) => [s, "s="]).forEach(validSymbol);
@@ -132,12 +101,10 @@ main() {
"while",
"with"
];
- reservedWords
- .expand((w) => [w, "$w=", "x.$w", "$w.x", "x.$w.x"])
- .forEach(invalidSymbol);
- reservedWords
- .expand((w) => ["${w}_", "${w}\$", "${w}q"])
- .forEach(validSymbol);
+ reservedWords.expand((w) => [w, "$w=", "x.$w" , "$w.x", "x.$w.x"])
+ .forEach(invalidSymbol);
+ reservedWords.expand((w) => ["${w}_", "${w}\$", "${w}q"])
+ .forEach(validSymbol);
// Built-in identifiers are valid identifiers that are restricted from being
// used in some cases, but they are all valid symbols.
@@ -158,11 +125,13 @@ main() {
"static",
"typedef"
];
- builtInIdentifiers
- .expand((w) => [w, "$w=", "x.$w", "$w.x", "x.$w.x", "$w=", "x.$w="])
- .forEach(validSymbol);
+ builtInIdentifiers.expand((w) => [w, "$w=", "x.$w" , "$w.x", "x.$w.x",
+ "$w=", "x.$w="])
+ .forEach(validSymbol);
- var privateSymbols = ['_', '_x', 'x._y', 'x._', 'x.y._', 'x._.y', '_true'];
+ var privateSymbols = [
+ '_', '_x', 'x._y', 'x._', 'x.y._', 'x._.y', '_true'
+ ];
privateSymbols.forEach(invalidSymbol);
privateSymbols.forEach(validPrivateSymbol); // //# 01: ok
}
« no previous file with comments | « tests/lib/mirrors/static_metatarget_test.dart ('k') | tests/lib/mirrors/syntax_error_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698