Avoid redundant calls to "pub list" when multiple dependencies change.

pkg/analysis_server/test/mocks.dart

 // Copyright (c) 2014, 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 mocks;
 
 @MirrorsUsed(targets: 'mocks', override: '*')
 import 'dart:mirrors';
 import 'dart:async';
 import 'dart:io';
 
 import 'package:analysis_server/src/analysis_server.dart';
 import 'package:analysis_server/src/channel/channel.dart';
 import 'package:analysis_server/src/operation/operation.dart';
 import 'package:analysis_server/src/operation/operation_analysis.dart';
 import 'package:analysis_server/src/protocol.dart' hide Element, ElementKind;
+import 'package:analysis_server/src/source/optimizing_pub_package_map_provider.dart';
 import 'package:analyzer/file_system/file_system.dart' as resource;
 import 'package:analyzer/file_system/memory_file_system.dart' as resource;
 import 'package:analyzer/source/package_map_provider.dart';
 import 'package:analyzer/src/generated/element.dart';
 import 'package:analyzer/src/generated/engine.dart';
 import 'package:analyzer/src/generated/source.dart';
 import 'package:matcher/matcher.dart';
 import 'package:typed_mock/typed_mock.dart';
 import 'package:unittest/unittest.dart';
 
@@ skipping 118 matching lines @@
 
 class MockMethodElement extends StringTypedMock implements MethodElement {
   final kind = ElementKind.METHOD;
   MockMethodElement([String name = 'method']) : super(name);
   noSuchMethod(Invocation invocation) => super.noSuchMethod(invocation);
 }
 
 /**
  * A mock [PackageMapProvider].
  */
-class MockPackageMapProvider implements PackageMapProvider {
+class MockPackageMapProvider implements OptimizingPubPackageMapProvider {
   /**
    * Package map that will be returned by the next call to [computePackageMap].
    */
   Map<String, List<resource.Folder>> packageMap =
       <String, List<resource.Folder>>{};
 
   /**
    * Package maps that will be returned by the next call to [computePackageMap].
    */
   Map<String, Map<String, List<resource.Folder>>> packageMaps = null;
 
   /**
    * Dependency list that will be returned by the next call to [computePackageMap].
    */
   Set<String> dependencies = new Set<String>();
 
+  /**
+   * Modification times that will be returned by the next call to
+   * [computePackageMap].  This will be filtered to include only those paths
+   * mentioned in the `dependencies` field of [computePackageMap]'s
+   * `previousInfo` argument.
+   */
+  Map<String, int> modificationTimes = <String, int>{};
+
+  /**
+   * Number of times [computePackageMap] has been called.
+   */
+  int computeCount = 0;
+
   @override
-  PackageMapInfo computePackageMap(resource.Folder folder) {
+  OptimizingPubPackageMapInfo computePackageMap(resource.Folder folder,
+      [OptimizingPubPackageMapInfo previousInfo]) {
+    ++computeCount;
+    Map<String, int> filteredModificationTimes = <String, int>{};
+    if (previousInfo != null) {
+      for (String dependency in previousInfo.dependencies) {
+        if (modificationTimes.containsKey(dependency)) {
+          filteredModificationTimes[dependency] = modificationTimes[dependency];
+        }
+      }
+    }
     if (packageMaps != null) {
-      return new PackageMapInfo(packageMaps[folder.path], dependencies);
+      return new OptimizingPubPackageMapInfo(
+          packageMaps[folder.path], dependencies, filteredModificationTimes);
     }
-    return new PackageMapInfo(packageMap, dependencies);
+    return new OptimizingPubPackageMapInfo(
+        packageMap, dependencies, filteredModificationTimes);
+  }
+
+  noSuchMethod(Invocation invocation) {
+    // No other methods should be called.
+    return super.noSuchMethod(invocation);
   }
 }
 
 class MockParameterElement extends TypedMock implements ParameterElement {
   final ElementKind kind = ElementKind.PARAMETER;
   noSuchMethod(Invocation invocation) => super.noSuchMethod(invocation);
 }
 
 class MockPropertyAccessorElement extends TypedMock
     implements PropertyAccessorElement {
@@ skipping 262 matching lines @@
     }
     return mismatchDescription;
   }
 
   @override
   bool matches(item, Map matchState) {
     Response response = item;
     return response != null && response.id == _id && response.error == null;
   }
 }