Implement ClosureMirror.

dart/tests/lib/mirrors/mirrors_test.dart

 // Copyright (c) 2011, 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.
 
 // TODO(rmacnak): Move the existing mirror tests here (a place for
 // cross-implementation tests).
 
 library MirrorsTest;
 import "dart:mirrors";
 import "../../../pkg/unittest/lib/unittest.dart";
@@ skipping 86 matching lines @@
   }));
 
   instMirror.setFieldAsync(const Symbol('field'), 44);
   future = instMirror.getFieldAsync(const Symbol('field'));
   future.then(expectAsync1((resultMirror) {
     expect(resultMirror.reflectee, equals(44));
     expect(instance.field, equals(44));
   }));
 }
 
 testClosureMirrors(mirrors) {

[kasperl, 2013/05/24 06:46:01]

Add a TODO here to add tests for optional parameters (positional and named
ones).

[ahe, 2013/05/24 11:57:37]

Done.

   var closure = (x, y, z) { return x + y + z; };
 
   var mirror = reflect(closure);
   expect(mirror is ClosureMirror, equals(true));
 
   var funcMirror = mirror.function;
   expect(funcMirror is MethodMirror, equals(true));
   expect(funcMirror.parameters.length, equals(3));
 
   expect(mirror.apply([7, 8, 9]).reflectee, equals(24));
 
   var future = mirror.applyAsync([2, 4, 8]);

[kasperl, 2013/05/24 06:46:01]

Should this be rewritten (not in this CL) to use apply rather than applyAsync?
Or should we have both?

[ahe, 2013/05/24 06:56:56]

Not sure if you're suggesting a change to dart:mirrors. If we have applyAsync,
this seems like a great place to test it. 

   future.then(expectAsync1((resultMirror) {
     expect(resultMirror.reflectee, equals(14));
   }));
 }
 
 testInvokeConstructor(mirrors) {
   var classMirror = reflectClass(Class);
 
   var instanceMirror = classMirror.newInstance(const Symbol(''),[]);
   expect(instanceMirror.reflectee is Class, equals(true));
@@ skipping 67 matching lines @@
   LibraryMirror valueLibrary = valueClass.owner;
   expect(check(valueLibrary.uri), isTrue);
 }
 
 mainWithArgument({bool isDart2js}) {
   var mirrors = currentMirrorSystem();
   test("Test reflective method invocation", () { testInvoke(mirrors); });
   test("Test instance field access", () { testInstanceFieldAccess(mirrors); });
   test('Test intercepted objects', () { testIntercepted(mirrors); });
   test("Test field access", () { testFieldAccess(mirrors); });
+  test("Test closure mirrors", () { testClosureMirrors(mirrors); });
   if (isDart2js) return;
-  test("Test closure mirrors", () { testClosureMirrors(mirrors); });
   test("Test invoke constructor", () { testInvokeConstructor(mirrors); });
   test("Test reflect type", () { testReflectClass(mirrors); });
   test("Test simple and qualifiedName", () { testNames(mirrors); });
   test("Test current library uri", () {
     testLibraryUri(new Class(),
       (Uri uri) => uri.path.endsWith('/mirrors_test.dart'));
   });
   test("Test dart library uri", () {
     testLibraryUri("test", (Uri uri) => uri == Uri.parse('dart:core'));
   });
 }
 
 main() {
   mainWithArgument(isDart2js: false);
 }