Update comments (dart-lang/test_reflective_loader#20)
diff --git a/pkgs/test_reflective_loader/CHANGELOG.md b/pkgs/test_reflective_loader/CHANGELOG.md index 8430219..e9a5ee6 100644 --- a/pkgs/test_reflective_loader/CHANGELOG.md +++ b/pkgs/test_reflective_loader/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
diff --git a/pkgs/test_reflective_loader/lib/test_reflective_loader.dart b/pkgs/test_reflective_loader/lib/test_reflective_loader.dart index efefaa6..5667fe3 100644 --- a/pkgs/test_reflective_loader/lib/test_reflective_loader.dart +++ b/pkgs/test_reflective_loader/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)); } }