Fix error reporting on invalid command line args.

utils/tests/pub/test_pub.dart

 // Copyright (c) 2012, 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.
 
 /**
  * Test infrastructure for testing pub. Unlike typical unit tests, most pub
  * tests are integration tests that stage some stuff on the file system, run
  * pub, and then validate the results. This library provides an API to build
  * tests like that.
  */
@@ skipping 702 matching lines @@
    * directory is going to be created.
    */
   final Pattern name;
 
   Descriptor(this.name);
 
   /**
    * Creates the file or directory within [dir]. Returns a [Future] that is
    * completed after the creation is done.
    */
-  abstract Future create(dir);
+  Future create(dir);
 
   /**
    * Validates that this descriptor correctly matches the corresponding file
    * system entry within [dir]. Returns a [Future] that completes to `null` if
    * the entry is valid, or throws an error if it failed.
    */
-  abstract Future validate(String dir);
+  Future validate(String dir);
 
   /**
    * Deletes the file or directory within [dir]. Returns a [Future] that is
    * completed after the deletion is done.
    */
-  abstract Future delete(String dir);
+  Future delete(String dir);
 
   /**
    * Loads the file at [path] from within this descriptor. If [path] is empty,
    * loads the contents of the descriptor itself.
    */
-  abstract InputStream load(List<String> path);
+  InputStream load(List<String> path);
 
   /**
    * Schedules the directory to be created before Pub is run with [runPub]. The
    * directory will be created relative to the sandbox directory.
    */
   // TODO(nweiz): Use implicit closurization once issue 2984 is fixed.
   void scheduleCreate() => _schedule((dir) => this.create(dir));
 
   /**
    * Schedules the file or directory to be deleted recursively.
@@ skipping 399 matching lines @@
     var sinkStream = new ListInputStream();
     var tempDir;
     // TODO(nweiz): propagate any errors to the return value. See issue 3657.
     createTempDir().chain((_tempDir) {
       tempDir = _tempDir;
       return create(tempDir);
     }).then((tar) {
       var sourceStream = tar.openInputStream();
       pipeInputToInput(sourceStream,
                        sinkStream,
-                       () => tempDir.delete(recursive: true));
+                       () => tempDir.deleteRecursively());
     });
     return sinkStream;
   }
 }
 
 /**
  * A descriptor that validates that no file exists with the given name.
  */
 class NothingDescriptor extends Descriptor {
   NothingDescriptor(String name) : super(name);
@@ skipping 51 matching lines @@
 }
 
 /**
  * Schedules a callback to be called after Pub is run with [runPub], even if it
  * fails.
  */
 void _scheduleCleanup(_ScheduledEvent event) {
   if (_scheduledCleanup == null) _scheduledCleanup = [];
   _scheduledCleanup.add(event);
 }