Switch to c8 for coverage

Relates to istanbuljs/nyc#1287

.github/workflows/ci.yml

@@ -45,7 +45,7 @@
                 },
                 {
                     "name": "Run tests",
-                    "run": "./node_modules/.bin/nyc --reporter=lcovonly npm test"
+                    "run": "npm test"
                 },
                 {
                     "name": "Push coverage to Codecov",

cspell.json

@@ -23,7 +23,8 @@
         "vpath",
         "gulpplugin",
         "gulpfile",
-        "automerged"
+        "automerged",
+        "lcovonly"
     ],
     "ignorePaths": [
         "cspell.json",

package.json

@@ -8,7 +8,7 @@
     "types": "./dist/main.d.ts",
     "scripts": {
         "pretest": "tsc && api-extractor run --local",
-        "test": "nyc --reporter=text ava",
+        "test": "c8 --all --reporter=text --reporter=lcovonly ava",
         "prepublishOnly": "tsc",
         "preversion": "npm test",
         "version": "npm run version:apiDocRefresh && npm run version:changelogBump",
@@ -58,8 +58,8 @@
         "@types/node": "^14.0.1",
         "@types/vinyl": "^2.0.4",
         "ava": "^3.7.1",
+        "c8": "^7.1.2",
         "changelog-updater": "^1.1.0",
-        "nyc": "^15.0.1",
         "p-defer": "^3.0.0",
         "sort-on": "^4.1.0",
         "typescript": "^3.8.3"

src/bundle-orchestrator.ts

@@ -127,7 +127,7 @@ export class BundleOrchestrator extends Transform {
         });
 
         // First up, we assign the logger if its there
-        /* istanbul ignore else */
+        /* c8 ignore else */
         if (config.Logger) this.logger = config.Logger;
 
         // Results callback
@@ -190,9 +190,7 @@ export class BundleOrchestrator extends Transform {
             callback();
         }
         catch (error) {
-            /* istanbul ignore next This is only here to ensure we are a polite pipeline member, and cannot be reasonably tested */
             this.logger.error("_transform completed with error", { error });
-            /* istanbul ignore next */
             callback(new PluginError(PluginName, error));
         }
     }

src/bundle.ts

@@ -95,7 +95,6 @@ export class Bundle {
             const orderedFiles: Vinyl[] = [];
             for (const path of this.initialPaths) {
                 const file = this.files.get(path);
-                /* istanbul ignore else map.get can technically be undefined */
                 if (file) orderedFiles.push(file);
                 else throw new Error("Unexpected condition, previously tracked file is undefined");
             }

src/test-util.ts

@@ -5,7 +5,6 @@ import { ExecutionContext } from "ava";
  * Maps lib logging to ava logging to assist debugging of failing tests.
  * @param t Execution context for test. Needed to associate logs with tests.
  */
-/* istanbul ignore next */
 export function mapAvaLoggerToStandard(t: ExecutionContext): TsLog.Logger {
     return {
         debug(message, ...optionalParams) {