Use a lockfile to persist Pub's installed version constellation.

utils/tests/pub/pub_test.dart

Index: utils/tests/pub/pub_test.dart
diff --git a/utils/tests/pub/pub_test.dart b/utils/tests/pub/pub_test.dart
index 2c0915b124b3934b026ce3592885d47afc7f1d9b..af79c5793e186e06f221493866453e9f97dcabb1 100644
--- a/utils/tests/pub/pub_test.dart
+++ b/utils/tests/pub/pub_test.dart
@@ -253,6 +253,7 @@ dependencies:
 
     // TODO(nweiz): remove this once we support pub update
     dir(packagesPath).scheduleDelete();
+    file('$appPath/pubspec.lock', '').scheduleDelete();
 
     git('foo.git', [
       file('foo.dart', 'main() => "foo 2";')
@@ -581,6 +582,293 @@ dependencies:
 
     run();
   });
+
+  test('keeps a Git package locked to the version in the lockfile', () {
+    ensureGit();
+
+    git('foo.git', [
+      file('foo.dart', 'main() => "foo";')
+    ]).scheduleCreate();
+
+    dir(appPath, [
+      file('pubspec.yaml', '''
+dependencies:
+  foo:
+    git: ../foo.git
+''')
+    ]).scheduleCreate();
+
+    // This install should lock the foo.git dependency to the current revision.
+    schedulePub(args: ['install'],
+        output: const RegExp(@"Dependencies installed!$"));
+
+    dir(packagesPath, [
+      dir('foo', [
+        file('foo.dart', 'main() => "foo";')
+      ])
+    ]).scheduleValidate();
+
+    // Delete the packages path to simulate a new checkout of the application.
+    dir(packagesPath).scheduleDelete();
+
+    git('foo.git', [
+      file('foo.dart', 'main() => "foo 2";')
+    ]).scheduleCommit();
+
+    // This install shouldn't update the foo.git dependency due to the lockfile.
+    schedulePub(args: ['install'],
+        output: const RegExp(@"Dependencies installed!$"));
+
+    dir(packagesPath, [
+      dir('foo', [
+        file('foo.dart', 'main() => "foo";')
+      ])
+    ]).scheduleValidate();
+
+    run();
+  });
+
+  test('updates a locked Git package with a new incompatible constraint', () {
+    ensureGit();
+
+    git('foo.git', [
+      file('foo.dart', 'main() => "foo";')
+    ]).scheduleCreate();
+
+    dir(appPath, [
+      file('pubspec.yaml', '''
+dependencies:
+  foo:
+    git: ../foo.git
+''')
+    ]).scheduleCreate();
+
+    schedulePub(args: ['install'],
+        output: const RegExp(@"Dependencies installed!$"));
+
+    dir(packagesPath, [
+      dir('foo', [
+        file('foo.dart', 'main() => "foo";')
+      ])
+    ]).scheduleValidate();
+
+    git('foo.git', [
+      file('foo.dart', 'main() => "foo 1.0.0";'),
+      file('pubspec.yaml', 'version: 1.0.0')
+    ]).scheduleCommit();
+
+    dir(appPath, [
+      file('pubspec.yaml', '''
+dependencies:
+  foo:
+    git: ../foo.git
+    version: ">=1.0.0"
+''')
+    ]).scheduleCreate();
+
+    schedulePub(args: ['install'],
+        output: const RegExp(@"Dependencies installed!$"));
+
+    dir(packagesPath, [
+      dir('foo', [
+        file('foo.dart', 'main() => "foo 1.0.0";')
+      ])
+    ]).scheduleValidate();
+
+    run();
+  });
+
+  test("doesn't update a locked Git package with a new compatible "
+      "constraint", () {
+    ensureGit();
+
+    git('foo.git', [
+      file('foo.dart', 'main() => "foo 1.0.0";'),
+      file('pubspec.yaml', 'version: 1.0.0')
+    ]).scheduleCreate();
+
+    dir(appPath, [
+      file('pubspec.yaml', '''
+dependencies:
+  foo:
+    git: ../foo.git
+''')
+    ]).scheduleCreate();
+
+    schedulePub(args: ['install'],
+        output: const RegExp(@"Dependencies installed!$"));
+
+    dir(packagesPath, [
+      dir('foo', [
+        file('foo.dart', 'main() => "foo 1.0.0";')
+      ])
+    ]).scheduleValidate();
+
+    git('foo.git', [
+      file('foo.dart', 'main() => "foo 1.0.1";'),
+      file('pubspec.yaml', 'version: 1.0.1')
+    ]).scheduleCommit();
+
+    dir(appPath, [
+      file('pubspec.yaml', '''
+dependencies:
+  foo:
+    git: ../foo.git
+    version: ">=1.0.0"
+''')
+    ]).scheduleCreate();
+
+    schedulePub(args: ['install'],
+        output: const RegExp(@"Dependencies installed!$"));
+
+    dir(packagesPath, [
+      dir('foo', [
+        file('foo.dart', 'main() => "foo 1.0.0";')
+      ])
+    ]).scheduleValidate();
+
+    run();
+  });
+
+  test('keeps a pub server package locked to the version in the lockfile', () {
+    var serverFuture =
+      servePackages("localhost", 3123, ['{name: foo, version: 1.0.0}']);
+
+    dir(appPath, [
+      file('pubspec.yaml', '''
+dependencies:
+  foo:
+    repo: {name: foo, url: http://localhost:3123}
+''')
+    ]).scheduleCreate();
+
+    // This install should lock the foo dependency to version 1.0.0.
+    schedulePub(args: ['install'],
+        output: const RegExp(@"Dependencies installed!$"));
+
+    dir(packagesPath, [
+      dir('foo', [
+        file('foo.dart', 'main() => print("foo 1.0.0");')
+      ])
+    ]).scheduleValidate();
+
+    // Delete the packages path to simulate a new checkout of the application.
+    dir(packagesPath).scheduleDelete();
+
+    // Start serving a newer package as well.
+    servePackages("localhost", 3123, [
+      '{name: foo, version: 1.0.0}',
+      '{name: foo, version: 1.0.1}'
+    ]);
+
+    // This install shouldn't update the foo dependency due to the lockfile.
+    schedulePub(args: ['install'],
+        output: const RegExp(@"Dependencies installed!$"));
+
+    dir(packagesPath, [
+      dir('foo', [
+        file('foo.dart', 'main() => print("foo 1.0.0");')
+      ])
+    ]).scheduleValidate();
+
+    run();
+  });
+
+  test('updates a locked pub server package with a new incompatible '
+      'constraint', () {
+    var serverFuture =
+      servePackages("localhost", 3123, ['{name: foo, version: 1.0.0}']);
+
+    dir(appPath, [
+      file('pubspec.yaml', '''
+dependencies:
+  foo:
+    repo: {name: foo, url: http://localhost:3123}
+''')
+    ]).scheduleCreate();
+
+    schedulePub(args: ['install'],
+        output: const RegExp(@"Dependencies installed!$"));
+
+    dir(packagesPath, [
+      dir('foo', [
+        file('foo.dart', 'main() => print("foo 1.0.0");')
+      ])
+    ]).scheduleValidate();
+
+    servePackages("localhost", 3123, [
+      '{name: foo, version: 1.0.0}',
+      '{name: foo, version: 1.0.1}'
+    ]);
+
+    dir(appPath, [
+      file('pubspec.yaml', '''
+dependencies:
+  foo:
+    repo: {name: foo, url: http://localhost:3123}
+    version: ">1.0.0"
+''')
+    ]).scheduleCreate();
+
+    schedulePub(args: ['install'],
+        output: const RegExp(@"Dependencies installed!$"));
+
+    dir(packagesPath, [
+      dir('foo', [
+        file('foo.dart', 'main() => print("foo 1.0.1");')
+      ])
+    ]).scheduleValidate();
+
+    run();
+  });
+
+  test("doesn't update a locked pub server package with a new compatible "
+      "constraint", () {
+    var serverFuture =
+      servePackages("localhost", 3123, ['{name: foo, version: 1.0.0}']);
+
+    dir(appPath, [
+      file('pubspec.yaml', '''
+dependencies:
+  foo:
+    repo: {name: foo, url: http://localhost:3123}
+''')
+    ]).scheduleCreate();
+
+    schedulePub(args: ['install'],
+        output: const RegExp(@"Dependencies installed!$"));
+
+    dir(packagesPath, [
+      dir('foo', [
+        file('foo.dart', 'main() => print("foo 1.0.0");')
+      ])
+    ]).scheduleValidate();
+
+    servePackages("localhost", 3123, [
+      '{name: foo, version: 1.0.0}',
+      '{name: foo, version: 1.0.1}'
+    ]);
+
+    dir(appPath, [
+      file('pubspec.yaml', '''
+dependencies:
+  foo:
+    repo: {name: foo, url: http://localhost:3123}
+    version: ">=1.0.0"
+''')
+    ]).scheduleCreate();
+
+    schedulePub(args: ['install'],
+        output: const RegExp(@"Dependencies installed!$"));
+
+    dir(packagesPath, [
+      dir('foo', [
+        file('foo.dart', 'main() => print("foo 1.0.0");')
+      ])
+    ]).scheduleValidate();
+
+    run();
+  });
 }
 
 versionCommand() {