dart2js: run dartfmt on tests

tests/compiler/dart2js/js_spec_optimization_test.dart

 // Copyright (c) 2015, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 import 'dart:async';
 import 'package:async_helper/async_helper.dart';
 import 'package:expect/expect.dart';
 import 'compiler_helper.dart';
 
 const String TEST_1 = r"""
@@ skipping 39 matching lines @@
         '#.toLowerCase()', s);
     var s2 = JS('returns:String;depends:none;effects:none;throws:null(1)',
         '#.toUpperCase()', s);
     print(s2);
 
     // absent: 'toLowerCase' - removed since s.toUpperCase() generates the same
     // noSuchMethod.
   }
 """;
 
-
 main() {
   RegExp directivePattern = new RegExp(
       //      \1                    \2        \3
       r'''// *(present|absent): (?:"([^"]*)"|'([^'']*)')''',
       multiLine: true);
 
   Future check(String test) {
     Uri uri = new Uri(scheme: 'dart', path: 'test');
     var compiler = compilerFor(test, uri, expectedErrors: 0);
     return compiler.run(uri).then((_) {
       var element = findElement(compiler, 'main');
       var backend = compiler.backend;
       String generated = backend.getGeneratedCode(element);
 
       for (Match match in directivePattern.allMatches(test)) {
         String directive = match.group(1);
         String pattern = match.groups([2, 3]).where((s) => s != null).single;
         if (directive == 'present') {
           Expect.isTrue(generated.contains(pattern),
               "Cannot find '$pattern' in:\n$generated");
         } else {
           assert(directive == 'absent');
           Expect.isFalse(generated.contains(pattern),
               "Must not find '$pattern' in:\n$generated");
         }
       }
     });
   }
 
-  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

 }