Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(148)

Side by Side Diff: pkg/scheduled_test/lib/src/descriptor/nothing_descriptor.dart

Issue 24276010: Change package scheduled_test to throw TestFailures rather than strings. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Also in async_test.dart Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 library descriptor.nothing; 5 library descriptor.nothing;
6 6
7 import 'dart:async'; 7 import 'dart:async';
8 import 'dart:io'; 8 import 'dart:io';
9 9
10 import 'package:path/path.dart' as path; 10 import 'package:path/path.dart' as path;
11 11
12 import '../../descriptor.dart'; 12 import '../../descriptor.dart';
13 import '../../scheduled_test.dart'; 13 import '../../scheduled_test.dart';
14 14
15 /// A descriptor that validates that no file exists with the given name. 15 /// A descriptor that validates that no file exists with the given name.
16 /// Creating this descriptor is a no-op and loading from it is invalid. 16 /// Creating this descriptor is a no-op and loading from it is invalid.
17 class NothingDescriptor extends Descriptor { 17 class NothingDescriptor extends Descriptor {
18 NothingDescriptor(String name) 18 NothingDescriptor(String name)
19 : super(name); 19 : super(name);
20 20
21 Future create([String parent]) => new Future.value(); 21 Future create([String parent]) => new Future.value();
22 22
23 Future validate([String parent]) => schedule(() => validateNow(parent), 23 Future validate([String parent]) => schedule(() => validateNow(parent),
24 "validating '$name' doesn't exist"); 24 "validating '$name' doesn't exist");
25 25
26 Future validateNow([String parent]) => new Future.sync(() { 26 Future validateNow([String parent]) => new Future.sync(() {
27 if (parent == null) parent = defaultRoot; 27 if (parent == null) parent = defaultRoot;
28 var fullPath = path.join(parent, name); 28 var fullPath = path.join(parent, name);
29 if (new File(fullPath).existsSync()) { 29 if (new File(fullPath).existsSync()) {
30 throw "Expected nothing to exist at '$fullPath', but found a file."; 30 fail("Expected nothing to exist at '$fullPath', but found a file.");
31 } else if (new Directory(fullPath).existsSync()) { 31 } else if (new Directory(fullPath).existsSync()) {
32 throw "Expected nothing to exist at '$fullPath', but found a " 32 fail("Expected nothing to exist at '$fullPath', but found a directory.");
33 "directory.";
34 } else { 33 } else {
35 return; 34 return;
36 } 35 }
37 }); 36 });
38 37
39 String describe() => "nothing at '$name'"; 38 String describe() => "nothing at '$name'";
40 } 39 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698