release-it 14.2.1 → 14.2.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "release-it",
-  "version": "14.2.1",
+  "version": "14.2.2",
   "description": "Generic CLI tool to automate versioning and package publishing related tasks.",
   "keywords": [
     "build",
@@ -56,11 +56,11 @@
   "license": "MIT",
   "dependencies": {
     "@iarna/toml": "2.2.5",
-    "@octokit/rest": "18.0.7",
+    "@octokit/rest": "18.0.9",
     "async-retry": "1.3.1",
     "chalk": "4.1.0",
     "cosmiconfig": "7.0.0",
-    "debug": "4.2.0",
+    "debug": "4.3.1",
     "deprecated-obj": "2.0.0",
     "execa": "4.1.0",
     "find-up": "5.0.0",
@@ -78,17 +78,17 @@
     "parse-json": "5.1.0",
     "semver": "7.3.2",
     "shelljs": "0.8.4",
-    "update-notifier": "5.0.0",
+    "update-notifier": "5.0.1",
     "url-join": "4.0.1",
     "uuid": "8.3.1",
     "yaml": "1.10.0",
-    "yargs-parser": "20.2.3"
+    "yargs-parser": "20.2.4"
   },
   "devDependencies": {
-    "@octokit/request-error": "2.0.2",
+    "@octokit/request-error": "2.0.3",
     "ava": "3.13.0",
-    "codecov": "3.8.0",
-    "eslint": "7.12.1",
+    "codecov": "3.8.1",
+    "eslint": "7.14.0",
     "eslint-config-prettier": "6.15.0",
     "eslint-plugin-ava": "11.0.0",
     "eslint-plugin-import": "2.22.1",
@@ -96,9 +96,9 @@
     "markdown-toc": "1.2.0",
     "mock-fs": "4.13.0",
     "mock-stdio": "1.0.3",
-    "nock": "13.0.4",
+    "nock": "13.0.5",
     "nyc": "15.1.0",
-    "prettier": "2.1.2",
+    "prettier": "2.2.0",
     "proxyquire": "2.1.3",
     "sinon": "9.2.1",
     "strip-ansi": "6.0.0"

package/test/plugins.js

@@ -1,3 +1,4 @@
+const path = require('path');
 const test = require('ava');
 const sh = require('shelljs');
 const proxyquire = require('proxyquire');
@@ -131,3 +132,69 @@ test.serial('should disable core plugins', async t => {
     version: undefined
   });
 });
+
+test.serial('should expose context to execute commands', async t => {
+  const { bare } = t.context;
+  const latestVersion = '1.0.0';
+  const project = path.basename(bare);
+  const pkgName = 'plugin-context';
+  const owner = path.basename(path.dirname(bare));
+  gitAdd(`{"name":"${pkgName}","version":"${latestVersion}"}`, 'package.json', 'Add package.json');
+
+  class MyPlugin extends Plugin {
+    init() {
+      this.exec('echo ${version.isPreRelease}');
+    }
+    beforeBump() {
+      const context = this.config.getContext();
+      t.is(context.name, pkgName);
+      this.exec('echo ${name} ${repo.owner} ${repo.project} ${latestVersion} ${version}');
+    }
+    bump() {
+      const repo = this.config.getContext('repo');
+      t.is(repo.owner, owner);
+      t.is(repo.project, project);
+      t.is(repo.repository, `${owner}/${project}`);
+      this.exec('echo ${name} ${repo.owner} ${repo.project} ${latestVersion} ${version}');
+    }
+    beforeRelease() {
+      const context = this.config.getContext();
+      t.is(context.name, pkgName);
+      this.exec('echo ${name} ${repo.owner} ${repo.project} ${latestVersion} ${version} ${tagName}');
+    }
+    release() {
+      const context = this.config.getContext();
+      t.is(context.latestVersion, latestVersion);
+      t.is(context.version, '1.0.1');
+      this.exec('echo ${name} ${repo.owner} ${repo.project} ${latestVersion} ${version} ${tagName}');
+    }
+    afterRelease() {
+      const context = this.config.getContext();
+      t.is(context.tagName, '1.0.1');
+      this.exec('echo ${name} ${repo.owner} ${repo.project} ${latestVersion} ${version} ${tagName}');
+    }
+  }
+  const statics = { isEnabled: () => true, disablePlugin: () => null };
+  const options = { '@global': true, '@noCallThru': true };
+  const runTasks = proxyquire('../lib/tasks', {
+    'my-plugin': Object.assign(MyPlugin, statics, options)
+  });
+
+  const container = getContainer({ plugins: { 'my-plugin': {} } });
+  const exec = sinon.spy(container.shell, 'execFormattedCommand');
+
+  await runTasks({}, container);
+
+  const pluginExecArgs = exec.args
+    .map(args => args[0])
+    .filter(arg => typeof arg === 'string' && arg.startsWith('echo'));
+
+  t.deepEqual(pluginExecArgs, [
+    'echo false',
+    `echo ${pkgName} ${owner} ${project} ${latestVersion} 1.0.1`,
+    `echo ${pkgName} ${owner} ${project} ${latestVersion} 1.0.1`,
+    `echo ${pkgName} ${owner} ${project} ${latestVersion} 1.0.1 1.0.1`,
+    `echo ${pkgName} ${owner} ${project} ${latestVersion} 1.0.1 1.0.1`,
+    `echo ${pkgName} ${owner} ${project} ${latestVersion} 1.0.1 1.0.1`
+  ]);
+});

