Formatter code transform bit-flip support.

pkg/analyzer_experimental/test/services/formatter_test.dart

 // Copyright (c) 2013, 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 'package:unittest/unittest.dart';
 
 import 'package:analyzer_experimental/src/generated/scanner.dart';
 import 'package:analyzer_experimental/src/services/formatter_impl.dart';
 import 'package:analyzer_experimental/src/services/writer.dart';
 
@@ skipping 523 matching lines @@
           'int x;\n'
         );
     });
 
     test('CU - EOF nl', () {
       expectCUFormatsTo(
           'var x = 1;',
           'var x = 1;\n'
       );
     });
-    
+
     test('CU - constructor', () {
       expectCUFormatsTo(
           'class A {\n'
           '  const _a;\n'
           '  A();\n'
           '  int a() => _a;\n'
           '}\n',
           'class A {\n'
           '  const _a;\n'
           '  A();\n'
@@ skipping 61 matching lines @@
           '}\n'
         );
     });
 
     test('CU - parts', () {
       expectCUFormatsTo(
         'part of foo;',
         'part of foo;\n'
       );
     });
-    
+
     test('CU (cons inits)', () {
       expectCUFormatsTo('class X {\n'
           '  var x, y;\n'
           '  X() : x = 1, y = 2;\n'
-          '}\n', 
+          '}\n',
           'class X {\n'
           '  var x, y;\n'
           '  X()\n'
-          '      : x = 1,\n' 
+          '      : x = 1,\n'
           '        y = 2;\n'
           '}\n'
       );
     });
 
     test('stmt', () {
       expectStmtFormatsTo(
          'if (true){\n'
          'if (true){\n'
          'if (true){\n'
@@ skipping 39 matching lines @@
         '}'
       );
     });
 
     test('stmt (generics)', () {
       expectStmtFormatsTo(
         'var numbers = <int>[1, 2, (3 + 4)];',
         'var numbers = <int>[1, 2, (3 + 4)];'
       );
     });
-    
+
     test('stmt (lists)', () {
       expectStmtFormatsTo(
         'var l = [1,2,3,4];',
         'var l = [1, 2, 3, 4];'
       );
       //Dangling ','
       expectStmtFormatsTo(
         'var l = [1,];',
         'var l = [1,];'
       );
     });
 
     test('stmt (maps)', () {
       expectStmtFormatsTo(
         'var map = const {"foo": "bar", "fuz": null};',
         'var map = const {"foo": "bar", "fuz": null};'
       );
-      
+
       //Dangling ','
       expectStmtFormatsTo(
         'var map = {"foo": "bar",};',
         'var map = {"foo": "bar",};'
       );
     });
 
     test('stmt (try/catch)', () {
       expectStmtFormatsTo(
         'try {\n'
@@ skipping 47 matching lines @@
       );
       expectStmtFormatsTo(
         'for (final foo in bar.foos) {\n'
         '  print(foo);\n'
         '}',
         'for (final foo in bar.foos) {\n'
         '  print(foo);\n'
         '}'
       );
     });
-    
+
     test('Statement (if)', () {
-      expectStmtFormatsTo('if (true) print("true!");', 
+      expectStmtFormatsTo('if (true) print("true!");',
                           'if (true) print("true!");');
-      expectStmtFormatsTo('if (true) { print("true!"); }', 
+      expectStmtFormatsTo('if (true) { print("true!"); }',
                           'if (true) {\n'
                           '  print("true!");\n'
                           '}');
-      expectStmtFormatsTo('if (true) print("true!"); else print("false!");', 
+      expectStmtFormatsTo('if (true) print("true!"); else print("false!");',
                           'if (true) {\n'
                           '  print("true!");\n'
                           '} else {\n'
                           '  print("false!");\n'
                           '}');
-    }); 
+      expectStmtFormatsTo('if (true) print("true!"); else print("false!");',

[Brian Wilkerson, 2013/10/07 21:37:29]

I expect you'll want to test the "transforms: true" case as well. You might
consider passing in three parameters to the expect method: the source to be
formatted, the result to expect if transforms is false, and an optional third
parameter that is the result to expect if transforms is true.

[pquitslund, 2013/10/07 22:33:44]

The expect above actually does that.  I like your idea of compressing this into
one call though.  I'll look at it.  Thanks!

+                          'if (true) print("true!"); else print("false!");',
+                          transforms: false);
+    });
 
     test('initialIndent', () {
       var formatter = new CodeFormatter(
           new FormatterOptions(initialIndentationLevel: 2));
-      var formattedSource = 
+      var formattedSource =
           formatter.format(CodeKind.STATEMENT, 'var x;').source;
       expect(formattedSource, startsWith('    '));
     });
 
     test('selections', () {
       expectSelectedPostFormat('class X {}', '}');
       expectSelectedPostFormat('class X{}', '{');
       expectSelectedPostFormat('class X{int y;}', ';');
       expectSelectedPostFormat('class X{int y;}', '}');
       expectSelectedPostFormat('class X {}', ' {');
     });
-    
+
   });
 
 
   /// Token streams
   group('token streams', () {
 
     test('string tokens', () {
       expectTokenizedEqual('class A{}', 'class A{ }');
       expectTokenizedEqual('class A{}', 'class A{\n  }\n');
       expectTokenizedEqual('class A {}', 'class A{ }');
@@ skipping 195 matching lines @@
     expect(()=> expectStreamsEqual(tokenize(s1), tokenize(s2)),
     throwsA(new isInstanceOf<FormatterException>()));
 
 expectStreamsEqual(Token t1, Token t2) =>
     new TokenStreamComparator(null, t1, t2).verifyEquals();
 
 expectStreamsNotEqual(Token t1, Token t2) =>
     expect(() => new TokenStreamComparator(null, t1, t2).verifyEquals(),
     throwsA(new isInstanceOf<FormatterException>()));
 
-expectCUFormatsTo(src, expected) => 
+expectCUFormatsTo(src, expected) =>
     expect(formatCU(src).source, equals(expected));
 
-expectStmtFormatsTo(src, expected) => 
-    expect(formatStatement(src), equals(expected));
+expectStmtFormatsTo(src, expected, {transforms: true}) =>
+    expect(formatStatement(src, options:
+      new FormatterOptions(codeTransforms: transforms)), equals(expected));