Fix tests

package.json

@@ -46,7 +46,7 @@
   "scripts": {
     "lint": "eslint lib test",
     "format": "prettier --write \"{lib,test}/**/*.js\"",
-    "test": "ava",
+    "test": "ava --timeout=20m",
     "coverage": "nyc --reporter=lcov --reporter=html --config=config/.codecov.yml npm test",
     "codecov": "nyc --reporter=json --config=config/.codecov.yml npm test && codecov -f coverage/coverage-final.json",
     "readme": "markdown-toc README.md -i --maxdepth=2 --bullets=-",

test/github.js

@@ -221,7 +221,9 @@ test('should release to git.pushRepo', async t => {
   exec.restore();
 });
 
-test('should throw for unauthenticated user', async t => {
+const testSkipOnActions = process.env.GITHUB_ACTIONS ? test.skip : test;
+
+testSkipOnActions('should throw for unauthenticated user', async t => {
   const options = { github: { tokenRef, pushRepo, host } };
   const github = factory(GitHub, { options });
   const stub = sinon.stub(github.client.users, 'getAuthenticated');
@@ -235,7 +237,7 @@ test('should throw for unauthenticated user', async t => {
   stub.restore();
 });
 
-test('should throw for non-collaborator', async t => {
+testSkipOnActions('should throw for non-collaborator', async t => {
   interceptAuthentication({ username: 'john' });
   const options = { github: { tokenRef, pushRepo, host } };
   const github = factory(GitHub, { options });
@@ -248,10 +250,11 @@ test('should throw for non-collaborator', async t => {
 });
 
 test.serial('should skip authentication and collaborator checks when running on GitHub Actions', async t => {
-  const GITHUB_ACTIONS = process.env.GITHUB_ACTIONS;
-  const GITHUB_ACTOR = process.env.GITHUB_ACTOR;
-  process.env.GITHUB_ACTIONS = 1;
-  process.env.GITHUB_ACTOR = 'release-it';
+  const { GITHUB_ACTIONS, GITHUB_ACTOR } = process.env;
+  if (!GITHUB_ACTIONS) {
+    process.env.GITHUB_ACTIONS = 1;
+    process.env.GITHUB_ACTOR = 'webpro';
+  }
 
   const options = { github: { tokenRef } };
   const github = factory(GitHub, { options });
@@ -262,12 +265,15 @@ test.serial('should skip authentication and collaborator checks when running on
 
   t.is(authStub.callCount, 0);
   t.is(collaboratorStub.callCount, 0);
-  t.is(github.getContext('username'), 'release-it');
+  t.is(github.getContext('username'), 'webpro');
 
   authStub.restore();
   collaboratorStub.restore();
-  process.env.GITHUB_ACTIONS = GITHUB_ACTIONS ?? '';
-  process.env.GITHUB_ACTOR = GITHUB_ACTOR ?? '';
+
+  if (!GITHUB_ACTIONS) {
+    process.env.GITHUB_ACTIONS = GITHUB_ACTIONS || '';
+    process.env.GITHUB_ACTOR = GITHUB_ACTOR || '';
+  }
 });
 
 test('should handle octokit client error (without retries)', async t => {

test/plugins.js

@@ -52,13 +52,15 @@ test.serial('should instantiate plugins and execute all release-cycle methods',
 
   sh.mkdir('my-plugin');
   sh.pushd('-q', 'my-plugin');
-  sh.exec('npm link release-it');
+  sh.exec('npm init -f');
+  sh.exec('npm install release-it');
   sh.ShellString("const { Plugin } = require('release-it'); module.exports = " + MyPlugin.toString()).toEnd('index.js');
   sh.popd();
 
   sh.mkdir('-p', 'my/plugin');
   sh.pushd('-q', 'my/plugin');
-  sh.exec('npm link release-it');
+  sh.exec('npm init -f');
+  sh.exec('npm install release-it');
   sh.ShellString("const { Plugin } = require('release-it'); module.exports = " + MyPlugin.toString()).toEnd('index.js');
   sh.popd();
 
@@ -67,7 +69,7 @@ test.serial('should instantiate plugins and execute all release-cycle methods',
       'my-plugin': {
         name: 'foo'
       },
-      './my/plugin': [
+      './my/plugin/index.js': [
         'named-plugin',
         {
           name: 'bar'
@@ -111,7 +113,8 @@ test.serial('should disable core plugins', async t => {
   sh.exec('npm init -f');
   sh.mkdir('replace-plugin');
   sh.pushd('-q', 'replace-plugin');
-  sh.exec('npm link release-it');
+  sh.exec('npm init -f');
+  sh.exec('npm install release-it');
   const content = "const { Plugin } = require('release-it'); module.exports = " + ReplacePlugin.toString();
   sh.ShellString(content).toEnd('index.js');
   sh.popd();
@@ -138,7 +141,7 @@ test.serial('should support ESM-based plugins', async t => {
   sh.mkdir('my-plugin');
   sh.pushd('-q', 'my-plugin');
   sh.ShellString('{"name":"my-plugin","version":"1.0.0","type": "module"}').toEnd('package.json');
-  sh.exec('npm link release-it');
+  sh.exec('npm install release-it@esm');
   const content = "import { Plugin } from 'release-it'; " + MyPlugin.toString() + '; export default MyPlugin;';
   sh.ShellString(content).toEnd('index.js');
   sh.popd();
@@ -166,7 +169,8 @@ test.serial('should expose context to execute commands', async t => {
 
   sh.mkdir('context-plugin');
   sh.pushd('-q', 'context-plugin');
-  sh.exec('npm link release-it');
+  sh.exec('npm init -f');
+  sh.exec('npm install release-it');
   const content = "const { Plugin } = require('release-it'); module.exports = " + ContextPlugin.toString();
   sh.ShellString(content).toEnd('index.js');
   sh.popd();

test/tasks.js

@@ -401,7 +401,7 @@ test.serial('should propagate errors', async t => {
     sh.mkdir('my-plugin');
     sh.pushd('-q', 'my-plugin');
     sh.exec('npm init -f');
-    sh.exec('npm link release-it');
+    sh.exec('npm install release-it');
     const plugin = "const { Plugin } = require('release-it'); module.exports = class MyPlugin extends Plugin {};";
     sh.ShellString(plugin).toEnd('index.js');
     sh.popd();