Run the sorter to reduce code churn

pkg/analyzer/test/src/summary/package_bundle_reader_test.dart

 // Copyright (c) 2016, 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.
 
 import 'package:analyzer/src/context/cache.dart';
 import 'package:analyzer/src/generated/engine.dart';
 import 'package:analyzer/src/generated/source.dart';
 import 'package:analyzer/src/summary/idl.dart';
 import 'package:analyzer/src/summary/package_bundle_reader.dart';
 import 'package:analyzer/src/task/dart.dart';
 import 'package:analyzer/task/dart.dart';
 import 'package:analyzer/task/general.dart';
 import 'package:test/test.dart';
 import 'package:test_reflective_loader/test_reflective_loader.dart';
 import 'package:typed_mock/typed_mock.dart';
 
 main() {
   defineReflectiveSuite(() {
     defineReflectiveTests(ResynthesizerResultProviderTest);
     defineReflectiveTests(SummaryDataStoreTest);
   });
 }
 
+/// A matcher for ConflictingSummaryException.
+const Matcher isConflictingSummaryException =
+    const _ConflictingSummaryException();
+
 UnlinkedPublicNamespace _namespaceWithParts(List<String> parts) {
   UnlinkedPublicNamespace namespace = new _UnlinkedPublicNamespaceMock();
   when(namespace.parts).thenReturn(parts);
   return namespace;
 }
 
 @reflectiveTest
 class ResynthesizerResultProviderTest {
   SourceFactory sourceFactory = new _SourceFactoryMock();
   InternalAnalysisContext context = new _InternalAnalysisContextMock();
@@ skipping 118 matching lines @@
       recordDependencyInfo: true, disallowOverlappingSummaries: true);
 
   PackageBundle bundle1 = new _PackageBundleMock();
   PackageBundle bundle2 = new _PackageBundleMock();
   UnlinkedUnit unlinkedUnit11 = new _UnlinkedUnitMock();
   UnlinkedUnit unlinkedUnit12 = new _UnlinkedUnitMock();
   UnlinkedUnit unlinkedUnit21 = new _UnlinkedUnitMock();
   LinkedLibrary linkedLibrary1 = new _LinkedLibraryMock();
   LinkedLibrary linkedLibrary2 = new _LinkedLibraryMock();
 
-  void _setupDataStore(SummaryDataStore store) {
-    // bundle1
-    when(unlinkedUnit11.publicNamespace)
-        .thenReturn(_namespaceWithParts(['package:p1/u2.dart']));
-    when(unlinkedUnit12.publicNamespace).thenReturn(_namespaceWithParts([]));
-    when(bundle1.unlinkedUnitUris)
-        .thenReturn(<String>['package:p1/u1.dart', 'package:p1/u2.dart']);
-    when(bundle1.unlinkedUnits)
-        .thenReturn(<UnlinkedUnit>[unlinkedUnit11, unlinkedUnit12]);
-    when(bundle1.linkedLibraryUris).thenReturn(<String>['package:p1/u1.dart']);
-    when(bundle1.linkedLibraries).thenReturn(<LinkedLibrary>[linkedLibrary1]);
-    when(bundle1.apiSignature).thenReturn('signature1');
-    store.addBundle('/p1.ds', bundle1);
-    // bundle2
-    when(unlinkedUnit21.publicNamespace).thenReturn(_namespaceWithParts([]));
-    when(bundle2.unlinkedUnitUris).thenReturn(<String>['package:p2/u1.dart']);
-    when(bundle2.unlinkedUnits).thenReturn(<UnlinkedUnit>[unlinkedUnit21]);
-    when(bundle2.linkedLibraryUris).thenReturn(<String>['package:p2/u1.dart']);
-    when(bundle2.linkedLibraries).thenReturn(<LinkedLibrary>[linkedLibrary2]);
-    when(bundle2.apiSignature).thenReturn('signature2');
-    store.addBundle('/p2.ds', bundle2);
-  }
-
   void setUp() {
     _setupDataStore(dataStore);
   }
 
   test_addBundle() {
     expect(dataStore.bundles, unorderedEquals([bundle1, bundle2]));
     expect(dataStore.dependencies[0].summaryPath, '/p1.ds');
     expect(dataStore.dependencies[0].apiSignature, 'signature1');
     expect(dataStore.dependencies[0].includedPackageNames, ['p1']);
     expect(dataStore.dependencies[0].includesFileUris, false);
@@ skipping 94 matching lines @@
     String partUri = 'package:p1/u2.dart';
     List<String> uris = dataStore.getContainingLibraryUris(partUri);
     expect(uris, unorderedEquals(['package:p1/u1.dart']));
   }
 
   test_getContainingLibraryUris_unknownUri() {
     String partUri = 'package:notInStore/foo.dart';
     List<String> uris = dataStore.getContainingLibraryUris(partUri);
     expect(uris, isNull);
   }
