Get rid of --use-repository-packages and --use-public-packages.

tools/testing/dart/test_suite.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.
 
 /**
  * Classes and methods for enumerating and preparing tests.
  *
  * This library includes:
  *
  * - Creating tests by listing all the Dart files in certain directories,
@@ skipping 368 matching lines @@
   /**
    * Create a directories for generated assets (tests, html files,
    * pubspec checkouts ...).
    */
 
   String createOutputDirectory(Path testPath, String optionsName) {
     var checked = configuration['checked'] ? '-checked' : '';
     var strong =  configuration['strong'] ? '-strong' : '';
     var minified = configuration['minified'] ? '-minified' : '';
     var sdk = configuration['use_sdk'] ? '-sdk' : '';
-    var packages =
-        configuration['use_public_packages'] ? '-public_packages' : '';
     var dirName = "${configuration['compiler']}-${configuration['runtime']}"
-        "$checked$strong$minified$packages$sdk";
+        "$checked$strong$minified$sdk";
     return createGeneratedTestDirectoryHelper(
         "tests", dirName, testPath, optionsName);
   }
 
   String createCompilationOutputDirectory(Path testPath) {
     var checked = configuration['checked'] ? '-checked' : '';
     var strong =  configuration['strong'] ? '-strong' : '';
     var minified = configuration['minified'] ? '-minified' : '';
     var csp = configuration['csp'] ? '-csp' : '';
     var sdk = configuration['use_sdk'] ? '-sdk' : '';
-    var packages =
-        configuration['use_public_packages'] ? '-public_packages' : '';
     var dirName = "${configuration['compiler']}"
-        "$checked$strong$minified$csp$packages$sdk";
+        "$checked$strong$minified$csp$sdk";
     return createGeneratedTestDirectoryHelper(
         "compilations", dirName, testPath, "");
   }
 
   String createPubspecCheckoutDirectory(Path directoryOfPubspecYaml) {
-    var sdk = configuration['use_sdk'] ? '-sdk' : '';
-    var pkg = configuration['use_public_packages']
-        ? 'public_packages'
-        : 'repo_packages';
+    var sdk = configuration['use_sdk'] ? 'sdk' : '';
     return createGeneratedTestDirectoryHelper(
-        "pubspec_checkouts", '$pkg$sdk', directoryOfPubspecYaml, "");
+        "pubspec_checkouts", sdk, directoryOfPubspecYaml, "");
   }
 
   String createPubPackageBuildsDirectory(Path directoryOfPubspecYaml) {
-    var pkg = configuration['use_public_packages']
-        ? 'public_packages'
-        : 'repo_packages';
     return createGeneratedTestDirectoryHelper(
-        "pub_package_builds", pkg, directoryOfPubspecYaml, "");
+        "pub_package_builds", 'public_packages', directoryOfPubspecYaml, "");
   }
 
   /**
    * Helper function for discovering the packages in the dart repository.
    */
   Future<List> listDir(Path path, Function isValid) {
     var dir = new Directory(path.toNativePath());
     return dir.exists().then((var exist) {
       if (!exist) return [];
       return dir
@@ skipping 73 matching lines @@
       for (var result in results) {
         for (var packageTuple in result) {
           String packageName = packageTuple[0];
           String fullPath = packageTuple[1];
           packageDirectories[packageName] = fullPath;
         }
       }
       return packageDirectories;
     });
   }
