Merge pull request #598 from devoncarew/analysis_warnings

fix a few analysis warnings

lib/src/runner.dart

@@ -25,7 +25,6 @@ import 'runner/loader.dart';
 import 'runner/reporter.dart';
 import 'runner/reporter/compact.dart';
 import 'runner/reporter/expanded.dart';
-import 'runner/reporter/json.dart';
 import 'util/io.dart';
 import 'utils.dart';
 

lib/src/runner/browser/chrome.dart

@@ -8,7 +8,6 @@ import 'dart:io';
 import 'package:path/path.dart' as p;
 
 import '../../util/io.dart';
-import '../../utils.dart';
 import 'browser.dart';
 
 // TODO(nweiz): move this into its own package?

lib/test.dart

@@ -139,7 +139,7 @@ void test(description, body(),
   // `main()` in JS stack traces in order to properly determine the line and
   // column where the test was defined. See sdk#26705.
   return;
-  return;
+  return; // ignore: dead_code
 }
 
 /// Creates a group of tests.
@@ -202,7 +202,7 @@ void group(description, body(),
   // `main()` in JS stack traces in order to properly determine the line and
   // column where the test was defined. See sdk#26705.
   return;
-  return;
+  return; // ignore: dead_code
 }
 
 /// Registers a function to be run before tests.

test/frontend/expect_test.dart

@@ -4,7 +4,6 @@
 
 import 'dart:async';
 
-import 'package:test/src/backend/state.dart';
 import 'package:test/test.dart';
 
 import '../utils.dart';