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

Unified Diff: utils/tests/pub/test_pub.dart

Issue 10253015: Fix checked mode bugs in pub (and try enabling on bots again). (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 8 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « utils/tests/pub/pub.status ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: utils/tests/pub/test_pub.dart
diff --git a/utils/tests/pub/test_pub.dart b/utils/tests/pub/test_pub.dart
index baace39528ea5b672e710ffe54e8d9fed18993bb..ddac2322e53425df15f7d2720bd284eeb36c1c23 100644
--- a/utils/tests/pub/test_pub.dart
+++ b/utils/tests/pub/test_pub.dart
@@ -84,6 +84,7 @@ void testPub(String description, [List<Descriptor> cache, Descriptor app,
future.handleException((error) {
deleteSandboxIfCreated();
+ return false;
});
});
}
@@ -116,7 +117,7 @@ Future<ProcessResult> _runPub(List<String> pubArgs, String workingDir) {
// Find the main pub entrypoint.
final pubPath = fs.joinPaths(scriptDir, '../../pub/pub.dart');
- final args = [pubPath];
+ final args = ['--enable-type-checks', '--enable-asserts', pubPath];
args.addAll(pubArgs);
return runProcess(dartBin, args, workingDir);
@@ -195,7 +196,7 @@ class Descriptor {
* Creates the file or directory within [dir]. Returns a [Future] that is
* completed after the creation is done.
*/
- abstract Future create(String dir);
+ abstract Future create(dir);
/**
* Validates that this descriptor correctly matches the corresponding file
@@ -222,7 +223,7 @@ class FileDescriptor extends Descriptor {
* Creates the file within [dir]. Returns a [Future] that is completed after
* the creation is done.
*/
- Future<File> create(String dir) {
+ Future<File> create(dir) {
return writeTextFile(join(dir, name), contents);
}
@@ -263,7 +264,7 @@ class DirectoryDescriptor extends Descriptor {
* Creates the file within [dir]. Returns a [Future] that is completed after
* the creation is done.
*/
- Future<Directory> create(String parentDir) {
+ Future<Directory> create(parentDir) {
final completer = new Completer<Directory>();
// Create the directory.
@@ -272,7 +273,7 @@ class DirectoryDescriptor extends Descriptor {
completer.complete(dir);
} else {
// Recursively create all of its children.
- final childFutures = contents.map((child) => child.create(dir.path));
+ final childFutures = contents.map((child) => child.create(dir));
Futures.wait(childFutures).then((_) {
// Only complete once all of the children have been created too.
completer.complete(dir);
« no previous file with comments | « utils/tests/pub/pub.status ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698