fix(linting): no-unused-vars

Author: lukekarrys

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')
   }
 }

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')

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 }))

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))

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 }),