fix: do not include skipped links in the final link count (#277)

Author: JustinBeckwith

src/cli.ts

@@ -246,15 +246,15 @@ async function main() {
   }
 
   const total = (Date.now() - start) / 1000;
-
+  const scannedLinks = result.links.filter(x => x.state !== LinkState.SKIPPED);
   if (!result.passed) {
     const borked = result.links.filter(x => x.state === LinkState.BROKEN);
     logger.error(
       chalk.bold(
         `${chalk.red('ERROR')}: Detected ${
           borked.length
         } broken links. Scanned ${chalk.yellow(
-          result.links.length.toString()
+          scannedLinks.length.toString()
         )} links in ${chalk.cyan(total.toString())} seconds.`
       )
     );
@@ -264,7 +264,7 @@ async function main() {
   logger.error(
     chalk.bold(
       `🤖 Successfully scanned ${chalk.green(
-        result.links.length.toString()
+        scannedLinks.length.toString()
       )} links in ${chalk.cyan(total.toString())} seconds.`
     )
   );

test/test.cli.ts

@@ -62,7 +62,11 @@ describe('cli', function () {
       '"LICENSE.md, unlinked.md"',
       'test/fixtures/markdown/README.md',
     ]);
-    assert.match(stripAnsi(res.stdout), /\[SKP\]/);
+    const stdout = stripAnsi(res.stdout);
+    const stderr = stripAnsi(res.stderr);
+    assert.match(stdout, /\[SKP\]/);
+    // make sure we don't report skipped links in the count
+    assert.match(stderr, /scanned 2 links/);
   });
 
   it('should provide CSV if asked nicely', async () => {