diff --git a/lib/errors.js b/lib/errors.js index 7aeac47..3ceaa45 100644 --- a/lib/errors.js +++ b/lib/errors.js @@ -8,7 +8,7 @@ class GitError extends Error { } class GitConnectionError extends GitError { - constructor (message) { + constructor () { super('A git connection error occurred') } @@ -18,13 +18,13 @@ class GitConnectionError extends GitError { } class GitPathspecError extends GitError { - constructor (message) { + constructor () { super('The git reference could not be found') } } class GitUnknownError extends GitError { - constructor (message) { + constructor () { super('An unknown git error occurred') } } diff --git a/test/clone.js b/test/clone.js index 6d802a9..988e727 100644 --- a/test/clone.js +++ b/test/clone.js @@ -28,7 +28,7 @@ let repoSha = '' let submodsRepoSha = '' t.setTimeout(120000) -t.test('create repo', { bail: true }, t => { +t.test('create repo', { bail: true }, () => { const git = (...cmd) => spawnGit(cmd, { cwd: repo }) const write = (f, c) => fs.writeFileSync(`${repo}/${f}`, c) return git('init', '-b', 'main') @@ -100,7 +100,7 @@ t.test('spawn daemon', { bail: true }, t => { daemon.on('close', () => fs.rmSync(me, { recursive: true, force: true })) }) -t.test('create a repo with a submodule', { bail: true }, t => { +t.test('create a repo with a submodule', { bail: true }, () => { const submoduleRepo = resolve(me, 'submodule-repo') const git = (...cmd) => spawnGit(cmd, { cwd: submoduleRepo }) const write = (f, c) => fs.writeFileSync(`${submoduleRepo}/${f}`, c) @@ -226,7 +226,7 @@ const clonedRepo = join(me, clonedRepoDir) const clonedRepoSpaces = join(me, clonedSpacesRepoDir) const clonedRepoSpaces2 = join(me, clonedSpacesRepoDir2) -t.test('setup aditional tests', t => { +t.test('setup aditional tests', () => { const git = (...cmd) => spawnGit(cmd, { cwd: regularRepo }) const write = (f, c) => fs.writeFileSync(`${regularRepo}/${f}`, c) return git('init', '-b', 'main') diff --git a/test/is-clean.js b/test/is-clean.js index a7e875e..491487d 100644 --- a/test/is-clean.js +++ b/test/is-clean.js @@ -8,7 +8,7 @@ const { promisify } = require('util') const writeFile = promisify(require('fs').writeFile) const write = (file, data) => writeFile(resolve(repo, file), data) -t.test('create git repo', t => +t.test('create git repo', () => spawn(['init'], { cwd: repo }) .then(() => spawn(['config', 'user.name', 'pacotedev'], { cwd: repo })) .then(() => spawn(['config', 'user.email', 'i+pacotedev@izs.me'], { cwd: repo })) diff --git a/test/revs.js b/test/revs.js index 4e14e6e..ba6e5da 100644 --- a/test/revs.js +++ b/test/revs.js @@ -25,7 +25,7 @@ const fixMainBranch = (err) => { return git('init') } const write = (f, c) => fs.writeFileSync(`${repo}/${f}`, c) -t.test('setup', t => +t.test('setup', () => git('init', '-b', mainBranch).catch(fixMainBranch) .then(() => git('config', 'user.name', 'pacotedev')) .then(() => git('config', 'user.email', 'i+pacotedev@izs.me')) @@ -57,7 +57,7 @@ t.test('point latest at HEAD', t => latest: '69.42.0', }))) -t.test('add a latest branch, point to 1.2.3 version', t => +t.test('add a latest branch, point to 1.2.3 version', () => git('checkout', '-b', 'latest') .then(() => git('reset', '--hard', 'version-1.2.3')) .then(() => git('checkout', mainBranch)) diff --git a/test/spawn.js b/test/spawn.js index da09581..b68fbc6 100644 --- a/test/spawn.js +++ b/test/spawn.js @@ -27,7 +27,7 @@ t.test('argument test for allowReplace', async t => { // for our purposes. This just tests that the argument is added // by default. const mockedSpawn = t.mock('../lib/spawn.js', { - '@npmcli/promise-spawn': async (exe, args, opts) => args, + '@npmcli/promise-spawn': async (exe, args) => args, }) const [allow, deny, allowWithArg, denyWithArg] = await Promise.all([ mockedSpawn(['a', 'b', 'c'], { allowReplace: true }),