commit b3cb5332dafd820551e81e0d3ee9226918b1b611
author Konstantin Scheglov <scheglov@google.com> Fri Jul 20 18:27:45 2018 -0700
committer GitHub <noreply@github.com> Fri Jul 20 18:27:45 2018 -0700
tree 9fa706e6678c6323dd3e0b2aaa66189bb24a6ddb
parent 0a760d0bc1811bdcd39ebc56387f7004f212d98e

Update comments (#20)

CHANGELOG.md

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 8430219..e9a5ee6 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,3 +1,8 @@
+## 0.1.7
+
+- Update documentation comments.
+- Remove `@MirrorsUsed` annotation on `dart:mirrors`.
+
 ## 0.1.6
 
 - Make `FailingTest` public, with the URI of the issue that causes

lib/test_reflective_loader.dart

diff --git a/lib/test_reflective_loader.dart b/lib/test_reflective_loader.dart
index efefaa6..5667fe3 100644
--- a/lib/test_reflective_loader.dart
+++ b/lib/test_reflective_loader.dart
@@ -5,21 +5,18 @@
 library test_reflective_loader;
 
 import 'dart:async';
-@MirrorsUsed(metaTargets: 'ReflectiveTest')
 import 'dart:mirrors';
 
 import 'package:test/test.dart' as test_package;
 
 /**
- * A marker annotation used to annotate overridden test methods (so we cannot
- * rename them to `fail_`) which are expected to fail at `assert` in the
- * checked mode.
+ * A marker annotation used to annotate test methods which are expected to fail
+ * when asserts are enabled.
  */
 const _AssertFailingTest assertFailingTest = const _AssertFailingTest();
 
 /**
- * A marker annotation used to annotate overridden test methods (so we cannot
- * rename them to `fail_`) which are expected to fail.
+ * A marker annotation used to annotate test methods which are expected to fail.
  */
 const FailingTest failingTest = const FailingTest(null);
 
@@ -267,10 +264,12 @@
 typedef dynamic _TestFunction();
 
 /**
- * A marker annotation used to annotate overridden test methods (so we cannot
- * rename them to `fail_`) which are expected to fail.
+ * A marker annotation used to annotate test methods which are expected to fail.
  */
 class FailingTest {
+  /**
+   * Initialize this annotation with the given issue URI.
+   */
   const FailingTest(String issueUri);
 }
 
@@ -279,15 +278,17 @@
  * information.
  */
 class TestTimeout {
-  final test_package.Timeout timeout;
+  final test_package.Timeout _timeout;
 
-  const TestTimeout(this.timeout);
+  /**
+   * Initialize this annotation with the given timeout.
+   */
+  const TestTimeout(test_package.Timeout timeout) : _timeout = timeout;
 }
 
 /**
- * A marker annotation used to annotate overridden test methods (so we cannot
- * rename them to `fail_`) which are expected to fail at `assert` in the
- * checked mode.
+ * A marker annotation used to annotate test methods which are expected to fail
+ * when asserts are enabled.
  */
 class _AssertFailingTest {
   const _AssertFailingTest();
@@ -309,7 +310,7 @@
       _TestFunction function) {
     String fullName = _combineNames(this.name, name);
     TestTimeout timeout = _getAnnotationInstance(memberMirror, TestTimeout);
-    tests.add(new _Test(isSolo, fullName, function, timeout?.timeout));
+    tests.add(new _Test(isSolo, fullName, function, timeout?._timeout));
   }
 }