test: remove usage of npx in tests (#241)

Author: JustinBeckwith

test/test.cli.ts

@@ -26,39 +26,34 @@ describe('cli', function () {
   });
 
   it('should show output for failures', async () => {
-    const res = await execa('npx', ['linkinator', 'test/fixtures/basic'], {
+    const res = await execa('linkinator', ['test/fixtures/basic'], {
       reject: false,
     });
     assert.match(res.stderr, /ERROR: Detected 1 broken links/);
   });
 
   it('should pass successful markdown scan', async () => {
-    const res = await execa('npx', [
-      'linkinator',
-      'test/fixtures/markdown/README.md',
-    ]);
+    const res = await execa('linkinator', ['test/fixtures/markdown/README.md']);
     assert.match(res.stderr, /Successfully scanned/);
   });
 
   it('should allow multiple paths', async () => {
-    const res = await execa('npx', [
-      'linkinator',
+    const res = await execa('linkinator', [
       'test/fixtures/markdown/unlinked.md',
       'test/fixtures/markdown/README.md',
     ]);
     assert.match(res.stderr, /Successfully scanned/);
   });
 
   it('should show help if no params are provided', async () => {
-    const res = await execa('npx', ['linkinator'], {
+    const res = await execa('linkinator', {
       reject: false,
     });
     assert.match(res.stdout, /\$ linkinator LOCATION \[ --arguments \]/);
   });
 
   it('should flag skipped links', async () => {
-    const res = await execa('npx', [
-      'linkinator',
+    const res = await execa('linkinator', [
       '--verbosity',
       'INFO',
       '--skip',
@@ -69,8 +64,7 @@ describe('cli', function () {
   });
 
   it('should provide CSV if asked nicely', async () => {
-    const res = await execa('npx', [
-      'linkinator',
+    const res = await execa('linkinator', [
       '--format',
       'csv',
       'test/fixtures/markdown/README.md',
@@ -79,8 +73,7 @@ describe('cli', function () {
   });
 
   it('should provide JSON if asked nicely', async () => {
-    const res = await execa('npx', [
-      'linkinator',
+    const res = await execa('linkinator', [
       '--format',
       'json',
       'test/fixtures/markdown/README.md',
@@ -90,17 +83,15 @@ describe('cli', function () {
   });
 
   it('should not show links if --silent', async () => {
-    const res = await execa('npx', [
-      'linkinator',
+    const res = await execa('linkinator', [
       '--silent',
       'test/fixtures/markdown/README.md',
     ]);
     assert.notMatch(res.stdout, /\[/);
   });
 
   it('should not show 200 links if verbosity is ERROR with JSON', async () => {
-    const res = await execa('npx', [
-      'linkinator',
+    const res = await execa('linkinator', [
       '--verbosity',
       'ERROR',
       '--format',
@@ -114,8 +105,7 @@ describe('cli', function () {
   });
 
   it('should accept a server-root', async () => {
-    const res = await execa('npx', [
-      'linkinator',
+    const res = await execa('linkinator', [
       '--markdown',
       '--server-root',
       'test/fixtures/markdown',
@@ -125,29 +115,24 @@ describe('cli', function () {
   });
 
   it('should accept globs', async () => {
-    const res = await execa('npx', [
-      'linkinator',
+    const res = await execa('linkinator', [
       'test/fixtures/markdown/*.md',
       'test/fixtures/markdown/**/*.md',
     ]);
     assert.match(res.stderr, /Successfully scanned/);
   });
 
   it('should throw on invalid format', async () => {
-    const res = await execa(
-      'npx',
-      ['linkinator', './README.md', '--format', 'LOL'],
-      {
-        reject: false,
-      }
-    );
+    const res = await execa('linkinator', ['./README.md', '--format', 'LOL'], {
+      reject: false,
+    });
     assert.match(res.stderr, /FORMAT must be/);
   });
 
   it('should throw on invalid verbosity', async () => {
     const res = await execa(
-      'npx',
-      ['linkinator', './README.md', '--VERBOSITY', 'LOL'],
+      'linkinator',
+      ['./README.md', '--VERBOSITY', 'LOL'],
       {
         reject: false,
       }
@@ -157,8 +142,8 @@ describe('cli', function () {
 
   it('should throw when verbosity and silent are flagged', async () => {
     const res = await execa(
-      'npx',
-      ['linkinator', './README.md', '--verbosity', 'DEBUG', '--silent'],
+      'linkinator',
+      ['./README.md', '--verbosity', 'DEBUG', '--silent'],
       {
         reject: false,
       }
@@ -168,8 +153,8 @@ describe('cli', function () {
 
   it('should show no output for verbosity=NONE', async () => {
     const res = await execa(
-      'npx',
-      ['linkinator', 'test/fixtures/basic', '--verbosity', 'NONE'],
+      'linkinator',
+      ['test/fixtures/basic', '--verbosity', 'NONE'],
       {
         reject: false,
       }
@@ -181,8 +166,8 @@ describe('cli', function () {
 
   it('should show callstacks for verbosity=DEBUG', async () => {
     const res = await execa(
-      'npx',
-      ['linkinator', 'test/fixtures/basic', '--verbosity', 'DEBUG'],
+      'linkinator',
+      ['test/fixtures/basic', '--verbosity', 'DEBUG'],
       {
         reject: false,
       }
@@ -192,8 +177,7 @@ describe('cli', function () {
   });
 
   it('should allow passing a config', async () => {
-    const res = await execa('npx', [
-      'linkinator',
+    const res = await execa('linkinator', [
       'test/fixtures/basic',
       '--config',
       'test/fixtures/config/skip-array-config.json',
@@ -223,8 +207,7 @@ describe('cli', function () {
     enableDestroy(server);
     await new Promise<void>(r => server.listen(port, r));
 
-    const res = await execa('npx', [
-      'linkinator',
+    const res = await execa('linkinator', [
       '--retry',
       'test/fixtures/retryCLI',
     ]);