build(deps): -fs-extra

Author: gr2m

index.js

@@ -64,4 +64,3 @@ export async function fail(pluginConfig, context) {
 
   await failGitHub(pluginConfig, context);
 }
-

lib/glob-assets.js

@@ -14,6 +14,7 @@ export default async function globAssets({cwd}, assets) {
         assets.map(async (asset) => {
           // Wrap single glob definition in Array
           let glob = castArray(isPlainObject(asset) ? asset.path : asset);
+
           // TODO Temporary workaround for https://github.com/mrmlnc/fast-glob/issues/47
           glob = uniq([...(await dirGlob(glob, {cwd})), ...glob]);
 

lib/publish.js

@@ -1,6 +1,6 @@
 import {resolve, basename, extname} from 'node:path';
+import {stat, readFile} from 'node:fs/promises';
 
-import {stat, readFile} from 'fs-extra';
 import {isPlainObject, template} from 'lodash-es';
 import {getType} from 'mime';
 import debugFactory from 'debug';

package-lock.json

@@ -26,7 +26,6 @@
     "bottleneck": "^2.18.1",
     "debug": "^4.0.0",
     "dir-glob": "^3.0.0",
-    "fs-extra": "^10.0.0",
     "globby": "^12.0.2",
     "http-proxy-agent": "^5.0.0",
     "https-proxy-agent": "^5.0.0",
@@ -40,8 +39,8 @@
   "devDependencies": {
     "ava": "4.0.0-alpha.2",
     "c8": "7.9.0",
-    "clear-module": "4.1.1",
     "codecov": "3.8.3",
+    "cpy": "^8.1.2",
     "nock": "13.1.3",
     "proxy": "1.0.2",
     "proxyquire": "2.1.3",

package.json

@@ -9,7 +9,7 @@ import * as RATE_LIMIT from './helpers/rate-limit.js';
 /* eslint camelcase: ["error", {properties: "never"}] */
 
 // mock rate limit imported via lib/get-client.js
-await quibble.esm('../lib/definitions/rate-limit.js', RATE_LIMIT)
+await quibble.esm('../lib/definitions/rate-limit.js', RATE_LIMIT) // eslint-disable-line
 const addChannel = (await import('../lib/add-channel.js')).default
 
 test.beforeEach((t) => {

test/add-channel.test.js

@@ -1,9 +1,10 @@
 import {resolve} from 'node:path';
+import {mkdir} from 'node:fs/promises';
 
 import test from 'ava';
-import {copy, ensureDir} from 'fs-extra';
 import {isPlainObject, sortBy} from 'lodash-es';
 import tempy from 'tempy';
+import cpy from 'cpy';
 
 import globAssets from '../lib/glob-assets.js';
 
@@ -13,31 +14,31 @@ const fixtures = 'test/fixtures/files';
 
 test('Retrieve file from single path', async (t) => {
   const cwd = tempy.directory();
-  await copy(fixtures, cwd);
+  await cpy(fixtures, cwd, {dot: true});
   const globbedAssets = await globAssets({cwd}, ['upload.txt']);
 
   t.deepEqual(globbedAssets, ['upload.txt']);
 });
 
 test('Retrieve multiple files from path', async (t) => {
   const cwd = tempy.directory();
-  await copy(fixtures, cwd);
+  await cpy(fixtures, cwd, {dot: true});
   const globbedAssets = await globAssets({cwd}, ['upload.txt', 'upload_other.txt']);
 
   t.deepEqual(sortAssets(globbedAssets), sortAssets(['upload_other.txt', 'upload.txt']));
 });
 
 test('Include missing files as defined, using Object definition', async (t) => {
   const cwd = tempy.directory();
-  await copy(fixtures, cwd);
+  await cpy(fixtures, cwd, {dot: true});
   const globbedAssets = await globAssets({cwd}, ['upload.txt', {path: 'miss*.txt', label: 'Missing'}]);
 
   t.deepEqual(sortAssets(globbedAssets), sortAssets(['upload.txt', {path: 'miss*.txt', label: 'Missing'}]));
 });
 
 test('Retrieve multiple files from Object', async (t) => {
   const cwd = tempy.directory();
-  await copy(fixtures, cwd);
+  await cpy(fixtures, cwd, {dot: true});
   const globbedAssets = await globAssets({cwd}, [
     {path: 'upload.txt', name: 'upload_name', label: 'Upload label'},
     'upload_other.txt',
@@ -51,7 +52,7 @@ test('Retrieve multiple files from Object', async (t) => {
 
 test('Retrieve multiple files without duplicates', async (t) => {
   const cwd = tempy.directory();
-  await copy(fixtures, cwd);
+  await cpy(fixtures, cwd, {dot: true});
   const globbedAssets = await globAssets({cwd}, [
     'upload_other.txt',
     'upload.txt',
@@ -66,7 +67,7 @@ test('Retrieve multiple files without duplicates', async (t) => {
 
 test('Favor Object over String values when removing duplicates', async (t) => {
   const cwd = tempy.directory();
-  await copy(fixtures, cwd);
+  await cpy(fixtures, cwd, {dot: true});
   const globbedAssets = await globAssets({cwd}, [
     'upload_other.txt',
     'upload.txt',
@@ -88,47 +89,47 @@ test('Favor Object over String values when removing duplicates', async (t) => {
 
 test('Retrieve file from single glob', async (t) => {
   const cwd = tempy.directory();
-  await copy(fixtures, cwd);
+  await cpy(fixtures, cwd, {dot: true});
   const globbedAssets = await globAssets({cwd}, ['upload.*']);
 
   t.deepEqual(globbedAssets, ['upload.txt']);
 });
 
 test('Retrieve multiple files from single glob', async (t) => {
   const cwd = tempy.directory();
-  await copy(fixtures, cwd);
+  await cpy(fixtures, cwd, {dot: true});
   const globbedAssets = await globAssets({cwd}, ['*.txt']);
 
   t.deepEqual(sortAssets(globbedAssets), sortAssets(['upload_other.txt', 'upload.txt']));
 });
 
 test('Accept glob array with one value', async (t) => {
   const cwd = tempy.directory();
-  await copy(fixtures, cwd);
+  await cpy(fixtures, cwd, {dot: true});
   const globbedAssets = await globAssets({cwd}, [['*load.txt'], ['*_other.txt']]);
 
   t.deepEqual(sortAssets(globbedAssets), sortAssets(['upload_other.txt', 'upload.txt']));
 });
 
 test('Include globs that resolve to no files as defined', async (t) => {
   const cwd = tempy.directory();
-  await copy(fixtures, cwd);
+  await cpy(fixtures, cwd, {dot: true});
   const globbedAssets = await globAssets({cwd}, [['upload.txt', '!upload.txt']]);
 
   t.deepEqual(sortAssets(globbedAssets), sortAssets(['!upload.txt', 'upload.txt']));
 });
 
 test('Accept glob array with one value for missing files', async (t) => {
   const cwd = tempy.directory();
-  await copy(fixtures, cwd);
+  await cpy(fixtures, cwd, {dot: true});
   const globbedAssets = await globAssets({cwd}, [['*missing.txt'], ['*_other.txt']]);
 
   t.deepEqual(sortAssets(globbedAssets), sortAssets(['upload_other.txt', '*missing.txt']));
 });
 
 test('Replace name by filename for Object that match multiple files', async (t) => {
   const cwd = tempy.directory();
-  await copy(fixtures, cwd);
+  await cpy(fixtures, cwd, {dot: true});
   const globbedAssets = await globAssets({cwd}, [{path: '*.txt', name: 'upload_name', label: 'Upload label'}]);
 
   t.deepEqual(
@@ -142,56 +143,56 @@ test('Replace name by filename for Object that match multiple files', async (t)
 
 test('Include dotfiles', async (t) => {
   const cwd = tempy.directory();
-  await copy(fixtures, cwd);
+  await cpy(fixtures, cwd, {dot: true});
   const globbedAssets = await globAssets({cwd}, ['.dot*']);
 
   t.deepEqual(globbedAssets, ['.dotfile']);
 });
 
 test('Ingnore single negated glob', async (t) => {
   const cwd = tempy.directory();
-  await copy(fixtures, cwd);
+  await cpy(fixtures, cwd, {dot: true});
   const globbedAssets = await globAssets({cwd}, ['!*.txt']);
 
   t.deepEqual(globbedAssets, []);
 });
 
 test('Ingnore single negated glob in Object', async (t) => {
   const cwd = tempy.directory();
-  await copy(fixtures, cwd);
+  await cpy(fixtures, cwd, {dot: true});
   const globbedAssets = await globAssets({cwd}, [{path: '!*.txt'}]);
 
   t.deepEqual(globbedAssets, []);
 });
 
 test('Accept negated globs', async (t) => {
   const cwd = tempy.directory();
-  await copy(fixtures, cwd);
+  await cpy(fixtures, cwd, {dot: true});
   const globbedAssets = await globAssets({cwd}, [['*.txt', '!**/*_other.txt']]);
 
   t.deepEqual(globbedAssets, ['upload.txt']);
 });
 
 test('Expand directories', async (t) => {
   const cwd = tempy.directory();
-  await copy(fixtures, resolve(cwd, 'dir'));
+  await cpy(fixtures, resolve(cwd, 'dir'), {dot: true});
   const globbedAssets = await globAssets({cwd}, [['dir']]);
 
   t.deepEqual(sortAssets(globbedAssets), sortAssets(['dir', 'dir/upload_other.txt', 'dir/upload.txt', 'dir/.dotfile']));
 });
 
 test('Include empty tempy.directory as defined', async (t) => {
   const cwd = tempy.directory();
-  await copy(fixtures, cwd);
-  await ensureDir(resolve(cwd, 'empty'));
+  await cpy(fixtures, cwd, {dot: true});
+  await mkdir(resolve(cwd, 'empty'), {recursive: true});
   const globbedAssets = await globAssets({cwd}, [['empty']]);
 
   t.deepEqual(globbedAssets, ['empty']);
 });
 
 test('Deduplicate resulting files path', async (t) => {
   const cwd = tempy.directory();
-  await copy(fixtures, cwd);
+  await cpy(fixtures, cwd, {dot: true});
   const globbedAssets = await globAssets({cwd}, ['./upload.txt', resolve(cwd, 'upload.txt'), 'upload.txt']);
 
   t.is(globbedAssets.length, 1);

test/glob-assets.test.js

@@ -1,38 +1,33 @@
 import {resolve} from 'node:path';
 import {escape} from 'node:querystring';
+import {stat} from 'node:fs/promises';
 
 import test from 'ava';
-import {stat} from 'fs-extra';
-import {cleanAll} from 'nock';
-import {stub} from 'sinon';
-import proxyquire from 'proxyquire';
-import clearModule from 'clear-module';
+import nock from 'nock';
+import sinon from 'sinon';
+import quibble from 'quibble';
 import SemanticReleaseError from '@semantic-release/error';
 
 import {authenticate, upload} from './helpers/mock-github.js';
-import rateLimit from './helpers/rate-limit.js';
+import * as RATE_LIMIT_MOCK from './helpers/rate-limit.js';
 
 const cwd = 'test/fixtures/files';
-const client = proxyquire('../lib/get-client', {'./definitions/rate-limit': rateLimit});
-
-test.beforeEach((t) => {
-  // Clear npm cache to refresh the module state
-  clearModule('..');
-  t.context.m = proxyquire('..', {
-    './lib/verify': proxyquire('../lib/verify', {'./get-client': client}),
-    './lib/publish': proxyquire('../lib/publish', {'./get-client': client}),
-    './lib/success': proxyquire('../lib/success', {'./get-client': client}),
-    './lib/fail': proxyquire('../lib/fail', {'./get-client': client}),
-  });
+
+test.beforeEach(async (t) => {
+  // Mock rate limit imported via lib/get-client.js
+  await quibble.reset();
+  await quibble.esm('../lib/definitions/rate-limit.js', RATE_LIMIT_MOCK);
+
+  t.context.m = await import('../index.js');
   // Stub the logger
-  t.context.log = stub();
-  t.context.error = stub();
+  t.context.log = sinon.stub();
+  t.context.error = sinon.stub();
   t.context.logger = {log: t.context.log, error: t.context.error};
 });
 
 test.afterEach.always(() => {
   // Clear nock
-  cleanAll();
+  nock.cleanAll();
 });
 
 test.serial('Verify GitHub auth', async (t) => {

test/integration.test.js

@@ -1,8 +1,8 @@
+import {stat} from 'node:fs/promises';
 import {resolve} from 'node:path';
 import {escape} from 'node:querystring';
 
 import test from 'ava';
-import {stat} from 'fs-extra';
 import {cleanAll} from 'nock';
 import {stub} from 'sinon';
 import proxyquire from 'proxyquire';