pkg/test: Fix a few analyzer warnings

test/io.dart

Index: test/io.dart
diff --git a/test/io.dart b/test/io.dart
index 331a220b38545ad6cf5213663178e6ec9b876fb9..b9eec58fda77f15024435de83be344343bde130d 100644
--- a/test/io.dart
+++ b/test/io.dart
@@ -8,7 +8,6 @@ library test.test.io;
 
 import 'dart:async';
 import 'dart:io';
-import 'dart:isolate';
 
 import 'package:package_resolver/package_resolver.dart';
 import 'package:path/path.dart' as p;
@@ -134,12 +133,11 @@ ScheduledProcess runTest(List<String> args, {String reporter,
 /// Runs Dart.
 ScheduledProcess runDart(List<String> args, {Map<String, String> environment,
     String description}) {
-  var allArgs = Platform.executableArguments
-      .where((arg) =>
-          !arg.startsWith("--package-root=") && !arg.startsWith("--packages="))
-      .toList()
-      ..add(PackageResolver.current.processArgument)
-      ..addAll(args);
+  var allArgs = <Object>[]

[nweiz, 2016/08/30 20:23:20]

<String>[]

[kevmoo, 2016/08/30 22:11:49]

Nope. This was the problem. This contains both `String` and `Future<String>`
hence the need for the fix.

+    ..addAll(Platform.executableArguments.where((arg) =>
+        !arg.startsWith("--package-root=") && !arg.startsWith("--packages=")))
+    ..add(PackageResolver.current.processArgument)
+    ..addAll(args);
 
   return new ScheduledProcess.start(
       p.absolute(Platform.resolvedExecutable), allArgs,