Make pub handle missing git more gracefully.

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 28 matching lines @@
 
   test('running pub with just -h displays usage', () =>
       runPub(args: ['-h'], output: USAGE_STRING));
 
   test('running pub with just --version displays version', () =>
       runPub(args: ['--version'], output: VERSION_STRING));
 
   group('an unknown command', () {
     test('displays an error message', () {
       runPub(args: ['quylthulg'],
-          output: '''
+          error: '''
           Unknown command "quylthulg".
           Run "pub help" to see available commands.
           ''',
           exitCode: 64);
     });
   });
 
   group('pub list', listCommand);
   group('pub install', installCommand);
   group('pub version', versionCommand);
@@ skipping 76 matching lines @@
         file('bar.dart', 'main() => "bar";'),
       ])
     ]).scheduleValidate();
 
     runPub(args: ['install'],
         output: '''
         Dependencies installed!
         ''');
   });
 
-/*
   test('checks out a package from Git', () {
-    git('foo.git', [
-      file('foo.dart', 'main() => "foo";')
-    ]).scheduleCreate();
+    withGit(() {
+      git('foo.git', [
+        file('foo.dart', 'main() => "foo";')
+      ]).scheduleCreate();
 
-    dir(appPath, [
-      file('pubspec', '''
+      dir(appPath, [
+        file('pubspec', '''
 dependencies:
   foo:
     git: ../foo.git
 ''')
-    ]).scheduleCreate();
+      ]).scheduleCreate();
 
-    dir(packagesPath, [
-      dir('foo', [
-        file('foo.dart', 'main() => "foo";')
-      ])
-    ]).scheduleValidate();
+      dir(packagesPath, [
+        dir('foo', [
+          file('foo.dart', 'main() => "foo";')
+        ])
+      ]).scheduleValidate();
 
-    runPub(args: ['install'],
-        output: const RegExp(@"Dependencies installed!$"));
+      runPub(args: ['install'],
+          output: const RegExp(@"Dependencies installed!$"));
+    });
   });
 
   test('checks out packages transitively from Git', () {
-    git('foo.git', [
-      file('foo.dart', 'main() => "foo";'),
-      file('pubspec', '''
+    withGit(() {
+      git('foo.git', [
+        file('foo.dart', 'main() => "foo";'),
+        file('pubspec', '''
 dependencies:
   bar:
     git: ../bar.git
 ''')
-    ]).scheduleCreate();
+      ]).scheduleCreate();
 
-    git('bar.git', [
-      file('bar.dart', 'main() => "bar";')
-    ]).scheduleCreate();
+      git('bar.git', [
+        file('bar.dart', 'main() => "bar";')
+      ]).scheduleCreate();
 
-    dir(appPath, [
-      file('pubspec', '''
+      dir(appPath, [
+        file('pubspec', '''
 dependencies:
   foo:
     git: ../foo.git
 ''')
-    ]).scheduleCreate();
+      ]).scheduleCreate();
 
-    dir(packagesPath, [
-      dir('foo', [
-        file('foo.dart', 'main() => "foo";')
-      ]),
-      dir('bar', [
-        file('bar.dart', 'main() => "bar";')
-      ])
-    ]).scheduleValidate();
+      dir(packagesPath, [
+        dir('foo', [
+          file('foo.dart', 'main() => "foo";')
+        ]),
+        dir('bar', [
+          file('bar.dart', 'main() => "bar";')
+        ])
+      ]).scheduleValidate();
 
-    runPub(args: ['install'],
-        output: const RegExp("Dependencies installed!\$"));
+      runPub(args: ['install'],
+          output: const RegExp("Dependencies installed!\$"));
+    });
   });
-*/
 }
 
 versionCommand() {
   test('displays the current version', () =>
     runPub(args: ['version'], output: VERSION_STRING));
 }