-
-  /**
-   * Helper function for building dependency_overrides for pubspec.yaml files.
-   */
-  Map buildPubspecDependencyOverrides(Map packageDirectories) {
-    Map overrides = {};
-    packageDirectories.forEach((String packageName, String fullPath) {
-      overrides[packageName] = {'path': fullPath};
-    });
-    return overrides;
-  }
 }
 
 Future<Iterable<String>> ccTestLister(String runnerPath) {
   return Process.run(runnerPath, ["--list"]).then((ProcessResult result) {
     if (result.exitCode != 0) {
       throw "Failed to list tests: '$runnerPath --list'. "
           "Process exited with ${result.exitCode}";
     }
     return result.stdout
         .split('\n')
@@ skipping 349 matching lines @@
 
   void enqueueTestCaseFromTestInformation(TestInformation info) {
     String testName = buildTestCaseDisplayName(suiteDir, info.originTestPath,
         multitestName:
             info.optionsFromFile['isMultitest'] ? info.multitestKey : "");
     Set<Expectation> expectations = testExpectations.expectations(testName);
     if (info is HtmlTestInformation) {
       enqueueHtmlTest(info, testName, expectations);
       return;
     }
-    var filePath = info.filePath;
     var optionsFromFile = info.optionsFromFile;
 
-    Map buildSpecialPackageRoot(Path pubspecYamlFile) {
-      var commands = <Command>[];
-      var packageDir = pubspecYamlFile.directoryPath;
-      var packageName = packageDir.filename;
-
-      var checkoutDirectory = createPubspecCheckoutDirectory(packageDir);
-      var modifiedYamlFile = new Path(checkoutDirectory).append("pubspec.yaml");
-      var pubCacheDirectory = new Path(checkoutDirectory).append("pub-cache");
-      var newPackageRoot = new Path(checkoutDirectory).append("packages");
-
-      // Remove the old packages directory, so we can do a clean 'pub get'.
-      var newPackagesDirectory = new Directory(newPackageRoot.toNativePath());
-      if (newPackagesDirectory.existsSync()) {
-        newPackagesDirectory.deleteSync(recursive: true);
-      }
-
-      // NOTE: We make a link in the package-root to [packageName], since
-      // 'pub get' doesn't create the link to the package containing
-      // pubspec.yaml if there is no lib directory.
-      var packageLink = newPackageRoot.append(packageName);
-      var packageLinkTarget = packageDir.append('lib');
-
-      // NOTE: We make a link in the package-root to pkg/expect, since
-      // 'package:expect' is not available on pub.dartlang.org!
-      var expectLink = newPackageRoot.append('expect');
-      var expectLinkTarget =
-          TestUtils.dartDir.append('pkg').append('expect').append('lib');
-
-      // Generate dependency overrides if we use repository packages.
-      var packageDirectories = {};
-      if (configuration['use_repository_packages']) {
-        packageDirectories = new Map.from(localPackageDirectories);
-
-        // Don't create a dependency override for pub, since it's an application
-        // package and it has a dependency on compiler_unsupported which isn't
-        // in the repo.
-        packageDirectories.remove('pub');
-
-        // Do not create an dependency override for the package itself.
-        if (packageDirectories.containsKey(packageName)) {
-          packageDirectories.remove(packageName);
-        }
-      }
-      var overrides = buildPubspecDependencyOverrides(packageDirectories);
-
-      commands.add(CommandBuilder.instance.getModifyPubspecCommand(
-          pubspecYamlFile.toNativePath(), overrides,
-          destinationFile: modifiedYamlFile.toNativePath()));
-      commands.add(CommandBuilder.instance.getPubCommand(
-          "get", pubPath, checkoutDirectory, pubCacheDirectory.toNativePath()));
-      if (new Directory(packageLinkTarget.toNativePath()).existsSync()) {
-        commands.add(CommandBuilder.instance.getMakeSymlinkCommand(
-            packageLink.toNativePath(), packageLinkTarget.toNativePath()));
-      }
-      commands.add(CommandBuilder.instance.getMakeSymlinkCommand(
-          expectLink.toNativePath(), expectLinkTarget.toNativePath()));
-
-      return {'commands': commands, 'package-root': newPackageRoot,};
-    }
-
     // If this test is inside a package, we will check if there is a
     // pubspec.yaml file and if so, create a custom package root for it.
     List<Command> baseCommands = <Command>[];
     Path packageRoot;
     Path packages;
-    if (configuration['use_repository_packages'] ||
-        configuration['use_public_packages']) {
-      Path pubspecYamlFile = _findPubspecYamlFile(filePath);
-      if (pubspecYamlFile != null) {
-        var result = buildSpecialPackageRoot(pubspecYamlFile);
-        baseCommands.addAll(result['commands']);
-        packageRoot = result['package-root'];
-        if (optionsFromFile['packageRoot'] == null ||
-            optionsFromFile['packageRoot'] == "") {
-          optionsFromFile['packageRoot'] = packageRoot.toNativePath();
-        }
-      }
-    }
+
     if (optionsFromFile['packageRoot'] == null &&
         optionsFromFile['packages'] == null) {
       if (configuration['package_root'] != null) {
         packageRoot = new Path(configuration['package_root']);
         optionsFromFile['packageRoot'] = packageRoot.toNativePath();
       }
       if (configuration['packages'] != null) {
         Path packages = new Path(configuration['packages']);
         optionsFromFile['packages'] = packages.toNativePath();
       }
@@ skipping 1020 matching lines @@
         var absoluteDirectoryPath = new Path(directory);
 
         // Early return if this package is not using pub.
         if (!fileExists(absoluteDirectoryPath.append('pubspec.yaml'))) {
           return;
         }
 
         var directoryPath = absoluteDirectoryPath.relativeTo(TestUtils.dartDir);
         var testName = "$directoryPath";
         var displayName = '$suiteName/$testName';
-        var packageName = directoryPath.filename;
 
-        // Collect necessary paths for pubspec.yaml overrides, pub-cache, ...
         var checkoutDir =
             createPubPackageBuildsDirectory(absoluteDirectoryPath);
         var cacheDir = new Path(checkoutDir).append("pub-cache").toNativePath();
-        var pubspecYamlFile =
-            new Path(checkoutDir).append('pubspec.yaml').toNativePath();
-
-        var packageDirectories = {};
-        if (!configuration['use_public_packages']) {
-          packageDirectories = new Map.from(localPackageDirectories);
-
-          // Don't create a dependency override for pub, since it's an
-          // application package and it has a dependency on compiler_unsupported
-          // which isn't in the repo.
-          packageDirectories.remove('pub');
-
-          if (packageDirectories.containsKey(packageName)) {
-            packageDirectories.remove(packageName);
-          }
-        }
-        var dependencyOverrides =
-            buildPubspecDependencyOverrides(packageDirectories);
 
         // Build all commands
-        var commands = new List<Command>();
-        commands.add(
-            CommandBuilder.instance.getCopyCommand(directory, checkoutDir));
-        commands.add(CommandBuilder.instance
-            .getModifyPubspecCommand(pubspecYamlFile, dependencyOverrides));
-        commands.add(CommandBuilder.instance
-            .getPubCommand("get", pubPath, checkoutDir, cacheDir));
+        var commands = [
+          CommandBuilder.instance.getCopyCommand(directory, checkoutDir),
+          CommandBuilder.instance.getPubCommand(
+              "get", pubPath, checkoutDir, cacheDir)
+        ];
 
         bool containsWebDirectory = dirExists(directoryPath.append('web'));
         bool containsBuildDartFile =
             fileExists(directoryPath.append('build.dart'));
         if (containsBuildDartFile) {
           var dartBinary = new File(dartVmBinaryFileName).absolute.path;
 
           commands.add(CommandBuilder.instance.getProcessCommand(
               "custom_build",
               dartBinary,
@@ skipping 191 matching lines @@
 
   static deleteTempSnapshotDirectory(Map configuration) {
     if (configuration['compiler'] == 'dart2app' ||
         configuration['compiler'] == 'dart2appjit' ||
         configuration['compiler'] == 'precompiler') {
       var checked = configuration['checked'] ? '-checked' : '';
       var strong =  configuration['strong'] ? '-strong' : '';
       var minified = configuration['minified'] ? '-minified' : '';
       var csp = configuration['csp'] ? '-csp' : '';
       var sdk = configuration['use_sdk'] ? '-sdk' : '';
-      var packages =
-          configuration['use_public_packages'] ? '-public_packages' : '';
       var dirName = "${configuration['compiler']}"
-          "$checked$strong$minified$csp$packages$sdk";
+          "$checked$strong$minified$csp$sdk";
       String generatedPath = "${TestUtils.buildDir(configuration)}"
           "/generated_compilations/$dirName";
       TestUtils.deleteDirectory(generatedPath);
     }
   }
 
   static Path debugLogfile() {
     return new Path(".debug.log");
   }
 
@@ skipping 260 matching lines @@
       }
       if (path.length > WINDOWS_SHORTEN_PATH_LIMIT) {
         ++shortNameCounter;
         var pathEnd = path.substring(path.length - WINDOWS_PATH_END_LENGTH);
         path = "short${shortNameCounter}_$pathEnd";
       }
     }
     return path;
   }
 }