Support running pub when it's not on the environment path

Author: kevmoo

webdev/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.1.3+1
+
+- Support running `pub` when it's not in the environment path.
+
 ## 0.1.3
 
 - Now runs on Windows.

webdev/lib/src/pubspec.dart

@@ -47,7 +47,7 @@ Future _runPubDeps() async {
 
   if (result.exitCode != 0) {
     throw new ProcessException(
-        'pub',
+        pubPath,
         ['deps'],
         '***OUT***\n${result.stdout}\n***ERR***\n${result.stderr}\n***',
         exitCode);

webdev/lib/src/util.dart

@@ -2,10 +2,26 @@
 // 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 'dart:io';
+
+import 'package:path/path.dart' as p;
 import 'package:pub_semver/pub_semver.dart';
 
 final supportedBuildRunnerVersionRange = new VersionRange(
     min: new Version(0, 8, 0),
     includeMin: true,
     max: new Version(0, 9, 0),
     includeMax: false);
+
+/// The path to the root directory of the SDK.
+final String _sdkDir = (() {
+  // The Dart executable is in "/path/to/sdk/bin/dart", so two levels up is
+  // "/path/to/sdk".
+  var aboveExecutable = p.dirname(p.dirname(Platform.resolvedExecutable));
+  assert(FileSystemEntity.isFileSync(p.join(aboveExecutable, 'version')));
+  return aboveExecutable;
+})();
+
+final String dartPath = p.join(_sdkDir, 'bin', 'dart');
+final String pubPath =
+    p.join(_sdkDir, 'bin', Platform.isWindows ? 'pub.bat' : 'pub');

webdev/pubspec.yaml

@@ -1,5 +1,5 @@
 name: webdev
-version: 0.1.3
+version: 0.1.3+1
 author: Dart Team <[email protected]>
 homepage: https://github.com/dart-lang/webdev
 description: >-

webdev/test/integration_test.dart

@@ -9,25 +9,14 @@ import 'package:path/path.dart' as p;
 import 'package:test/test.dart';
 import 'package:test_descriptor/test_descriptor.dart' as d;
 import 'package:test_process/test_process.dart';
+import 'package:webdev/src/util.dart';
 
 final _webdevBin = p.absolute(p.join('bin', 'webdev.dart'));
 
-/// The path to the root directory of the SDK.
-final String _sdkDir = (() {
-  // The Dart executable is in "/path/to/sdk/bin/dart", so two levels up is
-  // "/path/to/sdk".
-  var aboveExecutable = p.dirname(p.dirname(Platform.resolvedExecutable));
-  assert(FileSystemEntity.isFileSync(p.join(aboveExecutable, 'version')));
-  return aboveExecutable;
-})();
-
-final String _dartPath = p.join(_sdkDir, 'bin', 'dart');
-final String _pubPath = p.join(_sdkDir, 'bin', 'pub');
-
 Future<TestProcess> _runWebDev(List<String> args, {String workingDirectory}) {
   var fullArgs = [_webdevBin]..addAll(args);
 
-  return TestProcess.start(_dartPath, fullArgs,
+  return TestProcess.start(dartPath, fullArgs,
       workingDirectory: workingDirectory);
 }
 
@@ -218,7 +207,7 @@ dependencies:
 
   test('should succeed with valid configuration', () async {
     var exampleDirectory = p.absolute(p.join(p.current, '..', 'example'));
-    var process = await TestProcess.start(_pubPath, ['get'],
+    var process = await TestProcess.start(pubPath, ['get'],
         workingDirectory: exampleDirectory,
         environment: _getPubEnvironment(),
         runInShell: true);