dart2js: run dartfmt on tests

tests/compiler/dart2js/js_spec_optimization_test.dart

Index: tests/compiler/dart2js/js_spec_optimization_test.dart
diff --git a/tests/compiler/dart2js/js_spec_optimization_test.dart b/tests/compiler/dart2js/js_spec_optimization_test.dart
index 3c03c09e299b70f0200740d27350348d77c4ee08..4b22d2a8a2f67fb1a4156f07c33acebd5d93f8a6 100644
--- a/tests/compiler/dart2js/js_spec_optimization_test.dart
+++ b/tests/compiler/dart2js/js_spec_optimization_test.dart
@@ -57,7 +57,6 @@ const String TEST_3 = r"""
   }
 """;
 
-
 main() {
   RegExp directivePattern = new RegExp(
       //      \1                    \2        \3
@@ -87,9 +86,5 @@ main() {
     });
   }
 
-  asyncTest(() => Future.wait([
-    check(TEST_1),
-    check(TEST_2),
-    check(TEST_3),
-  ]));
+  asyncTest(() => Future.wait([check(TEST_1), check(TEST_2), check(TEST_3),]));

[Siggi Cherem (dart-lang), 2016/09/16 20:55:23]

may want to delete trailing ,

[Bob Nystrom, 2016/09/16 21:16:03]

This looks like you aren't running the latest dartfmt. As of 0.2.7, a list
literal will always split if it has a trailing comma.

[Harry Terkelsen, 2016/09/16 21:44:04]

I thought the trailing ',' was supposed to cause dartfmt to prefer doing
one-per-line. Maybe that's just for argument lists?

I'll delete the trailing comma since it's all on one line now

[Harry Terkelsen, 2016/09/16 21:44:04]

Oops, I was running 0.2.4.

[Harry Terkelsen, 2016/09/16 21:50:28]

I added the comma back, and it went one-per-line again

 }