When testing, make sure we run Dart from an absolute path.

test/io.dart

 // Copyright (c) 2015, 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 test.test.io;
 
 import 'dart:async';
 import 'dart:io';
 
 import 'package:path/path.dart' as p;
 import 'package:test/src/util/io.dart';
 
 /// The path to the root directory of the `test` package.
 final String packageDir = p.dirname(p.dirname(libraryPath(#test.test.io)));
 
 /// The path to the `pub` executable in the current Dart SDK.
-final _pubPath = p.join(
+final _pubPath = p.absolute(p.join(
     p.dirname(Platform.executable),
-    Platform.isWindows ? 'pub.bat' : 'pub');
+    Platform.isWindows ? 'pub.bat' : 'pub'));
 
 /// Runs the test executable with the package root set properly.
 ProcessResult runUnittest(List<String> args, {String workingDirectory,
     Map<String, String> environment}) {
   var allArgs = [
     p.absolute(p.join(packageDir, 'bin/test.dart')),
     "--package-root=${p.join(packageDir, 'packages')}"
   ]..addAll(args);
 
   if (environment == null) environment = {};
   environment.putIfAbsent("_UNITTEST_USE_COLOR", () => "false");
 
   // TODO(nweiz): Use ScheduledProcess once it's compatible.
   return runDart(allArgs, workingDirectory: workingDirectory,
       environment: environment);
 }
 
 /// Runs Dart.
 ProcessResult runDart(List<String> args, {String workingDirectory,
     Map<String, String> environment}) {
-  var allArgs = Platform.executableArguments.toList()..addAll(args);
+  var allArgs = Platform.executableArguments.map((arg) {
+    // The bots use a relative package root, which we need to make absolute if

[kevmoo, 2015/04/21 22:53:10]

want to make this comment more general than just for our 'bots' – could apply to
anyone using a relative package root, right?

[nweiz, 2015/04/21 22:57:33]

Done.

+    // we're going to run in a different working directory.
+    if (!arg.startsWith("--package-root=")) return arg;
+    return "--package-root=" +
+        p.absolute(arg.substring("--package-root=".length));
+  }).toList()..addAll(args);
 
   // TODO(nweiz): Use ScheduledProcess once it's compatible.
-  return Process.runSync(Platform.executable, allArgs,
+  return Process.runSync(p.absolute(Platform.executable), allArgs,
       workingDirectory: workingDirectory, environment: environment);
 }
 
 /// Runs Pub.
 ProcessResult runPub(List<String> args, {String workingDirectory,
     Map<String, String> environment}) {
   // TODO(nweiz): Use ScheduledProcess once it's compatible.
   return Process.runSync(_pubPath, args,
       workingDirectory: workingDirectory, environment: environment);
 }
@@ skipping 23 matching lines @@
       workingDirectory: workingDirectory, environment: environment);
 }
 
 /// Starts Pub.
 Future<Process> startPub(List<String> args, {String workingDirectory,
     Map<String, String> environment}) {
   // TODO(nweiz): Use ScheduledProcess once it's compatible.
   return Process.start(_pubPath, args,
       workingDirectory: workingDirectory, environment: environment);
 }