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..b93721de0ab049b7d6d96929db6d77e25840a942 100644
--- a/pkg/scheduled_test/test/descriptor/async_test.dart
+++ b/pkg/scheduled_test/test/descriptor/async_test.dart
@@ -72,8 +72,8 @@ 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.toString(),
+             matches(r"^File not found: '[^']+[\\/]name\.txt'\.$"));
     });

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

Should we even just drop the length check, and write

errors.single.error.toString(), matches..

or even

(errors.single as ScheduleError).error.toString(), matches...
?
Maybe the separate checks are clearer.

[nweiz, 2013/09/23 22:14:34]

I like this idea (probably with two checks rather than "as ScheduleError"). You
can make that change or not, as you prefer.

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