Actually enable the pub update tests.

utils/tests/pub/pub_test.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');
@@ skipping 41 matching lines @@
           error: '''
           Unknown command "quylthulg".
           Run "pub help" to see available commands.
           ''',
           exitCode: 64);
     });
   });
 
   group('pub list', listCommand);
   group('pub install', installCommand);
+  group('pub update', updateCommand);
   group('pub version', versionCommand);
 }
 
 listCommand() {
   // TODO(rnystrom): We don't currently have any sources that are cached, so
   // we can't test this right now.
   /*
   group('cache', () {
     test('treats an empty directory as a package', () {
       dir(cachePath, [
@@ skipping 808 matching lines @@
     git('bar.git', [
       file('bar.dart', 'main() => "bar";')
     ]).scheduleCreate();
 
     dir(appPath, [
       file('pubspec.yaml', '''
 dependencies:
   foo:
     git: ../foo.git
   bar:
-    git: ../foo.git
+    git: ../bar.git
 ''')
     ]).scheduleCreate();
 
     schedulePub(args: ['install'],
         output: const RegExp(@"Dependencies installed!$"));
 
     dir(packagesPath, [
       dir('foo', [
         file('foo.dart', 'main() => "foo";')
       ]),
@@ skipping 36 matching lines @@
       git('bar.git', [
         file('bar.dart', 'main() => "bar";')
       ]).scheduleCreate();
 
       dir(appPath, [
         file('pubspec.yaml', '''
 dependencies:
   foo:
     git: ../foo.git
   bar:
-    git: ../foo.git
+    git: ../bar.git
 ''')
       ]).scheduleCreate();
 
       schedulePub(args: ['install'],
           output: const RegExp(@"Dependencies installed!$"));
 
       dir(packagesPath, [
         dir('foo', [
           file('foo.dart', 'main() => "foo";')
         ]),
         dir('bar', [
           file('bar.dart', 'main() => "bar";')
         ])
       ]).scheduleValidate();
 
       git('foo.git', [
         file('foo.dart', 'main() => "foo 2";')
       ]).scheduleCommit();
 
       git('bar.git', [
         file('bar.dart', 'main() => "bar 2";')
       ]).scheduleCommit();
 
-      schedulePub(args: ['update foo'],
+      schedulePub(args: ['update', 'foo'],
           output: const RegExp(@"Dependencies updated!$"));
 
       dir(packagesPath, [
         dir('foo', [
           file('foo.dart', 'main() => "foo 2";')
         ]),
         dir('bar', [
           file('bar.dart', 'main() => "bar";')
         ])
       ]).scheduleValidate();
 
       run();
     });
 
-    test("updates one locked Git package's dependencies", () {
+    test("doesn't update one locked Git package's dependencies if it's not "
+        "necessary", () {
       ensureGit();
 
       git('foo.git', [
         file('foo.dart', 'main() => "foo";'),
         file('pubspec.yaml', '''
 dependencies:
   foo-dep:
     git: ../foo-dep.git
 ''')
       ]).scheduleCreate();
 
       git('foo-dep.git', [
         file('foo-dep.dart', 'main() => "foo-dep";')
       ]).scheduleCreate();
 
-      git('bar.git', [
-        file('bar.dart', 'main() => "bar";'),
-        file('pubspec.yaml', '''
-dependencies:
-  bar-dep:
-    git: ../bar-dep.git
-''')
-      ]).scheduleCreate();
-
-      git('bar-dep.git', [
-        file('bar-dep.dart', 'main() => "bar-dep";')
-      ]).scheduleCreate();
-
       dir(appPath, [
         file('pubspec.yaml', '''
 dependencies:
   foo:
     git: ../foo.git
-  bar:
-    git: ../foo.git
 ''')
       ]).scheduleCreate();
 
       schedulePub(args: ['install'],
           output: const RegExp(@"Dependencies installed!$"));
 
       dir(packagesPath, [
         dir('foo', [
           file('foo.dart', 'main() => "foo";'),
           file('pubspec.yaml', '''
 dependencies:
   foo-dep:
     git: ../foo-dep.git
 ''')
         ]),
         dir('foo-dep', [
           file('foo-dep.dart', 'main() => "foo-dep";')
-        ]),
-        dir('bar', [
-          file('bar.dart', 'main() => "bar";'),
-          file('pubspec.yaml', '''
-dependencies:
-  bar-dep:
-    git: ../bar-dep.git
-''')
-        ]),
-        dir('bar-dep', [
-          file('bar-dep.dart', 'main() => "bar-dep";')
         ])
       ]).scheduleValidate();
 
+      git('foo.git', [
+        file('foo.dart', 'main() => "foo 2";'),
+        file('pubspec.yaml', '''
+dependencies:
+  foo-dep:
+    git: ../foo-dep.git
+''')
+      ]).scheduleCreate();
+
       git('foo-dep.git', [
         file('foo-dep.dart', 'main() => "foo-dep 2";')
       ]).scheduleCommit();
 
-      git('bar-dep.git', [
-        file('bar-dep.dart', 'main() => "bar-dep 2";')
-      ]).scheduleCommit();
-
-      schedulePub(args: ['update foo'],
+      schedulePub(args: ['update', 'foo'],
           output: const RegExp(@"Dependencies updated!$"));
 
       dir(packagesPath, [
         dir('foo', [
-          file('foo.dart', 'main() => "foo";'),
+          file('foo.dart', 'main() => "foo 2";'),
           file('pubspec.yaml', '''
 dependencies:
   foo-dep:
     git: ../foo-dep.git
 ''')
         ]),
         dir('foo-dep', [
-          file('foo-dep.dart', 'main() => "foo-dep 2";')
+          file('foo-dep.dart', 'main() => "foo-dep";')
         ]),
-        dir('bar', [
-          file('bar.dart', 'main() => "bar";'),
+      ]).scheduleValidate();
+
+      run();
+    });
+
+    test("updates one locked pub server package's dependencies if it's "
+        "necessary", () {
+      servePackages("localhost", 3123, [
+        '''
+name: foo
+version: 1.0.0
+dependencies:
+  foo-dep:
+    repo:
+      name: foo-dep
+      url: http://localhost:3123
+''',
+        '{name: foo-dep, version: 1.0.0}'
+      ]);
+
+      dir(appPath, [
+        file('pubspec.yaml', '''
+dependencies:
+  foo:
+    repo:
+      name: foo
+      url: http://localhost:3123
+''')
+      ]).scheduleCreate();
+
+      schedulePub(args: ['install'],
+          output: const RegExp(@"Dependencies installed!$"));
+
+      dir(packagesPath, [
+        dir('foo', [
+          file('foo.dart', 'main() => print("foo 1.0.0");'),
           file('pubspec.yaml', '''
+name: foo
+version: 1.0.0
 dependencies:
-  bar-dep:
-    git: ../bar-dep.git
+  foo-dep:
+    repo:
+      name: foo-dep
+      url: http://localhost:3123
 ''')
         ]),
-        dir('bar-dep', [
-          file('bar-dep.dart', 'main() => "bar-dep";')
+        dir('foo-dep', [
+          file('foo-dep.dart', 'main() => print("foo-dep 1.0.0");')
         ])
       ]).scheduleValidate();
 
+      servePackages("localhost", 3123, [
+        '''
+name: foo
+version: 2.0.0
+dependencies:
+  foo-dep:
+    repo: {name: foo-dep, url: http://localhost:3123}
+    version: ">1.0.0"
+''',
+        '{name: foo-dep, version: 2.0.0}'
+      ]);
+
+      schedulePub(args: ['update', 'foo'],
+          output: const RegExp(@"Dependencies updated!$"));
+
+      dir(packagesPath, [
+        dir('foo', [
+          file('foo.dart', 'main() => print("foo 2.0.0");'),
+          file('pubspec.yaml', '''
+name: foo
+version: 2.0.0
+dependencies:
+  foo-dep:
+    repo: {name: foo-dep, url: http://localhost:3123}
+    version: ">1.0.0"
+''')
+        ]),
+        dir('foo-dep', [
+          file('foo-dep.dart', 'main() => print("foo-dep 2.0.0");')
+        ])
+      ]).scheduleValidate();
+
       run();
     });
   });
 }
 
 versionCommand() {
   test('displays the current version', () =>
     runPub(args: ['version'], output: VERSION_STRING));
 }