fix: handle package managers with a bin array correctly (#20)

* test: fix tests on older versions of node

* test: fix tests on Windows

* fix: handle package managers with a bin array correctly

Author: merceyz

.pnp.js

@@ -6,5 +6,6 @@ module.exports = {
     [`@babel/plugin-proposal-decorators`, {legacy: true}],
     [`@babel/plugin-proposal-class-properties`, {loose: true}],
     [`@babel/plugin-transform-modules-commonjs`],
+    [`@babel/plugin-proposal-nullish-coalescing-operator`],
   ],
 };

.yarn/cache/@babel-plugin-proposal-nullish-coalescing-operator-npm-7.10.4-dd7a5fdb9c-5a20d8bcbf.zip

@@ -13,6 +13,7 @@
     "@babel/core": "^7.11.0",
     "@babel/plugin-proposal-class-properties": "^7.10.4",
     "@babel/plugin-proposal-decorators": "^7.10.5",
+    "@babel/plugin-proposal-nullish-coalescing-operator": "^7.10.4",
     "@babel/plugin-transform-modules-commonjs": "^7.8.3",
     "@babel/preset-typescript": "^7.10.4",
     "@types/debug": "^4.1.5",

.yarn/cache/@babel-plugin-syntax-nullish-coalescing-operator-npm-7.8.3-8a723173b5-4ba0375375.zip

