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

Unified Diff: tests/lib_strong/mirrors/invocation_fuzz_test.dart

Issue 2774783002: Re-land "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
Index: tests/lib_strong/mirrors/invocation_fuzz_test.dart
diff --git a/tests/lib_strong/mirrors/invocation_fuzz_test.dart b/tests/lib_strong/mirrors/invocation_fuzz_test.dart
index d5f9ca96d73d57da2e06b662970ced13119e63d1..32749e7b2ee534b9ed25d5b8d48463718cddd58e 100644
--- a/tests/lib_strong/mirrors/invocation_fuzz_test.dart
+++ b/tests/lib_strong/mirrors/invocation_fuzz_test.dart
@@ -38,8 +38,8 @@ var blacklist = [
// These either cause the VM to segfault or throw uncatchable API errors.
// TODO(15274): Fix them and remove from blacklist.
- 'dart.io.SystemEncoding.decode', // Windows only
- 'dart.io.SystemEncoding.encode', // Windows only
+ 'dart.io.SystemEncoding.decode', // Windows only
+ 'dart.io.SystemEncoding.encode', // Windows only
// These construct an object with an uninitialized native field.
// TODO(23869): We could make this safer, but making the failure non-fatal
@@ -67,6 +67,7 @@ class Task {
var name;
var action;
}
+
var queue = new List();
checkMethod(MethodMirror m, ObjectMirror target, [origin]) {
@@ -76,15 +77,12 @@ checkMethod(MethodMirror m, ObjectMirror target, [origin]) {
task.name = '${MirrorSystem.getName(m.qualifiedName)} from $origin';
if (m.isRegularMethod) {
- task.action =
- () => target.invoke(m.simpleName,
- new List.filled(m.parameters.length, fuzzArgument));
+ task.action = () => target.invoke(
+ m.simpleName, new List.filled(m.parameters.length, fuzzArgument));
} else if (m.isGetter) {
- task.action =
- () => target.getField(m.simpleName);
+ task.action = () => target.getField(m.simpleName);
} else if (m.isSetter) {
- task.action =
- () => target.setField(m.simpleName, null);
+ task.action = () => target.setField(m.simpleName, null);
} else if (m.isConstructor) {
return;
} else {
@@ -117,8 +115,7 @@ checkClass(classMirror) {
task.name = MirrorSystem.getName(m.qualifiedName);
task.action = () {
- var instance = classMirror.newInstance(
- m.constructorName,
+ var instance = classMirror.newInstance(m.constructorName,
new List.filled(m.parameters.length, fuzzArgument));
checkInstance(instance, task.name);
};
@@ -151,7 +148,7 @@ doOneTask() {
print(task.name);
try {
task.action();
- } catch(e) {}
+ } catch (e) {}
// Register the next task in a timer callback so as to yield to async code
// scheduled in the current task. This isn't necessary for the test itself,
@@ -173,15 +170,30 @@ main() {
currentMirrorSystem().libraries.values.forEach(checkLibrary);
- var valueObjects =
- [true, false, null, [], {}, dynamic,
- 0, 0xEFFFFFF, 0xFFFFFFFF, 0xFFFFFFFFFFFFFFFF, 3.14159,
- "foo", 'blåbærgrød', 'Îñţérñåţîöñåļîžåţîờñ', "\u{1D11E}", #symbol];
+ var valueObjects = [
+ true,
+ false,
+ null,
+ [],
+ {},
+ dynamic,
+ 0,
+ 0xEFFFFFF,
+ 0xFFFFFFFF,
+ 0xFFFFFFFFFFFFFFFF,
+ 3.14159,
+ "foo",
+ 'blåbærgrød',
+ 'Îñţérñåţîöñåļîžåţîờñ',
+ "\u{1D11E}",
+ #symbol
+ ];
valueObjects.forEach((v) => checkInstance(reflect(v), 'value object'));
- uncaughtErrorHandler(self, parent, zone, error, stack) {};
+ uncaughtErrorHandler(self, parent, zone, error, stack) {}
+ ;
var zoneSpec =
- new ZoneSpecification(handleUncaughtError: uncaughtErrorHandler);
+ new ZoneSpecification(handleUncaughtError: uncaughtErrorHandler);
testZone = Zone.current.fork(specification: zoneSpec);
testZone.createTimer(Duration.ZERO, doOneTask);
}

Powered by Google App Engine
This is Rietveld 408576698