package/test/stub/shell.js

@@ -1,7 +1,7 @@
 const debug = require('debug')('release-it:shell-stub');
 const Shell = require('../../lib/shell');
 
-module.exports = class ShellStub extends Shell {
+class ShellStub extends Shell {
   exec(command) {
     if (/^(npm (ping|publish|show)|git fetch)/.test(command)) {
       debug(command);
@@ -17,4 +17,6 @@ module.exports = class ShellStub extends Shell {
     }
     return super.exec(...arguments);
   }
-};
+}
+
+module.exports = ShellStub;

package/test/tasks.js

@@ -398,22 +398,11 @@ test.serial('should propagate errors', async t => {
 });
 
 {
-  const myPlugin = sandbox.createStubInstance(Plugin);
-  myPlugin.namespace = 'my-plugin';
-  const MyPlugin = sandbox.stub().callsFake(() => myPlugin);
-  const myLocalPlugin = sandbox.createStubInstance(Plugin);
-  const MyLocalPlugin = sandbox.stub().callsFake(() => myLocalPlugin);
-  const replacePlugin = sandbox.createStubInstance(Plugin);
-  const ReplacePlugin = sandbox.stub().callsFake(() => replacePlugin);
-
+  class MyPlugin extends Plugin {}
   const statics = { isEnabled: () => true, disablePlugin: () => null };
   const options = { '@global': true, '@noCallThru': true };
   const runTasks = proxyquire('../lib/tasks', {
-    'my-plugin': Object.assign(MyPlugin, statics, options),
-    '/my/plugin': Object.assign(MyLocalPlugin, statics, options),
-    'replace-plugin': Object.assign(ReplacePlugin, statics, options, {
-      disablePlugin: () => ['version', 'git']
-    })
+    'my-plugin': Object.assign(MyPlugin, statics, options)
   });
 
   test.serial('should run all hooks', async t => {
@@ -421,9 +410,9 @@ test.serial('should propagate errors', async t => {
     const hooks = {};
     ['before', 'after'].forEach(prefix => {
       ['version', 'git', 'npm', 'my-plugin'].forEach(ns => {
-        ['init', 'beforeBump', 'bump', 'beforeRelease', 'release', 'afterRelease'].forEach(lifecycle => {
-          hooks[`${prefix}:${lifecycle}`] = `echo ${prefix}:${lifecycle}`;
-          hooks[`${prefix}:${ns}:${lifecycle}`] = `echo ${prefix}:${ns}:${lifecycle}`;
+        ['init', 'beforeBump', 'bump', 'beforeRelease', 'release', 'afterRelease'].forEach(cycle => {
+          hooks[`${prefix}:${cycle}`] = `echo ${prefix}:${cycle}`;
+          hooks[`${prefix}:${ns}:${cycle}`] = `echo ${prefix}:${ns}:${cycle}`;
         });
       });
     });