Change package scheduled_test to throw TestFailures rather than strings.

pkg/scheduled_test/test/descriptor/directory_test.dart

 // Copyright (c) 2013, 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.
 
 import 'dart:async';
 import 'dart:io';
 
 import 'package:path/path.dart' as path;
 import 'package:scheduled_test/descriptor.dart' as d;
 import 'package:scheduled_test/scheduled_test.dart';
@@ skipping 107 matching lines @@
         d.dir('subdir', [
           d.file('subfile1.txt', 'subcontents1'),
           d.file('subfile2.txt', 'subcontents2')
         ]),
         d.file('file1.txt', 'contents1'),
         d.file('file2.txt', 'contents2')
       ]).validate();
     });
 
     test('test 2', () {
-      expect(errors, everyElement(new isInstanceOf<ScheduleError>()));
-      expect(errors.length, equals(1));
-      expect(errors.first.error.toString(),
+      expect(errors.single, new isInstanceOf<ScheduleError>());
+      expect(errors.single.error.toString(),
           matches(r"^Directory not found: '[^']+[\\/]dir[\\/]subdir'\.$"));
     });
   }, passing: ['test 2']);
 
   expectTestsPass("directory().validate() fails if a file isn't found", () {
     var errors;
     test('test 1', () {
       scheduleSandbox();
 
       currentSchedule.onException.schedule(() {
@@ skipping 19 matching lines @@
         d.dir('subdir', [
           d.file('subfile1.txt', 'subcontents1'),
           d.file('subfile2.txt', 'subcontents2')
         ]),
         d.file('file1.txt', 'contents1'),
         d.file('file2.txt', 'contents2')
       ]).validate();
     });
 
     test('test 2', () {
-      expect(errors, everyElement(new isInstanceOf<ScheduleError>()));
-      expect(errors.length, equals(1));
-      expect(errors.first.error.toString(),
+      expect(errors.single, new isInstanceOf<ScheduleError>());
+      expect(errors.single.error.toString(),
           matches(r"^File not found: '[^']+[\\/]dir[\\/]file2\.txt'\.$"));
     });
   }, passing: ['test 2']);
 
   expectTestsPass("directory().validate() fails if multiple children aren't "
       "found or have the wrong contents", () {
     var errors;
     test('test 1', () {
       scheduleSandbox();
 
@@ skipping 18 matching lines @@
         d.dir('subdir', [
           d.file('subfile1.txt', 'subcontents1'),
           d.file('subfile2.txt', 'subcontents2')
         ]),
         d.file('file1.txt', 'contents1'),
         d.file('file2.txt', 'contents2')
       ]).validate();
     });
 
     test('test 2', () {
-      expect(errors, everyElement(new isInstanceOf<ScheduleError>()));
-      expect(errors.length, equals(1));
-      expect(errors.first.error.toString(), matches(
+      expect(errors.single, new isInstanceOf<ScheduleError>());
+      expect(errors.single.error.toString(), matches(
           r"^\* File not found: '[^']+[\\/]dir[\\/]subdir[\\/]subfile1\.txt'\."
               r"\n"
           r"\* File 'subfile2\.txt' should contain:\n"
           r"  \| subcontents2\n"
           r"  but actually contained:\n"
           r"  X subwrongtents2\n"
           r"\* File not found: '[^']+[\\/]dir[\\/]file2\.txt'\.$"));
     });
   }, passing: ['test 2']);
 
@@ skipping 28 matching lines @@
         d.dir('subdir', [
           d.file('subfile1.txt', 'subcontents1'),
           d.file('subfile2.txt', 'subcontents2')
         ]),
         d.file('file1.txt', 'contents1'),
         d.file('file2.txt', 'contents2')
       ]).validate();
     });
 
     test('test 2', () {
-      expect(errors, everyElement(new isInstanceOf<ScheduleError>()));
-      expect(errors.map((e) => e.error.toString()), equals([
-        "File 'subfile1.txt' should contain:\n"
-        "| subcontents1\n"
-        "but actually contained:\n"
-        "X wrongtents1"
-      ]));
+      expect(errors.single, new isInstanceOf<ScheduleError>());
+      expect(errors.single.error.toString(), equals(
+          "File 'subfile1.txt' should contain:\n"
+          "| subcontents1\n"
+          "but actually contained:\n"
+          "X wrongtents1"));
     });
   }, passing: ['test 2']);
 
   expectTestsPass("directory().load() loads a file", () {
     test('test', () {
       var dir = d.dir('dir', [d.file('name.txt', 'contents')]);
       expect(byteStreamToString(dir.load('name.txt')),
           completion(equals('contents')));
     });
   });
@@ skipping 14 matching lines @@
 
   expectTestsPass("directory().load() fails to load a nested directory", () {
     test('test', () {
       var dir = d.dir('dir', [
         d.dir('subdir', [
           d.file('name.txt', 'subcontents')
         ]),
         d.file('name.txt', 'contents')
       ]);
 
-      expect(dir.load('subdir').toList(),
-          throwsA(equals("Couldn't find a readable entry named 'subdir' within "
-              "'dir'.")));
+      expect(dir.load('subdir').toList(), throwsA(predicate(
+          (x) => x.toString() == "Couldn't find a readable entry named "
+                                 "'subdir' within 'dir'.")));
     });
   });
 
   expectTestsPass("directory().load() fails to load an absolute path", () {
     test('test', () {
       var dir = d.dir('dir', [d.file('name.txt', 'contents')]);
 
       expect(dir.load('/name.txt').toList(), throwsArgumentError);
     });
   });
 
   expectTestsPass("directory().load() fails to load '.', '..', or ''", () {
     test('test', () {
       var dir = d.dir('dir', [d.file('name.txt', 'contents')]);
 
       expect(dir.load('.').toList(), throwsArgumentError);
 
       expect(dir.load('..').toList(), throwsArgumentError);
 
       expect(dir.load('').toList(), throwsArgumentError);
     });
   });
 
   expectTestsPass("directory().load() fails to load a file that doesn't exist",
       () {
     test('test', () {
       var dir = d.dir('dir', [d.file('name.txt', 'contents')]);
 
-      expect(dir.load('not-name.txt').toList(),
-          throwsA(equals("Couldn't find a readable entry named 'not-name.txt' "
-              "within 'dir'.")));
+      expect(dir.load('not-name.txt').toList(), throwsA(predicate(
+          (x) => x.toString() == "Couldn't find a readable entry named "
+                                 "'not-name.txt' within 'dir'.")));
     });
   });
 
   expectTestsPass("directory().load() fails to load a file that exists "
       "multiple times", () {
     test('test', () {
       var dir = d.dir('dir', [
         d.file('name.txt', 'contents'),
         d.file('name.txt', 'contents')
       ]);
 
-      expect(dir.load('name.txt').toList(),
-          throwsA(equals("Found multiple readable entries named 'name.txt' "
-              "within 'dir'.")));
+      expect(dir.load('name.txt').toList(), throwsA(predicate(
+          (x) => x.toString() == "Found multiple readable entries named "
+                                 "'name.txt' within 'dir'.")));
     });
   });
 
   expectTestsPass("directory().load() loads a file next to a subdirectory with "
       "the same name", () {
     test('test', () {
       var dir = d.dir('dir', [
         d.file('name', 'contents'),
         d.dir('name', [d.file('subfile', 'contents')])
       ]);
@@ skipping 44 matching lines @@
           "'-- file2.txt"));
     });
   });
 
   expectTestsPass("directory().describe() with no contents returns the "
       "directory name", () {
     test('test', () {
       expect(d.dir('dir').describe(), equals('dir'));
     });
   });
-}
+}