Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-10-26 08:36:12 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: remove extraneous devDependencies on package merge

Browse Source 
/cc @pitaj
This commit is contained in:
Julian Lam
2022-02-08 11:13:51 -05:00
parent 9e4147f07c
commit a2c7d69e1b
2 changed files with 12 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/cli/package-install.js
View File

@@ -38,11 +38,12 @@ pkgInstall.updatePackageFile = () => {
}
});
const { devDependencies } = defaultPackageContents;
// Sort dependencies alphabetically
dependencies = sortDependencies({ ...dependencies, ...defaultPackageContents.dependencies });
const packageContents = { ..._.merge(oldPackageContents, defaultPackageContents), dependencies };
const packageContents = { ..._.merge(oldPackageContents, defaultPackageContents), dependencies, devDependencies };
fs.writeFileSync(paths.currentPackage, JSON.stringify(packageContents, null, 2));
};

9
test/package-install.js
View File

@@ -86,6 +86,15 @@ describe('Package install lib', () => {
assert.strictEqual(updated.scripts.preinstall, 'echo "What are you?";');
});
it('should remove extraneous devDependencies', async () => {
current.devDependencies.expect = '27.5.1';
await fs.writeFile(packageFilePath, JSON.stringify(current, null, 4));
pkgInstall.updatePackageFile();
const updated = JSON.parse(await fs.readFile(packageFilePath, 'utf8'));
assert.strictEqual(updated.devDependencies.hasOwnProperty('expect'), false);
});
after(async () => {
// Clean up
await fs.rename(path.resolve(__dirname, '../install/package.json.bak'), sourcePackagePath);