@@ -130,7 +130,13 @@ export async function installVersion(installTarget: string, locator: Locator, {s
 export async function runVersion(installSpec: { location: string, spec: PackageManagerSpec }, locator: Locator, binName: string, args: Array<string>, context: Context) {
   let binPath: string | null = null;
   if (Array.isArray(installSpec.spec.bin)) {
-    binPath = path.join(installSpec.location, `${binName}.js`);
+    if (installSpec.spec.bin.some(bin => bin === binName)) {
+      const parsedUrl = new URL(installSpec.spec.url);
+      const ext = path.posix.extname(parsedUrl.pathname);
+      if (ext === `.js`) {
+        binPath = path.join(installSpec.location, path.posix.basename(parsedUrl.pathname));
+      }
+    }
   } else {
     for (const [name, dest] of Object.entries(installSpec.spec.bin)) {
       if (name === binName) {
@@ -141,7 +147,7 @@ export async function runVersion(installSpec: { location: string, spec: PackageM
   }
 
   if (!binPath)
-    throw new Error(`Assertion failed: Unable to locate bin path`);
+    throw new Error(`Assertion failed: Unable to locate path for bin '${binName}'`);
 
   return new Promise<number>((resolve, reject) => {
     process.on(`SIGINT`, () => {

babel.config.js

@@ -9,7 +9,7 @@ import {runCli}                                    from './_runCli';
 const engine = new Engine();
 
 beforeEach(async () => {
-  process.env.COREPACK_HOME = await xfs.mktempPromise();
+  process.env.COREPACK_HOME = npath.fromPortablePath(await xfs.mktempPromise());
 });
 
 async function makeBin(cwd: PortablePath, name: Filename) {

package.json

@@ -9,7 +9,7 @@ import {runCli}                                               from './_runCli';
 const engine = new Engine();
 
 beforeEach(async () => {
-  process.env.COREPACK_HOME = await xfs.mktempPromise();
+  process.env.COREPACK_HOME = npath.fromPortablePath(await xfs.mktempPromise());
 });
 
 async function makeBin(cwd: PortablePath, name: Filename) {

sources/pmmUtils.ts

@@ -1,11 +1,11 @@
-import {Filename, ppath, xfs} from '@yarnpkg/fslib';
+import {Filename, ppath, xfs, npath} from '@yarnpkg/fslib';
 
-import config                 from '../config.json';
+import config                        from '../config.json';
 
-import {runCli}               from './_runCli';
+import {runCli}                      from './_runCli';
 
 beforeEach(async () => {
-  process.env.COREPACK_HOME = await xfs.mktempPromise();
+  process.env.COREPACK_HOME = npath.fromPortablePath(await xfs.mktempPromise());
 });
 
 for (const [name, version] of [[`yarn`, `1.22.4`], [`yarn`, `2.0.0-rc.30`], [`pnpm`, `4.11.6`], [`npm`, `6.14.2`]]) {
@@ -169,7 +169,7 @@ it(`should support hydrating package managers from cached archives`, async () =>
     });
 
     // Use a new cache
-    process.env.COREPACK_HOME = await xfs.mktempPromise();
+    process.env.COREPACK_HOME = npath.fromPortablePath(await xfs.mktempPromise());
 
     // Disable the network to make sure we don't succeed by accident
     process.env.COREPACK_ENABLE_NETWORK = `0`;
@@ -193,3 +193,21 @@ it(`should support hydrating package managers from cached archives`, async () =>
     }
   });
 });
+
+it(`should support running package managers with bin array`, async () => {
+  await xfs.mktempPromise(async cwd => {
+    await xfs.writeJsonPromise(ppath.join(cwd, `package.json` as Filename), {
+      packageManager: `yarn@2.2.2`,
+    });
+
+    await expect(runCli(cwd, [`yarn`, `yarnpkg`, `--version`])).resolves.toMatchObject({
+      stdout: `2.2.2\n`,
+      exitCode: 0,
+    });
+
+    await expect(runCli(cwd, [`yarn`, `yarn`, `--version`])).resolves.toMatchObject({
+      stdout: `2.2.2\n`,
+      exitCode: 0,
+    });
+  });
+});

tests/Disable.test.ts

@@ -403,6 +403,18 @@ __metadata:
   languageName: node
   linkType: hard
 
+"@babel/plugin-proposal-nullish-coalescing-operator@npm:^7.10.4":
+  version: 7.10.4
+  resolution: "@babel/plugin-proposal-nullish-coalescing-operator@npm:7.10.4"
+  dependencies:
+    "@babel/helper-plugin-utils": ^7.10.4
+    "@babel/plugin-syntax-nullish-coalescing-operator": ^7.8.0
+  peerDependencies:
+    "@babel/core": ^7.0.0-0
+  checksum: 5a20d8bcbf2926dde3e9edcf847eaa5485d0d0fea76d0683ef1cafb11e0c35e46620391916283e1a9c0f76351e8c5ecccebf0d3a6bdf24559c5ad381433a0e3a
+  languageName: node
+  linkType: hard
+
 "@babel/plugin-syntax-bigint@npm:^7.0.0":
   version: 7.8.3
   resolution: "@babel/plugin-syntax-bigint@npm:7.8.3"
@@ -425,6 +437,17 @@ __metadata:
   languageName: node
   linkType: hard
 
+"@babel/plugin-syntax-nullish-coalescing-operator@npm:^7.8.0":
+  version: 7.8.3
+  resolution: "@babel/plugin-syntax-nullish-coalescing-operator@npm:7.8.3"
+  dependencies:
+    "@babel/helper-plugin-utils": ^7.8.0
+  peerDependencies:
+    "@babel/core": ^7.0.0-0
+  checksum: 4ba03753759a2d9783b792c060147a20f474f76c42edf77cbf89c6669f9f22ffb3cbba4facdd8ce651129db6089a81feca1f7e42da75244eabedecba37bd20be
+  languageName: node
+  linkType: hard
+
 "@babel/plugin-syntax-object-rest-spread@npm:^7.0.0":
   version: 7.8.3
   resolution: "@babel/plugin-syntax-object-rest-spread@npm:7.8.3"
@@ -2212,6 +2235,7 @@ __metadata:
     "@babel/core": ^7.11.0
     "@babel/plugin-proposal-class-properties": ^7.10.4
     "@babel/plugin-proposal-decorators": ^7.10.5
+    "@babel/plugin-proposal-nullish-coalescing-operator": ^7.10.4
     "@babel/plugin-transform-modules-commonjs": ^7.8.3
     "@babel/preset-typescript": ^7.10.4
     "@types/debug": ^4.1.5