Make "--no-packages-dir" the default.

test/downgrade/doesnt_change_git_dependencies_test.dart

Index: test/downgrade/doesnt_change_git_dependencies_test.dart
diff --git a/test/downgrade/doesnt_change_git_dependencies_test.dart b/test/downgrade/doesnt_change_git_dependencies_test.dart
index 0fa29443543294a9a7fc92e62cc651ebe3452394..6bf5f357363247c135e93a994bfe730767ecfc50 100644
--- a/test/downgrade/doesnt_change_git_dependencies_test.dart
+++ b/test/downgrade/doesnt_change_git_dependencies_test.dart
@@ -18,7 +18,7 @@ main() {
       "foo": {"git": "../foo.git"}
     }).create();
 
-    pubGet();
+    pubGet(args: ["--packages-dir"]);

[nweiz, 2016/09/28 20:01:21]

Why not package root here? Same question for the other git tests.

[Bob Nystrom, 2016/09/29 00:21:26]

We don't have an easy way of testing that a path into the Git cache contains
what we want it to. We'll need to support that eventually when we totally remove
support for "packages" directories, but I didn't want to go through all of that
right now for this ostensibly tiny patch.

 
     d.dir(packagesPath, [
       d.dir('foo', [
@@ -31,7 +31,7 @@ main() {
       d.libPubspec('foo', '1.0.0')
     ]).commit();
 
-    pubDowngrade();
+    pubDowngrade(args: ["--packages-dir"]);
 
     d.dir(packagesPath, [
       d.dir('foo', [