Make pub publish more user friendly:

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 611 matching lines @@
     return server.url.chain((url) {
       var tokenEndpoint = url.resolve('/token');
       if (args == null) args = [];
       args = flatten(['lish', '--server', url.toString(), args]);
       return _doPub(startProcess, sandboxDir, args, tokenEndpoint);
     });
   });
   return new ScheduledProcess("pub lish", process);
 }
 
+/// Handles the beginning confirmation process for uploading a packages.
+/// Ensures that the right output is shown and then enters "y" to confirm the
+/// upload.
+void confirmPublish(ScheduledProcess pub) {

[nweiz, 2012/12/12 21:21:21]

I don't like that this only works with a specific package structure. Maybe just
validate that the first line matches 'Publishing ".*" [0-9]\.[0-9]\.[0-9]:",
then add some sort of ScheduledProcess.ignoreStdoutUntil function and use that?

You could also just add a TODO.

[Bob Nystrom, 2012/12/12 21:45:36]

Agreed completely. Added a TODO.

+  expectLater(pub.nextLine(), equals('Publishing "test_pkg" 1.0.0:'));
+  expectLater(pub.nextLine(), equals("|-- LICENSE"));
+  expectLater(pub.nextLine(), equals("|-- lib"));
+  expectLater(pub.nextLine(), equals("|   '-- test_pkg.dart"));
+  expectLater(pub.nextLine(), equals("'-- pubspec.yaml"));
+  expectLater(pub.nextLine(), equals(""));
+
+  pub.writeLine("y");
+}
+
+
 /// Calls [fn] with appropriately modified arguments to run a pub process. [fn]
 /// should have the same signature as [startProcess], except that the returned
 /// [Future] may have a type other than [Process].
 Future _doPub(Function fn, sandboxDir, List<String> args, Uri tokenEndpoint) {
   String pathInSandbox(path) => join(getFullPath(sandboxDir), path);
 
   return ensureDir(pathInSandbox(appPath)).chain((_) {
     // Find a Dart executable we can use to spawn. Use the same one that was
     // used to run this script itself.
     var dartBin = new Options().executable;
@@ skipping 999 matching lines @@
 /// calling [completion] is unnecessary.
 void expectLater(Future actual, matcher, {String reason,
     FailureHandler failureHandler, bool verbose: false}) {
   _schedule((_) {
     return actual.transform((value) {
       expect(value, matcher, reason: reason, failureHandler: failureHandler,
           verbose: false);
     });
   });
 }