A bunch of strong mode fixes

Author: kevmoo

.analysis_options

@@ -0,0 +1,2 @@
+analyzer:
+  strong-mode: true

lib/src/generate/build.dart

@@ -42,7 +42,7 @@ Future<BuildResult> build(List<List<Phase>> phaseGroups,
     AssetReader reader,
     AssetWriter writer,
     Level logLevel,
-    onLog(LogRecord),
+    onLog(LogRecord record),
     Stream terminateEventStream}) async {
   var options = new BuildOptions(
       packageGraph: packageGraph,
@@ -87,7 +87,7 @@ Stream<BuildResult> watch(List<List<Phase>> phaseGroups,
     AssetReader reader,
     AssetWriter writer,
     Level logLevel,
-    onLog(LogRecord),
+    onLog(LogRecord record),
     Duration debounceDelay,
     DirectoryWatcherFactory directoryWatcherFactory,
     Stream terminateEventStream}) {
@@ -124,7 +124,7 @@ Stream<BuildResult> serve(List<List<Phase>> phaseGroups,
     AssetReader reader,
     AssetWriter writer,
     Level logLevel,
-    onLog(LogRecord),
+    onLog(LogRecord record),
     Duration debounceDelay,
     DirectoryWatcherFactory directoryWatcherFactory,
     Stream terminateEventStream,

lib/src/generate/options.dart

@@ -37,7 +37,7 @@ class BuildOptions {
       {this.debounceDelay,
       this.directoryWatcherFactory,
       Level logLevel,
-      onLog(LogRecord),
+      onLog(LogRecord record),
       this.packageGraph,
       this.reader,
       this.writer,

lib/src/package_graph/package_graph.dart

@@ -166,8 +166,8 @@ PackageDependencyType _dependencyType(source) {
 }
 
 /// Gets the deps from a yaml file, taking into account dependency_overrides.
-Map<String, YamlMap> _depsFromYaml(YamlMap yaml, {bool isRoot: false}) {
-  var deps = new Map.from(yaml['dependencies'] ?? {});
+Map<String, dynamic> _depsFromYaml(YamlMap yaml, {bool isRoot: false}) {
+  var deps = new Map<String, dynamic>.from(yaml['dependencies'] ?? {});
   if (isRoot) {
     deps.addAll(new Map.from(yaml['dev_dependencies'] ?? {}));
     yaml['dependency_overrides']?.forEach((dep, source) {

test/analyzer/resolver_test.dart

@@ -10,6 +10,8 @@ import 'package:build/build.dart';
 import 'package:build/src/builder/build_step_impl.dart';
 import 'package:build/src/util/barback.dart';
 
+import 'package:logging/logging.dart';
+
 import '../common/common.dart';
 
 // Ported from
@@ -18,7 +20,7 @@ main() {
   var entryPoint = makeAssetId('a|web/main.dart');
   Future validateResolver(
       {Map<String, String> inputs,
-      validator(Resolver),
+      validator(Resolver resolver),
       List messages: const []}) async {
     var writer = new InMemoryAssetWriter();
     var reader = new InMemoryAssetReader(writer.assets);
@@ -28,7 +30,7 @@ main() {
     var builder = new TestBuilder(validator);
     var buildStep =
         new BuildStepImpl(assets[entryPoint], [], reader, writer, 'a');
-    var logs = [];
+    var logs = <LogRecord>[];
     if (messages != null) {
       buildStep.logger.onRecord.listen(logs.add);
     }

test/asset/cache_test.dart

@@ -110,7 +110,7 @@ main() {
 
     test('Multiple readAsString calls wait on the same future', () async {
       childReaderAssets[a.id] = new DatedString(a.stringContents);
-      var futures = [];
+      var futures = <Future>[];
       futures.add(reader.readAsString(a.id));
       futures.add(reader.readAsString(a.id));
       expect(futures[0], futures[1]);
@@ -122,7 +122,7 @@ main() {
 
     test('Multiple hasInput calls return the same future', () async {
       childReaderAssets[a.id] = new DatedString(a.stringContents);
-      var futures = [];
+      var futures = <Future>[];
       futures.add(reader.hasInput(a.id));
       futures.add(reader.hasInput(a.id));
       expect(futures[0], futures[1]);

test/builder/build_step_impl_test.dart

@@ -76,7 +76,7 @@ main() {
         Logger.root.level = Level.ALL;
         var logger = buildStep.logger;
         expect(logger.fullName, primary.id.toString());
-        var logs = [];
+        var logs = <LogRecord>[];
         var listener = logger.onRecord.listen(logs.add);
         logger.fine('hello');
         logger.warning('world');

test/common/common.dart

@@ -56,7 +56,7 @@ void checkOutputs(Map<String, String> outputs, BuildResult result,
 }
 
 Future<BuildResult> nextResult(results) {
-  var done = new Completer();
+  var done = new Completer<BuildResult>();
   var startingLength = results.length;
   () async {
     while (results.length == startingLength) {

test/common/copy_builder.dart

@@ -44,7 +44,7 @@ class CopyBuilder implements Builder {
   }
 
   List<AssetId> declareOutputs(AssetId input) {
-    var outputs = [];
+    var outputs = <AssetId>[];
     for (int i = 0; i < numCopies; i++) {
       outputs.add(_copiedAssetId(input, numCopies == 1 ? null : i));
     }

test/common/matchers.dart

@@ -58,10 +58,13 @@ class _AssetGraphMatcher extends Matcher {
         return false;
       }
       if (item is GeneratedAssetNode) {
-        if (expectedNode is! GeneratedAssetNode) return false;
-        if (item.primaryInput != expectedNode.primaryInput) return false;
-        if (item.needsUpdate != expectedNode.needsUpdate) return false;
-        if (item.wasOutput != expectedNode.wasOutput) return false;
+        if (expectedNode is GeneratedAssetNode) {
+          if (item.primaryInput != expectedNode.primaryInput) return false;
+          if (item.needsUpdate != expectedNode.needsUpdate) return false;
+          if (item.wasOutput != expectedNode.wasOutput) return false;
+        } else {
+          return false;
+        }
       }
     }
     return true;