upgrade scenario with deeper dependencies

Author: bestander

test/commands/install.js

@@ -15,6 +15,8 @@ import { run as uninstall } from "../../src/cli/commands/uninstall.js";
 import Config from "../../src/config.js";
 import * as fs from "../../src/util/fs.js";
 import assert from "assert";
+import semver from "semver";
+
 
 let test = require("ava");
 let path = require("path");
@@ -335,6 +337,60 @@ test("upgrade scenario", () => {
   }, clean);
 });
 
+test.only("upgrade scenario 2 (with sub dependencies)", async () => {
+  // mime-types@2.0.0 is saved in local mirror and gets updated to mime-types@2.1.11
+  // files in mirror, fbkpm.lock, package.json and node_modules should reflect that
+
+  let mirrorPath = "mirror-for-offline";
+  let fixture = "install-upgrade-scenario-2";
+  let cwd = path.join(fixturesLoc, fixture);
+  await fs.copy(path.join(cwd, "fbkpm.lock.before"), path.join(cwd, "fbkpm.lock"));
+  await fs.copy(path.join(cwd, "package.json.before"), path.join(cwd, "package.json"));
+
+  return run({}, [], fixture, async (config) => {
+    assert(semver.satisfies(
+      JSON.parse(await fs.readFile(path.join(config.cwd, "node_modules/mime-db/package.json"))).version,
+      "~1.0.1")
+    );
+    assert.equal(
+      JSON.parse(await fs.readFile(path.join(config.cwd, "node_modules/mime-types/package.json"))).version,
+      "2.0.0"
+    );
+
+    return run({save: true}, ["mime-types@2.1.11"], fixture, async (config) => {
+      assert(semver.satisfies(
+        JSON.parse(await fs.readFile(path.join(config.cwd, "node_modules/mime-db/package.json"))).version,
+        "~1.23.0"
+      ));
+      assert.equal(
+        JSON.parse(await fs.readFile(path.join(config.cwd, "node_modules/mime-types/package.json"))).version,
+        "2.1.11"
+      );
+
+      let lockFileWritten = await fs.readFile(path.join(config.cwd, "fbkpm.lock"));
+      let lockFileLines = lockFileWritten.split("\n").filter((line) => !!line);
+      assert.equal(lockFileLines[0], "mime-db@~1.23.0:");
+      assert.notEqual(lockFileLines[3].indexOf("resolved mime-db-"), -1);
+      assert.equal(lockFileLines[4], "mime-types@2.1.11:");
+      assert.notEqual(lockFileLines[7].indexOf("resolved mime-types-2.1.11.tgz"), -1);
+
+      let mirror = await fs.walk(path.join(config.cwd, mirrorPath));
+      assert.equal(mirror.length, 4);
+      let newFilesInMirror = mirror.filter((elem) => {
+        return elem.relative !== "mime-db-1.0.3.tgz" && elem.relative !== "mime-types-2.0.0.tgz";
+      });
+
+      assert.equal(newFilesInMirror.length, 2);
+
+      await fs.unlink(newFilesInMirror[0].absolute);
+      await fs.unlink(newFilesInMirror[1].absolute);
+
+      await fs.unlink(path.join(config.cwd, "fbkpm.lock"));
+      await fs.unlink(path.join(config.cwd, "package.json"));
+    });
+  });
+});
+
 test("downgrade scenario", () => {
   // left-pad first installed 1.1.0 then downgraded to 0.0.9
   // files in mirror, fbkpm.lock, package.json and node_modules should reflect that

test/fixtures/install/install-upgrade-scenario-2/.npmrc

@@ -0,0 +1 @@
+kpm-offline-mirror=./mirror-for-offline

test/fixtures/install/install-upgrade-scenario-2/fbkpm.lock.before

@@ -0,0 +1,10 @@
+mime-db@~1.0.1:
+  name mime-db
+  version "1.0.3"
+  resolved mime-db-1.0.3.tgz#5680b12300b3cecea5620f20f269ee80f2632d81
+mime-types@2.0.0:
+  name mime-types
+  version "2.0.0"
+  resolved mime-types-2.0.0.tgz#4a85688446a4d94a03909e0ae292766744a3c313
+  dependencies:
+    mime-db "~1.0.1"

test/fixtures/install/install-upgrade-scenario-2/mirror-for-offline/mime-db-1.0.3.tgz

@@ -0,0 +1,5 @@
+{
+  "dependencies": {
+    "mime-types": "2.0.0"
+  }
+}