Add tests for "pub run --list" command.

Issue dart-lang#1323

Author: antonmoiseev

lib/src/command/run.dart

@@ -22,15 +22,13 @@ class RunCommand extends PubCommand {
   bool get allowTrailingOptions => false;
 
   RunCommand() {
-    argParser.addFlag("checked", abbr: "c",
-        help: "Enable runtime type checks and assertions.");
+    argParser.addFlag("checked",
+        abbr: "c", help: "Enable runtime type checks and assertions.");
+    argParser.addFlag('list',
+        help: 'List all available executables.', negatable: false);
     argParser.addOption("mode",
         help: 'Mode to run transformers in.\n'
-              '(defaults to "release" for dependencies, "debug" for '
-                'entrypoint)');
-    argParser.addFlag('list',
-        help: 'List all executables that can by invoked with pub run',
-        negatable: false);
+            '(defaults to "release" for dependencies, "debug" for entrypoint)');
   }
 
   Future run() async {
@@ -55,10 +53,10 @@ class RunCommand extends PubCommand {
       executable = components[1];
 
       if (p.split(executable).length > 1) {
-      // TODO(nweiz): Use adjacent strings when the new async/await compiler
-      // lands.
-        usageException("Cannot run an executable in a subdirectory of a " +
-            "dependency.");
+        // TODO(nweiz): Use adjacent strings when the new async/await compiler
+        // lands.
+        usageException(
+            "Cannot run an executable in a subdirectory of a dependency.");
       }
     } else if (onlyIdentifierRegExp.hasMatch(executable)) {
       // "pub run foo" means the same thing as "pub run foo:foo" as long as
@@ -82,7 +80,8 @@ class RunCommand extends PubCommand {
 
   /// Lists all executables reachable from [entrypoint].
   void _listExecutables() {
-    var ownExecutables = _listExecutablesFor(entrypoint.root);
+    var ownExecutables = _listExecutablesFor(entrypoint.root)
+        .map((executable) => p.join('bin', executable));
 
     var packageExecutables = entrypoint.root.immediateDependencies
         .map((dep) => entrypoint.packageGraph.packages[dep.name])
@@ -94,7 +93,7 @@ class RunCommand extends PubCommand {
       ..addAll(packageExecutables)
       ..sort();
 
-    log.message('Executables:\n  ${allExecutables.join('\n  ')}');
+    log.message(log.bold(allExecutables.join('\n')));
   }
 
   /// Lists all Dart files in the `bin` directory of the [package].

test/run/errors_if_no_executable_is_given_test.dart

@@ -20,6 +20,7 @@ Must specify an executable to run.
 Usage: pub run <executable> [args...]
 -h, --help            Print this usage information.
 -c, --[no-]checked    Enable runtime type checks and assertions.
+    --list            List all available executables.
     --mode            Mode to run transformers in.
                       (defaults to "release" for dependencies, "debug" for entrypoint)
 

test/run/errors_if_path_in_dependency_test.dart

@@ -27,6 +27,7 @@ Cannot run an executable in a subdirectory of a dependency.
 Usage: pub run <executable> [args...]
 -h, --help            Print this usage information.
 -c, --[no-]checked    Enable runtime type checks and assertions.
+    --list            List all available executables.
     --mode            Mode to run transformers in.
                       (defaults to "release" for dependencies, "debug" for entrypoint)
 

test/run/list_test.dart

@@ -0,0 +1,106 @@
+// Copyright (c) 2016, 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 'package:path/path.dart' as p;
+
+import '../descriptor.dart' as d;
+import '../test_pub.dart';
+
+main() {
+  integration("lists executables in entrypoint's bin", () {
+    d.dir(appPath, [
+      d.appPubspec(),
+      d.dir('bin', [
+        d.file('foo.dart'),
+        d.file('bar.dart'),
+        d.dir('sub', [d.file('baz.dart')])
+      ])
+    ]).create();
+
+    schedulePub(
+        args: ['run', '--list'],
+        output: '''
+${p.join('bin', 'bar')}
+${p.join('bin', 'foo')}
+${p.join('bin', 'sub', 'baz')}
+''');
+  });
+
+  integration('lists only Dart files', () {
+    d.dir(appPath, [
+      d.appPubspec(),
+      d.dir('bin', [d.file('foo.dart'), d.file('bar.sh')])
+    ]).create();
+
+    schedulePub(args: ['run', '--list'], output: p.join('bin', 'foo'));
+  });
+
+  integration('lists executables from a dependency', () {
+    d.dir('foo', [
+      d.libPubspec('foo', '1.0.0'),
+      d.dir('bin', [
+        d.file('bar.dart'),
+        d.dir('sub', [d.file('baz.dart'),])
+      ])
+    ]).create();
+
+    d.dir(appPath, [
+      d.appPubspec({
+        'foo': {'path': '../foo'}
+      })
+    ]).create();
+
+    pubGet();
+    schedulePub(
+        args: ['run', '--list'],
+        output: '''
+foo:bar
+foo:${p.join('sub', 'baz')}
+''');
+  });
+
+  integration('lists executables only from immediate dependencies', () {
+    d.dir('foo', [
+      d.libPubspec('foo', '1.0.0'),
+      d.dir('bin', [d.file('baz.dart')])
+    ]).create();
+
+    d.dir('bar', [
+      d.libPubspec('bar', '1.0.0', deps: {
+        'foo': {'path': '../foo'}
+      }),
+      d.dir('bin', [d.file('baz.dart')])
+    ]).create();
+
+    d.dir(appPath, [
+      d.appPubspec({
+        'bar': {'path': '../bar'}
+      })
+    ]).create();
+
+    pubGet();
+    schedulePub(args: ['run', '--list'], output: 'bar:baz');
+  });
+
+  integration('normilizes executable names', () {
+    d.dir('foo', [
+      d.libPubspec('foo', '1.0.0'),
+      d.dir('bin', [d.file('foo.dart'), d.file('bar.dart')])
+    ]).create();
+
+    d.dir(appPath, [
+      d.appPubspec({
+        'foo': {'path': '../foo'}
+      })
+    ]).create();
+
+    pubGet();
+    schedulePub(
+        args: ['run', '--list'],
+        output: '''
+foo
+foo:bar
+''');
+  });
+}