Create and test source mapping for invocations.

pkg/js_ast/test/printer_callback_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.
 
 // Note: This test relies on LF line endings in the source file.
 
 // Test that JS printer callbacks occur when expected.
 
 library js_ast.printer.callback_test;
 
 import 'package:js_ast/js_ast.dart';
 import 'package:unittest/unittest.dart';
 
 enum TestMode {
+  INPUT,
   NONE,
   ENTER,
   DELIMITER,
   EXIT,
 }
 
-const DATA = const [
-  const {
+class TestCase {
+  final Map<TestMode, String> data;
+
+  /// The used template names.
+  final List<String> environment;
+
+  const TestCase(
+      this.data,
+      [this.environment = const []]);
+}
+
+const List<TestCase> DATA = const <TestCase>[
+  const TestCase(const {
     TestMode.NONE: """
 function(a, b) {
   return null;
 }""",
     TestMode.ENTER: """
 @0function(@1a, @2b) @3{
   @4return @5null;
 }""",
     TestMode.DELIMITER: """
 function(a, b) {
   return null;
 @0}""",
    TestMode.EXIT: """
 function(a@1, b@2) {
   return null@5;
 @4}@3@0"""
-  },
+  }),
 
-  const {
+  const TestCase(const {
     TestMode.NONE: """
 function() {
   if (true) {
     foo1();
     foo2();
-  } else {
+  } else if (false) {
     bar1();
     bar2();
   }
   while (false) {
     baz3();
     baz4();
   }
 }""",
     TestMode.ENTER: """
 @0function() @1{
   @2if (@3true) @4{
     @5@6@7foo1();
     @8@9@10foo2();
-  } else @11{
-    @12@13@14bar1();
-    @15@16@17bar2();
+  } else @11if (@12false) @13{
+    @14@15@16bar1();
+    @17@18@19bar2();
   }
-  @18while (@19false) @20{
-    @21@22@23baz3();
-    @24@25@26baz4();
+  @20while (@21false) @22{
+    @23@24@25baz3();
+    @26@27@28baz4();
   }
 }""",
     TestMode.DELIMITER: """
 function() {
   if (true) {
     foo1();
     foo2();
-  } else {
+  } else if (false) {
     bar1();
     bar2();
   }
   while (false) {
     baz3();
     baz4();
   }
 @0}""",
     TestMode.EXIT: """
 function() {
   if (true@3) {
     foo1@7()@6;
 @5    foo2@10()@9;
-@8  }@4 else {
-    bar1@14()@13;
-@12    bar2@17()@16;
-@15  }@11
-@2  while (false@19) {
-    baz3@23()@22;
-@21    baz4@26()@25;
-@24  }@20
-@18}@1@0""",
-  },
+@8  }@4 else if (false@12) {
+    bar1@16()@15;
+@14    bar2@19()@18;
+@17  }@13
+@11@2  while (false@21) {
+    baz3@25()@24;
+@23    baz4@28()@27;
+@26  }@22
+@20}@1@0""",
+  }),
+
+  const TestCase(const {
+    TestMode.NONE: """
+function() {
+  function foo() {
+  }
+}""",
+    TestMode.ENTER: """
+@0function() @1{
+  @2@3function @4foo() @5{
+  }
+}""",
+    TestMode.DELIMITER: """
+function() {
+  function foo() {
+  @3}
+@0}""",
+   TestMode.EXIT: """
+function() {
+  function foo@4() {
+  }@5@3
+@2}@1@0"""
+  }),
+
+  const TestCase(const {
+    TestMode.INPUT: """
+function() {
+  a['b'];
+  [1,, 2];
+}""",
+    TestMode.NONE: """
+function() {
+  a.b;
+  [1,, 2];
+}""",
+    TestMode.ENTER: """
+@0function() @1{
+  @2@3@4a.@5b;
+  @6@7[@81,@9, @102];
+}""",
+    TestMode.DELIMITER: """
+function() {
+  a.b;
+  [1,, 2];
+@0}""",
+   TestMode.EXIT: """
+function() {
+  a@4.b@5@3;
+@2  [1@8,,@9 2@10]@7;
+@6}@1@0""",
+  }),
+
+  const TestCase(const {
+    TestMode.INPUT: "a.#foo",
+    TestMode.NONE: "a.foo",
+    TestMode.ENTER: "@0@1a.@2foo",
+    TestMode.DELIMITER: "a.foo",
+    TestMode.EXIT: "a@1.foo@2@0",
+  }, const ['foo']),

[floitsch, 2015/06/29 16:30:44]

maybe give it different names, to make it clearer, that there is a substitution?

[Johnni Winther, 2015/07/02 12:43:05]

Done.

 ];
 
-void check(Map<TestMode, String> map) {
-  String code = map[TestMode.NONE];
+class FixedName extends Name {
+  final String name;
+
+  FixedName(this.name);
+
+  @override
+  int compareTo(other) => 0;
+}
+
+void check(TestCase testCase) {
+  Map<TestMode, String> map = testCase.data;
+  String code = map[TestMode.INPUT];
+  if (code == null) {
+    // Input is the same as output.
+    code = map[TestMode.NONE];
+  }
   JavaScriptPrintingOptions options = new JavaScriptPrintingOptions();
-  Node node = js.parseForeignJS(code).instantiate({});
+  Map arguments = {};
+  testCase.environment.forEach((String name) {
+    arguments[name] = new FixedName(name);
+  });
+  Node node = js.parseForeignJS(code).instantiate(arguments);
+  print('$code:${node.runtimeType}');

[floitsch, 2015/06/29 16:30:44]

debug print.

[Johnni Winther, 2015/07/02 08:38:09]

Removed.

   map.forEach((TestMode mode, String expectedOutput) {
+    if (mode == TestMode.INPUT) return;
     Context context = new Context(mode);
     new Printer(options, context).visit(node);
     expect(context.getText(), equals(expectedOutput),
         reason: "Unexpected output for $code in $mode");
   });
 }
 
 class Context extends SimpleJavaScriptPrintingContext {
   final TestMode mode;
   final Map<Node, int> idMap = {};
@@ skipping 38 matching lines @@
     if (offset < text.length) {
       sb.write(text.substring(offset));
     }
     return sb.toString();
   }
 }
 
 void main() {
   DATA.forEach(check);
 }