Update dependencies, rely on Node.js 18, other small changes

* Update dependencies * Rely on Node.js 18 language features * Other small changes * Upgrade to tap@18 * Develop and test with [email protected] * Rebuild lockfile --------- Co-authored-by: Mark Wubben <[email protected]>
avajs · Nov 6, 2023 · cf0fa4c · cf0fa4c
1 parent 03a6723
commit cf0fa4c
Show file tree

Hide file tree

Showing 33 changed files with 8,514 additions and 8,187 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -41,7 +41,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        ts-version: [~5.1]
+        ts-version: [~5.2]
     steps:
       - uses: actions/checkout@v3
       - uses: actions/setup-node@v3

diff --git a/.gitignore b/.gitignore
@@ -1,3 +1,4 @@
+/.tap
 /coverage
 /media/**/node_modules/
 /node_modules/

diff --git a/.taprc b/.taprc
@@ -1,10 +1,7 @@
-browser: false
-coverage: false
+disable-coverage: true
+allow-empty-coverage: true
 files:
   - "test-tap/*.js"
   - "test-tap/reporters/*.js"
   - "test-tap/integration/*.js"
-flow: false
-jsx: false
 timeout: 300
-ts: false
diff --git a/.xo-config.cjs b/.xo-config.cjs
@@ -9,8 +9,10 @@ module.exports = {
 		'media/**',
 		'test/config/fixtures/config-errors/test.js',
 		'test/line-numbers/fixtures/line-numbers.js',
+		'test/**/fixtures',
 		'test-tap/fixture/snapshots/test-sourcemaps/build/**',
 		'test-tap/fixture/report/edgecases/ast-syntax-error.cjs',
+		'test-types',
 		'examples/typescript-*/**/*.ts',
 	],
 	rules: {
@@ -25,6 +27,7 @@ module.exports = {
 		],
 		'import/newline-after-import': 'error',
 		'unicorn/require-post-message-target-origin': 'off',
+		'unicorn/prefer-event-target': 'off',
 	},
 	overrides: [
 		{

diff --git a/docs/recipes/endpoint-testing.md b/docs/recipes/endpoint-testing.md
@@ -11,7 +11,7 @@ Since tests run concurrently, it's best to create a fresh server instance at lea
 Check out the example below:
 
 ```js
-import http from 'http';
+import http from 'node:http';
 import test from 'ava';
 import got from 'got';
 import listen from 'test-listen';

diff --git a/docs/recipes/typescript.md b/docs/recipes/typescript.md
@@ -4,7 +4,7 @@ Translations: [Español](https://github.com/avajs/ava-docs/blob/main/es_ES/docs/
 
 AVA comes bundled with a TypeScript definition file. This allows developers to leverage TypeScript for writing tests.
 
-This guide assumes you've already set up TypeScript for your project. Note that AVA's definition expects at least version 5.1.
+This guide assumes you've already set up TypeScript for your project. Note that AVA's definition expects at least version 5.2.
 
 ## Enabling AVA's support for TypeScript test files
 

diff --git a/docs/recipes/when-to-use-plan.md b/docs/recipes/when-to-use-plan.md
@@ -98,8 +98,8 @@ As stated in the previous example, using the `t.throws()` assertion with `async`
 In most cases, it's a bad idea to use any complex branching inside your tests. A notable exception is for tests that are auto-generated (perhaps from a JSON document). Below `t.plan()` is used to ensure the correctness of the JSON input:
 
 ```js
-import fs from 'fs';
-import path from 'path';
+import fs from 'node:fs';
+import path from 'node:path';
 
 const testData = JSON.parse(fs.readFileSync(new URL('./fixtures/test-definitions.json', import.meta.url)));
 

diff --git a/lib/line-numbers.js b/lib/line-numbers.js
@@ -13,7 +13,7 @@ const sortNumbersAscending = array => {
 };
 
 const parseNumber = string => Number.parseInt(string, 10);
-const removeAllWhitespace = string => string.replace(/\s/g, '');
+const removeAllWhitespace = string => string.replaceAll(/\s/g, '');
 const range = (start, end) => Array.from({length: end - start + 1}).fill(start).map((element, index) => element + index);
 
 const parseLineNumbers = suffix => sortNumbersAscending(distinctArray(

diff --git a/lib/load-config.js b/lib/load-config.js
@@ -4,7 +4,7 @@ import process from 'node:process';
 import url from 'node:url';
 
 import {isPlainObject} from 'is-plain-object';
-import {packageConfig, packageJsonPath} from 'pkg-conf';
+import {packageConfig, packageJsonPath} from 'package-config';
 
 const NO_SUCH_FILE = Symbol('no ava.config.js file');
 const MISSING_DEFAULT_EXPORT = Symbol('missing default export');

diff --git a/lib/parse-test-args.js b/lib/parse-test-args.js
@@ -2,7 +2,7 @@ const buildTitle = (raw, implementation, args) => {
 	let value = implementation?.title?.(raw, ...args) ?? raw;
 	const isValid = typeof value === 'string';
 	if (isValid) {
-		value = value.trim().replace(/\s+/g, ' ');
+		value = value.trim().replaceAll(/\s+/g, ' ');
 	}
 
 	return {

diff --git a/lib/reporters/default.js b/lib/reporters/default.js
@@ -338,7 +338,7 @@ export default class Reporter {
 				// we multiplex stdout and stderr into a single stream. However as
 				// long as stdStream is different from reportStream users can read
 				// their original output by redirecting the streams.
-				if (event.chunk[event.chunk.length - 1] !== 0x0A) {
+				if (event.chunk.at(-1) !== 0x0A) {
 					this.reportStream.write(os.EOL);
 				}
 
@@ -353,7 +353,7 @@ export default class Reporter {
 				// we multiplex stdout and stderr into a single stream. However as
 				// long as stdStream is different from reportStream users can read
 				// their original output by redirecting the streams.
-				if (event.chunk[event.chunk.length - 1] !== 0x0A) {
+				if (event.chunk.at(-1) !== 0x0A) {
 					this.reportStream.write(os.EOL);
 				}
 			}
@@ -648,7 +648,7 @@ export default class Reporter {
 		this.lineWriter.writeLine();
 
 		if (this.failures.length > 0) {
-			const lastFailure = this.failures[this.failures.length - 1];
+			const lastFailure = this.failures.at(-1);
 			for (const event of this.failures) {
 				this.writeFailure(event);
 				if (event !== lastFailure) {

diff --git a/lib/reporters/tap.js b/lib/reporters/tap.js
@@ -122,7 +122,7 @@ export default class TapReporter {
 		this.reportStream.write(`# ${stripAnsi(title)}${os.EOL}`);
 		if (evt.logs) {
 			for (const log of evt.logs) {
-				const logLines = indentString(log, 4).replace(/^ {4}/gm, '#   ');
+				const logLines = indentString(log, 4).replaceAll(/^ {4}/gm, '#   ');
 				this.reportStream.write(`${logLines}${os.EOL}`);
 			}
 		}

diff --git a/lib/slash.cjs b/lib/slash.cjs
@@ -30,7 +30,7 @@ function slash(path) {
 		return path;
 	}
 
-	return path.replace(/\\/g, '/');
+	return path.replaceAll('\\', '/');
 }
 
 module.exports = slash;
diff --git a/lib/snapshot-manager.js b/lib/snapshot-manager.js
@@ -104,7 +104,7 @@ function combineEntries({blocks}) {
 	const combined = new BufferBuilder();
 
 	for (const {title, snapshots} of blocks) {
-		const last = snapshots[snapshots.length - 1];
+		const last = snapshots.at(-1);
 		combined.write(`\n\n## ${title}\n\n`);
 
 		for (const [index, snapshot] of snapshots.entries()) {

diff --git a/lib/worker/channel.cjs b/lib/worker/channel.cjs
@@ -105,8 +105,8 @@ if (isRunningInChildProcess) {
 // Node.js. In order to keep track, explicitly reference before attaching.
 handle.ref();
 
-exports.options = selectAvaMessage(handle.channel, 'options').then(message => message.ava.options); // eslint-disable-line unicorn/prefer-top-level-await
-exports.peerFailed = selectAvaMessage(handle.channel, 'peer-failed'); // eslint-disable-line unicorn/prefer-top-level-await
+exports.options = selectAvaMessage(handle.channel, 'options').then(message => message.ava.options);
+exports.peerFailed = selectAvaMessage(handle.channel, 'peer-failed');
 exports.send = handle.send.bind(handle);
 exports.unref = handle.unref.bind(handle);
 

diff --git a/license b/license
@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) Sindre Sorhus <[email protected]> (sindresorhus.com)
+Copyright (c) Sindre Sorhus <[email protected]> (https://sindresorhus.com)
 
 Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions: