feat: fix tests

Signed-off-by: Ruben Romero Montes <[email protected]>

Author: ruromero

package-lock.json

@@ -1,6 +1,6 @@
 import fs from 'node:fs'
 import os from "node:os";
-import { handleSpacesInPath, invokeCommand } from "../tools.js";
+import { invokeCommand } from "../tools.js";
 import path from 'node:path'
 import Sbom from '../sbom.js'
 import { PackageURL } from 'packageurl-js'
@@ -171,7 +171,7 @@ export default class Base_javascript {
 	#executeListCmd(includeTransitive, manifestDir) {
 		const listArgs = this._listCmdArgs(includeTransitive, manifestDir);
 		try {
-			invokeCommand(this._cmdName(), listArgs)
+			return invokeCommand(this._cmdName(), listArgs)
 		} catch (error) {
 			throw new Error(`failed to list dependencies via "${this._cmdName()} ${listArgs.join(' ')}" - Error: ${error}`, {cause: error});
 		}
@@ -195,9 +195,8 @@ export default class Base_javascript {
 		if (os.platform() === 'win32') {
 			process.chdir(manifestDir)
 		}
-
+		const args = this._updateLockFileCmdArgs(manifestDir);
 		try {
-			const args = this._updateLockFileCmdArgs(manifestDir);
 			invokeCommand(this._cmdName(), args)
 		} catch (error) {
 			throw new Error(`failed to create lockfile "${args}" - Error: ${error}`, {cause: error});

src/providers/base_javascript.js

@@ -11,17 +11,17 @@ export default class Javascript_pnpm extends Base_javascript {
 	}
 
 	_listCmdArgs(includeTransitive, manifestDir) {
-		const args = ['ls', includeTransitive ? '--all' : '--depth=0', '--prod', '--json'];
+		const args = ['ls', includeTransitive ? '--depth=Infinity' : '--depth=0', '--prod', '--json'];
 		if (manifestDir) {
-			args.push('--prefix', manifestDir);
+			args.push('--dir', manifestDir);
 		}
 		return args;
 	}
 
 	_updateLockFileCmdArgs(manifestDir) {
 		const args = ['install', '--frozen-lockfile'];
 		if (manifestDir) {
-			args.push('--prefix', manifestDir)
+			args.push('--dir', manifestDir)
 		}
 		args.push(...[])
 		return args;

src/providers/javascript_pnpm.js

@@ -9,15 +9,15 @@ let clock
 
 async function mockProvider(providerName, listingOutput) {
 
-	const mockExecSync = (cmd) => {
-		if (cmd.includes('--version')) { return ''; }
+	const mockInvokeCommand = (_cmd, args) => {
+		if (args.includes('--version')) {return '';}
 		return listingOutput;
-	}
+	};
 
-	return esmock(`../../src/providers/Javascript_${providerName}.js`, {
-		'../../src/providers/Base_javascript.js': await esmock('../../src/providers/base_javascript.js', {
-			'node:child_process': {
-				execSync: mockExecSync
+	return esmock(`../../src/providers/javascript_${providerName}.js`, {
+		'../../src/providers/base_javascript.js': await esmock('../../src/providers/base_javascript.js', {
+			'../../src/tools.js': {
+				invokeCommand: mockInvokeCommand
 			}
 		})
 	});