Change package scheduled_test to throw TestFailures rather than strings.

pkg/scheduled_test/test/descriptor/async_test.dart

Index: pkg/scheduled_test/test/descriptor/async_test.dart
diff --git a/pkg/scheduled_test/test/descriptor/async_test.dart b/pkg/scheduled_test/test/descriptor/async_test.dart
index b4fde985b93c34be5d57d71f44aad2f13700a903..3c6b4ad2b1a6722be1cd2b27e5094164b49d8dbd 100644
--- a/pkg/scheduled_test/test/descriptor/async_test.dart
+++ b/pkg/scheduled_test/test/descriptor/async_test.dart
@@ -72,8 +72,9 @@ void main() {
     test('test 2', () {
       expect(errors, everyElement(new isInstanceOf<ScheduleError>()));
       expect(errors.length, equals(1));
-      expect(errors.first.error,
-          matches(r"^File not found: '[^']+[\\/]name\.txt'\.$"));
+      expect(errors.first.error, new isInstanceOf<TestFailure>());

[nweiz, 2013/09/20 21:45:41]

The type of this exception isn't an important part of the interface. We don't
expect users of the library to catch it, and it may change in the future. As
such, we shouldn't test for it (here or elsewhere).

[Bill Hesse, 2013/09/23 17:15:25]

Done.

+      expect(errors.first.error.message,

[nweiz, 2013/09/20 21:45:41]

In keeping with the above, we should test [error.toString()] rather than
[error.message].

[Bill Hesse, 2013/09/23 17:15:25]

Done.

+             matches(r"^File not found: '[^']+[\\/]name\.txt'\.$"));
     });
   }, passing: ['test 2']);
 
@@ -115,7 +116,8 @@ void main() {
     test('test 2', () {
       expect(errors, everyElement(new isInstanceOf<ScheduleError>()));
       expect(errors.length, equals(1));
-      expect(errors.first.error,
+      expect(errors.first.error, new isInstanceOf<TestFailure>());
+      expect(errors.first.error.message,
           matches(r"^Directory not found: '[^']+[\\/]dir'\.$"));
     });
   }, passing: ['test 2']);