fix: lockfile convert option (#466)

> pick #465
cnpm · Sep 20, 2023 · dee32a1 · dee32a1
1 parent 6442f66
commit dee32a1
Show file tree

Hide file tree

Showing 4 changed files with 184 additions and 16 deletions.
diff --git a/bin/install.js b/bin/install.js
@@ -362,7 +362,9 @@ debug('argv: %j, env: %j', argv, env);
   if (lockfilePath) {
     try {
       const lockfileData = await fs.readFile(lockfilePath, 'utf8');
-      config.dependenciesTree = lockfileConverter(JSON.parse(lockfileData));
+      config.dependenciesTree = lockfileConverter(JSON.parse(lockfileData), {
+        ignoreOptionalDependencies: true,
+      });
     } catch (error) {
       console.warn(chalk.yellow('npminstall WARN load lockfile from %s error :%s'), lockfilePath, error.message);
     }

diff --git a/test/fixtures/lockfile/package-lock.json b/test/fixtures/lockfile/package-lock.json
diff --git a/test/fixtures/lockfile/package.json b/test/fixtures/lockfile/package.json
@@ -4,6 +4,8 @@
     "lodash.get": "3",
     "lodash._baseget": "3.7.1",
     "lodash._baseslice": "^3.0.1",
-    "lodash.has3": "npm:lodash.has@3"
-  }
-}
+    "lodash.has3": "npm:lodash.has@3",
+    "chokidar": "^3"
+  },
+  "repository": "git@github.com:cnpm/npminstall.git"
+}
diff --git a/test/install-with-lockfile.test.js b/test/install-with-lockfile.test.js
@@ -26,6 +26,7 @@ describe('test/install-with-lockfile.test.js', () => {
         ], { cwd })
         .debug()
         .expect('code', 0)
+        .notExpect('stdout', 'TypeError: Cannot read properties of undefined (reading \'ignoreOptionalDependencies\')')
         .end();
       assert.strictEqual(
         await fs.readlink(path.join(cwd, 'node_modules', 'lodash.has3'), 'utf8'),
@@ -42,7 +43,7 @@ describe('test/install-with-lockfile.test.js', () => {
         ignoreOptionalDependencies: true,
       }, nested);
 
-      assert.strictEqual(Object.keys(dependenciesTree).length, 12);
+      assert.strictEqual(Object.keys(dependenciesTree).length, 29);
     });
   }
 });