Track Source objects in PackageNames.

test/pubspec_test.dart

 // Copyright (c) 2012, 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:pub/src/package.dart';
 import 'package:pub/src/pubspec.dart';
 import 'package:pub/src/source.dart';
 import 'package:pub/src/source/path.dart';
 import 'package:pub/src/source_registry.dart';
 import 'package:pub/src/system_cache.dart';
 import 'package:pub_semver/pub_semver.dart';
 import 'package:test/test.dart';
 
 class MockSource extends Source {
   final String name = "mock";
 
   BoundSource bind(SystemCache cache) =>
       throw new UnsupportedError("Cannot download mock packages.");
 
   PackageRef parseRef(String name, description, {String containingPath}) {
     if (description != 'ok') throw new FormatException('Bad');
-    return new PackageRef(name, this.name, description);
+    return new PackageRef(name, this, description);
   }
 
   PackageId parseId(String name, Version version, description) =>
-      new PackageId(name, this.name, version, description);
+      new PackageId(name, this, version, description);
 
   bool descriptionsEqual(description1, description2) =>
       description1 == description2;
 
+  int hashDescription(description) => description.hashCode;
+
   String packageName(description) => 'foo';
 }
 
 main() {
   group('parse()', () {
     var sources = new SourceRegistry();
     sources.register(new MockSource());
 
     var throwsPubspecException =
         throwsA(new isInstanceOf<PubspecException>());
@@ skipping 99 matching lines @@
 
     test("allows an unknown source", () {
       var pubspec = new Pubspec.parse('''
 dependencies:
   foo:
     unknown: blah
 ''', sources);
 
       var foo = pubspec.dependencies[0];
       expect(foo.name, equals('foo'));
-      expect(foo.source, equals('unknown'));
+      expect(foo.source, equals(sources['unknown']));
     });
 
     test("throws if a package is in dependencies and dev_dependencies", () {
       expectPubspecException('''
 dependencies:
   foo:
     mock: ok
 dev_dependencies:
   foo:
     mock: ok
@@ skipping 359 matching lines @@
       test("uses the key if the value is null", () {
         var pubspec = new Pubspec.parse('''
 executables:
   command:
 ''', sources);
         expect(pubspec.executables['command'], equals('command'));
       });
     });
   });
 }