semantic-release-vsce 5.7.3 → 6.0.0

package/test/index.test.js

@@ -1,13 +1,17 @@
-const test = require('ava').serial;
-const sinon = require('sinon');
-const proxyquire = require('proxyquire');
+import avaTest from 'ava';
+import esmock from 'esmock';
+import { resolve } from 'path';
+import { fake, stub } from 'sinon';
+
+// Run tests serially to avoid env pollution
+const test = avaTest.serial;
 
 const semanticReleasePayload = {
   nextRelease: {
     version: '1.0.0',
   },
   logger: {
-    log: sinon.fake(),
+    log: fake(),
   },
   cwd: process.cwd(),
 };
@@ -18,9 +22,9 @@ const pluginConfig = {
 
 test.beforeEach((t) => {
   t.context.stubs = {
-    verifyVsceStub: sinon.stub().resolves(),
-    vscePublishStub: sinon.stub().resolves(),
-    vscePrepareStub: sinon.stub().resolves(),
+    verifyVsceStub: stub().resolves(),
+    vscePublishStub: stub().resolves(),
+    vscePrepareStub: stub().resolves(),
   };
 });
 
@@ -32,10 +36,16 @@ test.afterEach((t) => {
 
 test('verifyConditions', async (t) => {
   const { verifyVsceStub, vscePrepareStub, vscePublishStub } = t.context.stubs;
-  const { verifyConditions } = proxyquire('../index.js', {
-    './lib/verify': verifyVsceStub,
-    './lib/publish': vscePublishStub,
-    './lib/prepare': vscePrepareStub,
+  const { verifyConditions } = await esmock('../index.js', {
+    '../lib/verify.js': {
+      verify: verifyVsceStub,
+    },
+    '../lib/prepare.js': {
+      prepare: vscePrepareStub,
+    },
+    '../lib/publish.js': {
+      publish: vscePublishStub,
+    },
   });
 
   await verifyConditions(pluginConfig, semanticReleasePayload);
@@ -50,11 +60,16 @@ test('verifyConditions', async (t) => {
 
 test('prepare and unverified', async (t) => {
   const { verifyVsceStub, vscePrepareStub, vscePublishStub } = t.context.stubs;
-  const { prepare } = proxyquire('../index.js', {
-    './lib/verify': verifyVsceStub,
-    './lib/publish': vscePublishStub,
-    './lib/prepare': vscePrepareStub,
-    verified: false,
+  const { prepare } = await esmock('../index.js', {
+    '../lib/verify.js': {
+      verify: verifyVsceStub,
+    },
+    '../lib/prepare.js': {
+      prepare: vscePrepareStub,
+    },
+    '../lib/publish.js': {
+      publish: vscePublishStub,
+    },
   });
 
   await prepare(pluginConfig, semanticReleasePayload);
@@ -75,10 +90,16 @@ test('prepare and unverified', async (t) => {
 
 test('prepare and verified', async (t) => {
   const { verifyVsceStub, vscePrepareStub, vscePublishStub } = t.context.stubs;
-  const { prepare, verifyConditions } = proxyquire('../index.js', {
-    './lib/verify': verifyVsceStub,
-    './lib/publish': vscePublishStub,
-    './lib/prepare': vscePrepareStub,
+  const { prepare, verifyConditions } = await esmock('../index.js', {
+    '../lib/verify.js': {
+      verify: verifyVsceStub,
+    },
+    '../lib/prepare.js': {
+      prepare: vscePrepareStub,
+    },
+    '../lib/publish.js': {
+      publish: vscePublishStub,
+    },
   });
 
   await verifyConditions(pluginConfig, semanticReleasePayload);
@@ -95,10 +116,16 @@ test('prepare and verified', async (t) => {
 
 test('publish that is unverified and unprepared', async (t) => {
   const { verifyVsceStub, vscePrepareStub, vscePublishStub } = t.context.stubs;
-  const { publish } = proxyquire('../index.js', {
-    './lib/verify': verifyVsceStub,
-    './lib/publish': vscePublishStub,
-    './lib/prepare': vscePrepareStub,
+  const { publish } = await esmock('../index.js', {
+    '../lib/verify.js': {
+      verify: verifyVsceStub,
+    },
+    '../lib/prepare.js': {
+      prepare: vscePrepareStub,
+    },
+    '../lib/publish.js': {
+      publish: vscePublishStub,
+    },
   });
 
   await publish(pluginConfig, semanticReleasePayload);
@@ -115,10 +142,16 @@ test('publish that is unverified and unprepared', async (t) => {
 
 test('publish that is verified but unprepared', async (t) => {
   const { verifyVsceStub, vscePrepareStub, vscePublishStub } = t.context.stubs;
-  const { publish, verifyConditions } = proxyquire('../index.js', {
-    './lib/verify': verifyVsceStub,
-    './lib/publish': vscePublishStub,
-    './lib/prepare': vscePrepareStub,
+  const { publish, verifyConditions } = await esmock('../index.js', {
+    '../lib/verify.js': {
+      verify: verifyVsceStub,
+    },
+    '../lib/prepare.js': {
+      prepare: vscePrepareStub,
+    },
+    '../lib/publish.js': {
+      publish: vscePublishStub,
+    },
   });
 
   await verifyConditions(pluginConfig, semanticReleasePayload);
@@ -136,10 +169,16 @@ test('publish that is verified but unprepared', async (t) => {
 
 test('publish that is already verified & prepared', async (t) => {
   const { verifyVsceStub, vscePrepareStub, vscePublishStub } = t.context.stubs;
-  const { prepare, publish, verifyConditions } = proxyquire('../index.js', {
-    './lib/verify': verifyVsceStub,
-    './lib/publish': vscePublishStub,
-    './lib/prepare': vscePrepareStub,
+  const { prepare, publish, verifyConditions } = await esmock('../index.js', {
+    '../lib/verify.js': {
+      verify: verifyVsceStub,
+    },
+    '../lib/prepare.js': {
+      prepare: vscePrepareStub,
+    },
+    '../lib/publish.js': {
+      publish: vscePublishStub,
+    },
   });
 
   await verifyConditions(pluginConfig, semanticReleasePayload);
@@ -158,10 +197,16 @@ test('publish that is already verified & prepared', async (t) => {
 
 test('it does not publish the package if publishing is disabled', async (t) => {
   const { verifyVsceStub, vscePrepareStub, vscePublishStub } = t.context.stubs;
-  const { prepare, publish, verifyConditions } = proxyquire('../index.js', {
-    './lib/verify': verifyVsceStub,
-    './lib/publish': vscePublishStub,
-    './lib/prepare': vscePrepareStub,
+  const { prepare, publish, verifyConditions } = await esmock('../index.js', {
+    '../lib/verify.js': {
+      verify: verifyVsceStub,
+    },
+    '../lib/prepare.js': {
+      prepare: vscePrepareStub,
+    },
+    '../lib/publish.js': {
+      publish: vscePublishStub,
+    },
   });
 
   await verifyConditions(
@@ -176,14 +221,20 @@ test('it does not publish the package if publishing is disabled', async (t) => {
 
 test('it can publish when `OVSX_PAT` is present but `VSCE_PAT` is missing', async (t) => {
   const token = 'abc123';
-  sinon.stub(process, 'env').value({
+  stub(process, 'env').value({
     OVSX_PAT: token,
   });
   const { verifyVsceStub, vscePrepareStub, vscePublishStub } = t.context.stubs;
-  const { prepare, publish, verifyConditions } = proxyquire('../index.js', {
-    './lib/verify': verifyVsceStub,
-    './lib/publish': vscePublishStub,
-    './lib/prepare': vscePrepareStub,
+  const { prepare, publish, verifyConditions } = await esmock('../index.js', {
+    '../lib/verify.js': {
+      verify: verifyVsceStub,
+    },
+    '../lib/prepare.js': {
+      prepare: vscePrepareStub,
+    },
+    '../lib/publish.js': {
+      publish: vscePublishStub,
+    },
   });
 
   await verifyConditions({ ...pluginConfig }, semanticReleasePayload);
@@ -202,14 +253,18 @@ test('it can publish when `OVSX_PAT` is present but `VSCE_PAT` is missing', asyn
 
 test('expand globs if publishPackagePath is set', async (t) => {
   const { verifyVsceStub, vscePrepareStub, vscePublishStub } = t.context.stubs;
-  const { publish } = proxyquire('../index.js', {
-    './lib/verify': verifyVsceStub,
-    './lib/publish': vscePublishStub,
-    './lib/prepare': vscePrepareStub,
+  const { publish } = await esmock.p('../index.js', {
+    '../lib/verify.js': {
+      verify: verifyVsceStub,
+    },
+    '../lib/prepare.js': {
+      prepare: vscePrepareStub,
+    },
+    '../lib/publish.js': {
+      publish: vscePublishStub,
+    },
     glob: {
-      glob: {
-        sync: sinon.stub().returns(['package1.vsix', 'package2.vsix']),
-      },
+      glob: stub().resolves(['package1.vsix', 'package2.vsix']),
     },
   });
 
@@ -232,21 +287,25 @@ test('expand globs if publishPackagePath is set', async (t) => {
 
 test('publishes an extension in a non-root folder', async (t) => {
   const { verifyVsceStub, vscePrepareStub, vscePublishStub } = t.context.stubs;
-  const { publish } = proxyquire('../index.js', {
-    './lib/verify': verifyVsceStub,
-    './lib/publish': vscePublishStub,
-    './lib/prepare': vscePrepareStub,
+  const { publish } = await esmock.p('../index.js', {
+    '../lib/verify.js': {
+      verify: verifyVsceStub,
+    },
+    '../lib/prepare.js': {
+      prepare: vscePrepareStub,
+    },
+    '../lib/publish.js': {
+      publish: vscePublishStub,
+    },
     glob: {
-      glob: {
-        sync: sinon.stub().returns(['package1.vsix', 'package2.vsix']),
-      },
+      glob: stub().resolves(['package1.vsix', 'package2.vsix']),
     },
   });
 
   const pluginConfig = {
     packageRoot: './vscode-extension',
   };
-  const resolvedCwd = `${semanticReleasePayload.cwd}/vscode-extension`;
+  const resolvedCwd = resolve(`${semanticReleasePayload.cwd}/vscode-extension`);
 
   await publish(pluginConfig, semanticReleasePayload);
 

package/test/prepare.test.js

@@ -1,16 +1,19 @@
+import avaTest from 'ava';
+import { fake, stub } from 'sinon';
+import esmock from 'esmock';
+import process from 'node:process';
+
 // Run tests serially to avoid env pollution
-const test = require('ava').serial;
-const sinon = require('sinon');
-const proxyquire = require('proxyquire');
+const test = avaTest.serial;
 
 const logger = {
-  log: sinon.fake(),
+  log: fake(),
 };
 const cwd = process.cwd();
 
 test.beforeEach((t) => {
   t.context.stubs = {
-    execaStub: sinon.stub(),
+    execaStub: stub().resolves(),
   };
 });
 
@@ -20,8 +23,10 @@ test.afterEach((t) => {
 
 test('packageVsix is disabled', async (t) => {
   const { execaStub } = t.context.stubs;
-  const prepare = proxyquire('../lib/prepare', {
-    execa: execaStub,
+  const { prepare } = await esmock('../lib/prepare.js', {
+    execa: {
+      execa: execaStub,
+    },
   });
 
   const version = '1.0.0';
@@ -32,8 +37,10 @@ test('packageVsix is disabled', async (t) => {
 
 test('packageVsix is not specified', async (t) => {
   const { execaStub } = t.context.stubs;
-  const prepare = proxyquire('../lib/prepare', {
-    execa: execaStub,
+  const { prepare } = await esmock('../lib/prepare.js', {
+    execa: {
+      execa: execaStub,
+    },
   });
 
   const version = '1.0.0';
@@ -44,8 +51,10 @@ test('packageVsix is not specified', async (t) => {
 
 test('packageVsix is a string', async (t) => {
   const { execaStub } = t.context.stubs;
-  const prepare = proxyquire('../lib/prepare', {
-    execa: execaStub,
+  const { prepare } = await esmock('../lib/prepare.js', {
+    execa: {
+      execa: execaStub,
+    },
   });
 
   const version = '1.0.0';
@@ -65,10 +74,12 @@ test('packageVsix is true', async (t) => {
   const { execaStub } = t.context.stubs;
   const name = 'test';
 
-  const prepare = proxyquire('../lib/prepare', {
-    execa: execaStub,
-    'fs-extra': {
-      readJson: sinon.stub().returns({
+  const { prepare } = await esmock('../lib/prepare.js', {
+    execa: {
+      execa: execaStub,
+    },
+    'fs-extra/esm': {
+      readJson: stub().resolves({
         name,
       }),
     },
@@ -92,16 +103,18 @@ test('packageVsix is not set but OVSX_PAT is', async (t) => {
   const { execaStub } = t.context.stubs;
   const name = 'test';
 
-  const prepare = proxyquire('../lib/prepare', {
-    execa: execaStub,
-    'fs-extra': {
-      readJson: sinon.stub().returns({
+  const { prepare } = await esmock('../lib/prepare.js', {
+    execa: {
+      execa: execaStub,
+    },
+    'fs-extra/esm': {
+      readJson: stub().resolves({
         name,
       }),
     },
   });
 
-  sinon.stub(process, 'env').value({
+  stub(process, 'env').value({
     OVSX_PAT: 'abc123',
   });
 
@@ -123,20 +136,24 @@ test('packageVsix when target is set', async (t) => {
   const { execaStub } = t.context.stubs;
   const name = 'test';
 
-  const prepare = proxyquire('../lib/prepare', {
-    execa: execaStub,
-    'fs-extra': {
-      readJson: sinon.stub().returns({
+  const target = 'linux-x64';
+
+  const { prepare } = await esmock('../lib/prepare.js', {
+    execa: {
+      execa: execaStub,
+    },
+    'fs-extra/esm': {
+      readJson: stub().resolves({
         name,
       }),
     },
   });
 
   const version = '1.0.0';
-  const target = 'linux-x64';
+
   const packagePath = `${name}-${target}-${version}.vsix`;
 
-  sinon.stub(process, 'env').value({
+  stub(process, 'env').value({
     VSCE_TARGET: target,
   });
 
@@ -162,10 +179,12 @@ test('packageVsix when target is set to universal', async (t) => {
   const { execaStub } = t.context.stubs;
   const name = 'test';
 
-  const prepare = proxyquire('../lib/prepare', {
-    execa: execaStub,
-    'fs-extra': {
-      readJson: sinon.stub().returns({
+  const { prepare } = await esmock('../lib/prepare.js', {
+    execa: {
+      execa: execaStub,
+    },
+    'fs-extra/esm': {
+      readJson: stub().resolves({
         name,
       }),
     },
@@ -174,7 +193,7 @@ test('packageVsix when target is set to universal', async (t) => {
   const version = '1.0.0';
   const packagePath = `${name}-${version}.vsix`;
 
-  sinon.stub(process, 'env').value({
+  stub(process, 'env').value({
     VSCE_TARGET: 'universal',
   });