Refactor command code and add support for --help and --version.

utils/tests/pub/pub_tests.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.
 
 #library('pub_tests');
 
 #import('dart:io');
 
 #import('test_pub.dart');
 #import('../../../lib/unittest/unittest.dart');
 
+final USAGE_STRING = """
+    Pub is a package manager for Dart.
+
+    Usage:
+
+      pub command [arguments]
+
+    The commands are:
+
+      list      print the contents of repositories
+      update    update a package's dependencies
+      version   print Pub version

[nweiz, 2012/04/25 20:49:41]

Since this output isn't sorted, this test is potentially flaky.

[Bob Nystrom, 2012/04/25 22:56:40]

It is sorted. See pub.dart:98.

[nweiz, 2012/04/25 23:09:22]

Oh, I see. I was confused by the TODO about the sorted map.

+
+    Use "pub help [command]" for more information about a command.
+    """;
+
+final VERSION_STRING = '''
+    Pub 0.0.0
+    ''';
+
 main() {
-  group('running pub with no command', () {
-    testPub('displays usage',
-      args: [],
-      output: '''
-      Pub is a package manager for Dart.
+  testPub('running pub with no command displays usage',
+    args: [],
+    output: USAGE_STRING);
 
-      Usage:
+  testPub('running pub with just --help displays usage',
+    args: ['--help'],
+    output: USAGE_STRING);

[nweiz, 2012/04/25 20:49:41]

May as well test for just "help" too.

[Bob Nystrom, 2012/04/25 22:56:40]

Help will be a full command (so you can do "help [command]" too, and it hasn't
been implemented yet. :)

 
-        pub command [arguments]
-
-      The commands are:
-
-        list      print the contents of repositories
-        version   print Pub version
-
-      Use "pub help [command]" for more information about a command.''');
-  });
+  testPub('running pub with just --version displays version',
+    args: ['--version'],
+    output: VERSION_STRING);
 
   group('an unknown command', () {
     testPub('displays an error message',
       args: ['quylthulg'],
       output: '''
       Unknown command "quylthulg".
-      Run "pub help" to see available commands.''',
+      Run "pub help" to see available commands.
+      ''',
       exitCode: 64);
   });
 
-  listCommand();
-  versionCommand();
+  group('pub list', listCommand);
+  group('pub update', updateCommand);
+  group('pub version', versionCommand);
 }
 
 listCommand() {
-  group('list cache', () {
+  group('cache', () {
     testPub('treats an empty directory as a package',
       cache: [
         dir('apple'),
         dir('banana'),
         dir('cherry')
       ],
       args: ['list', 'cache'],
       output: '''
       apple
       banana
-      cherry''');
+      cherry
+      ''');
   });
 }
 
+updateCommand() {
+  testPub('adds a dependent package',
+    cache: [
+      dir('foo', [
+        file('foo.dart', 'main() => "foo";')
+      ])
+    ],
+    app: dir('myapp', [
+      file('pubspec', 'foo')
+    ]),
+    args: ['update'],
+    expectedPackageDir: [
+      dir('foo', [
+        file('foo.dart', 'main() => "foo";')
+      ])
+    ],
+    output: '');
+
+  testPub('adds a transitively dependent package',
+    cache: [
+      dir('foo', [
+        file('foo.dart', 'main() => "foo";'),
+        file('pubspec', 'bar')
+      ]),
+      dir('bar', [
+        file('bar.dart', 'main() => "bar";'),
+      ])
+    ],
+    app: dir('myapp', [
+      file('pubspec', 'foo')
+    ]),
+    args: ['update'],
+    expectedPackageDir: [
+      dir('foo', [
+        file('foo.dart', 'main() => "foo";')
+      ]),
+      dir('bar', [
+        file('bar.dart', 'main() => "bar";'),
+      ])
+    ],
+    output: '');
+}
+
 versionCommand() {
-  group('the version command', () {
-    testPub('displays the current version',
-      args: ['version'],
-      output: 'Pub 0.0.0');
-  });
+  testPub('displays the current version',
+    args: ['version'],
+    output: VERSION_STRING);
 }