Use PUB_HOSTED_SERVER to set server URL for pub tests.

sdk/lib/_internal/pub/test/test_pub.dart

 // Copyright (c) 2013, 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.
 
 /// Test infrastructure for testing pub. Unlike typical unit tests, most pub
 /// tests are integration tests that stage some stuff on the file system, run
 /// pub, and then validate the results. This library provides an API to build
 /// tests like that.
 library test_pub;
 
@@ skipping 73 matching lines @@
   });
 }
 
 /// Creates an HTTP server to serve [contents] as static files. This server will
 /// exist only for the duration of the pub run.
 ///
 /// Subsequent calls to [serve] will replace the previous server.
 void serve([List<d.Descriptor> contents]) {
   var baseDir = d.dir("serve-dir", contents);
 
+  _hasServer = true;
+
   schedule(() {
     return _closeServer().then((_) {
       return SafeHttpServer.bind("localhost", 0).then((server) {
         _server = server;
         server.listen((request) {
           currentSchedule.heartbeat();
           var response = request.response;
           try {
             var path = request.uri.path.replaceFirst("/", "");
             _requestedPaths.add(path);
@@ skipping 33 matching lines @@
   if (_server == null) return new Future.value();
   _server.close();
   _server = null;
   _portCompleterCache = null;
   // TODO(nweiz): Remove this once issue 4155 is fixed. Pumping the event loop
   // *seems* to be enough to ensure that the server is actually closed, but I'm
   // putting this at 10ms to be safe.
   return sleep(10);
 }
 
+/// `true` if the current test spins up an HTTP server.
+bool _hasServer = false;
+
 /// The [d.DirectoryDescriptor] describing the server layout of `/api/packages`
 /// on the test server.
 ///
 /// This contains metadata for packages that are being served via
 /// [servePackages]. It's `null` if [servePackages] has not yet been called for
 /// this test.
 d.DirectoryDescriptor _servedApiPackageDir;
 
 /// The [d.DirectoryDescriptor] describing the server layout of `/packages` on
 /// the test server.
@@ skipping 294 matching lines @@
   if (tokenEndpoint == null) tokenEndpoint = new Future.value();
   var environmentFuture = tokenEndpoint.then((tokenEndpoint) {
     var environment = {};
     environment['_PUB_TESTING'] = 'true';
     environment['PUB_CACHE'] = pathInSandbox(cachePath);
     environment['DART_SDK'] = pathInSandbox(sdkPath);
     if (tokenEndpoint != null) {
       environment['_PUB_TEST_TOKEN_ENDPOINT'] =
         tokenEndpoint.toString();
     }
+
+    // If there is a server running, tell pub what its URL is so hosted
+    // dependencies will look there.
+    if (_hasServer) {
+      return port.then((p) {
+        environment['PUB_HOSTED_URL'] = "http://localhost:$p";
+        return environment;
+      });
+    }
+
     return environment;
   });
 
   return new PubProcess.start(dartBin, dartArgs, environment: environmentFuture,
       workingDirectory: pathInSandbox(appPath),
       description: args.isEmpty ? 'pub' : 'pub ${args.first}');
 }
 
 /// A subclass of [ScheduledProcess] that parses pub's verbose logging output
 /// and makes [nextLine], [nextErrLine], [remainingStdout], and
@@ skipping 131 matching lines @@
   };
   if (dependencies != null) {
     package["dependencies"] = dependencyListToMap(dependencies);
   }
   return package;
 }
 
 /// Describes a map representing a dependency on a package in the package
 /// repository.
 Map dependencyMap(String name, [String versionConstraint]) {
-  var url = port.then((p) => "http://localhost:$p");
-  var dependency = {"hosted": {"name": name, "url": url}};
+  var dependency = {"hosted": name};
   if (versionConstraint != null) dependency["version"] = versionConstraint;
   return dependency;
 }
 
 /// Converts a list of dependencies as passed to [package] into a hash as used
 /// in a pubspec.
 Future<Map> dependencyListToMap(List<Map> dependencies) {
   return awaitObject(dependencies).then((resolvedDependencies) {
     var result = <String, Map>{};
     for (var dependency in resolvedDependencies) {
@@ skipping 167 matching lines @@
   bool matches(item, Map matchState) {
     if (item is! Pair) return false;
     return _firstMatcher.matches(item.first, matchState) &&
         _lastMatcher.matches(item.last, matchState);
   }
 
   Description describe(Description description) {
     description.addAll("(", ", ", ")", [_firstMatcher, _lastMatcher]);
   }
 }