Update to Pa11y v7 and other dependency updates

.github/workflows/publish.yml

@@ -11,12 +11,12 @@ on:
 
 jobs:
   publish:
-    runs-on: ubuntu-20.04
+    runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
       - uses: actions/setup-node@v3
         with:
-          node-version: 16
+          node-version: 20
           registry-url: https://registry.npmjs.org
       - run: npm ci
 

.github/workflows/tests.yml

@@ -11,7 +11,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-node@v3
         with:
-          node-version: 12
+          node-version: 18
       - run: npm ci
       - run: npm run lint
 
@@ -21,8 +21,8 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        os: [ubuntu-20.04, windows-latest, macos-latest]
-        node-version: [12, 14, 16, 18, 20]
+        os: [ubuntu-latest, windows-latest, macos-latest]
+        node-version: [18, 20]
     steps:
       - name: Normalise line-ending handling in Git
         shell: bash

.nvmrc

@@ -1 +1 @@
-14
+18

README.md

@@ -11,7 +11,7 @@ Use this tool to test against a list of URLs or a sitemap, and report on issues
 
 ## Requirements
 
-This command line tool requires a stable (even-numbered) [Node.js] version of 12 or above.
+This command line tool requires a stable (even-numbered) [Node.js] version of 18 or above.
 
 ### Pa11y CI 3 and Ubuntu
 

bin/pa11y-ci.js

@@ -64,29 +64,30 @@ const urls = globby.sync(commander.args, {
 }).map(protocolify);
 
 // Start the promise chain to actually run everything
+const options = commander.opts();
 Promise.resolve()
 	.then(() => {
 		// Load config based on the `--config` flag
-		return loadConfig(commander.config);
+		return loadConfig(options.config);
 	})
 	.then(config => {
 		// Load a sitemap based on the `--sitemap` flag
-		if (commander.sitemap) {
-			return loadSitemapIntoConfig(commander, config);
+		if (options.sitemap) {
+			return loadSitemapIntoConfig(options, config);
 		}
 		return config;
 	})
 	.then(config => {
 		// Override config reporters with CLI argument
-		if (commander.reporter) {
-			config.defaults.reporters = [commander.reporter];
+		if (options.reporter) {
+			config.defaults.reporters = [options.reporter];
 		}
 		// Actually run Pa11y CI
 		return pa11yCi(urls.concat(config.urls || []), config.defaults);
 	})
 	.then(report => {
 		// Output JSON if asked for it
-		if (commander.json) {
+		if (options.json) {
 			console.log(JSON.stringify(report, (key, value) => {
 				if (value instanceof Error) {
 					return {
@@ -98,16 +99,16 @@ Promise.resolve()
 		}
 		// Decide on an exit code based on whether
 		// errors are below threshold or everything passes
-		if (report.errors >= parseInt(commander.threshold, 10) && report.passes < report.total) {
-			process.exit(2);
+		if (report.errors >= parseInt(options.threshold, 10) && report.passes < report.total) {
+			process.exitCode = 2;
 		} else {
-			process.exit(0);
+			process.exitCode = 0;
 		}
 	})
 	.catch(error => {
 		// Handle any errors
 		console.error(error.message);
-		process.exit(1);
+		process.exitCode = 1;
 	});
 
 // This function loads the JSON or JavaScript config
@@ -128,7 +129,7 @@ function loadConfig(configPath) {
 			if (!config) {
 				config = loadLocalConfigWithJson(configPath);
 			}
-			if (commander.config && !config) {
+			if (options.config && !config) {
 				return reject(new Error(`The config file "${configPath}" could not be loaded`));
 			}
 		} catch (error) {
@@ -200,7 +201,7 @@ function defaultConfig(config) {
 	config.defaults.log = config.defaults.log || console;
 	// 	Setting to undefined rather than 0 allows for a fallback to the default
 	config.defaults.wrapWidth = process.stdout.columns || undefined;
-	if (commander.json) {
+	if (options.json) {
 		delete config.defaults.log;
 	}
 	return config;

lib/pa11y-ci.js

@@ -76,7 +76,7 @@
 
 		// Create a Pa11y test function and an async queue
 		const taskQueue = queue(testRunner, options.concurrency);
-		taskQueue.drain = testRunComplete;
+		taskQueue.drain(testRunComplete);
 
 		// Push the URLs on to the queue
 		taskQueue.push(urls);
@@ -105,7 +105,7 @@
 
 		// This is the actual test runner, which the queue will
 		// execute on each of the URLs
 		async function testRunner(config) {
 			let url;
 			if (typeof config === 'string') {
 				url = config;