+
+  void _setupDataStore(SummaryDataStore store) {
+    // bundle1
+    when(unlinkedUnit11.publicNamespace)
+        .thenReturn(_namespaceWithParts(['package:p1/u2.dart']));
+    when(unlinkedUnit12.publicNamespace).thenReturn(_namespaceWithParts([]));
+    when(bundle1.unlinkedUnitUris)
+        .thenReturn(<String>['package:p1/u1.dart', 'package:p1/u2.dart']);
+    when(bundle1.unlinkedUnits)
+        .thenReturn(<UnlinkedUnit>[unlinkedUnit11, unlinkedUnit12]);
+    when(bundle1.linkedLibraryUris).thenReturn(<String>['package:p1/u1.dart']);
+    when(bundle1.linkedLibraries).thenReturn(<LinkedLibrary>[linkedLibrary1]);
+    when(bundle1.apiSignature).thenReturn('signature1');
+    store.addBundle('/p1.ds', bundle1);
+    // bundle2
+    when(unlinkedUnit21.publicNamespace).thenReturn(_namespaceWithParts([]));
+    when(bundle2.unlinkedUnitUris).thenReturn(<String>['package:p2/u1.dart']);
+    when(bundle2.unlinkedUnits).thenReturn(<UnlinkedUnit>[unlinkedUnit21]);
+    when(bundle2.linkedLibraryUris).thenReturn(<String>['package:p2/u1.dart']);
+    when(bundle2.linkedLibraries).thenReturn(<LinkedLibrary>[linkedLibrary2]);
+    when(bundle2.apiSignature).thenReturn('signature2');
+    store.addBundle('/p2.ds', bundle2);
+  }
+}
+
+class _ConflictingSummaryException extends TypeMatcher {
+  const _ConflictingSummaryException() : super("ConflictingSummaryException");
+  bool matches(item, Map matchState) => item is ConflictingSummaryException;
 }
 
 class _InternalAnalysisContextMock extends TypedMock
     implements InternalAnalysisContext {}
 
 class _LinkedLibraryMock extends TypedMock implements LinkedLibrary {}
 
 class _PackageBundleMock extends TypedMock implements PackageBundle {}
 
 class _SourceFactoryMock extends TypedMock implements SourceFactory {}
@@ skipping 26 matching lines @@
   @override
   bool hasResultsForSource(Source source) {
     return sourcesWithResults.contains(source);
   }
 }
 
 class _UnlinkedPublicNamespaceMock extends TypedMock
     implements UnlinkedPublicNamespace {}
 
 class _UnlinkedUnitMock extends TypedMock implements UnlinkedUnit {}
-
-/// A matcher for ConflictingSummaryException.
-const Matcher isConflictingSummaryException =
-    const _ConflictingSummaryException();
-
-class _ConflictingSummaryException extends TypeMatcher {
-  const _ConflictingSummaryException() : super("ConflictingSummaryException");
-  bool matches(item, Map matchState) => item is ConflictingSummaryException;
-}