test: update Ava to v8

.eslintrc

@@ -1,3 +1,16 @@
 {
-  "extends": "shellscape"
+  "extends": "shellscape",
+  "overrides": [
+    {
+      "files": "test/**/*.js",
+      "parserOptions": {
+        "sourceType": "module"
+      },
+      "rules": {
+        "import/no-unresolved": "off",
+        "import/extensions": "off",
+        "import/no-extraneous-dependencies": "off"
+      }
+    }
+  ]
 }

.github/workflows/test.yml

@@ -22,10 +22,10 @@ jobs:
         uses: actions/checkout@v7
 
       - name: NPM Rebuild
-        run: npm install
+        run: yarn
 
       - name: Run unit tests
-        run: npm run ci:coverage
+        run: yarn ci:coverage
 
       - name: Submit coverage data to codecov.
         run: bash <(curl -s https://codecov.io/bash)
@@ -39,12 +39,8 @@ jobs:
         uses: actions/checkout@v7
 
       - name: NPM Rebuild
-        run: npm install
+        run: yarn
 
       - name: Run Linting
-        run: npm run lint
-
-      # Disabled temporarily
-      # - name: Run NPM Audit
-      #   run: npm run security
+        run: yarn lint
 

package.json

@@ -32,7 +32,7 @@
   "devDependencies": {
     "@commitlint/cli": "12.1.4",
     "@commitlint/config-conventional": "12.1.4",
-    "ava": "3.15.0",
+    "ava": "8.1.1",
     "cheerio": "1.0.0-rc.10",
     "eslint-config-shellscape": "6.0.2",
     "eslint-plugin-filenames": "1.2.0",

test/integration.test.js

@@ -4,19 +4,19 @@
 // const { readFileSync } = require('fs');
 // const { join } = require('path');
 
-const test = require('ava');
-const cheerio = require('cheerio');
-const isAbsoluteUrl = require('is-absolute-url');
-const fetch = require('node-fetch');
-const postcss = require('postcss');
-const urljoin = require('url-join');
+import test from 'ava';
+import cheerio from 'cheerio';
+import isAbsoluteUrl from 'is-absolute-url';
+import urljoin from 'url-join';
+import postcss from 'postcss';
 
-const syntax = require('../lib');
+import syntax from '../lib/index.js';
 
 const sites = ['https://github.com', 'https://news.ycombinator.com'];
 
 for (const site of sites) {
   test(`integration: ${site}`, async (t) => {
+    t.timeout(50000);
     const res = await fetch(site);
     const html = await res.text();
     const $ = cheerio.load(html);
@@ -37,7 +37,6 @@ for (const site of sites) {
     for (const href of hrefs) {
       const req = await fetch(href);
       const css = await req.text();
-
       await postcss().process(css, {
         from: null,
         map: { annotation: false },

test/map.test.js

@@ -1,13 +1,14 @@
-const { readFileSync } = require('fs');
+import { readFileSync } from 'node:fs';
 
-const { join } = require('path');
+import { join } from 'node:path';
 
-const test = require('ava');
-const postcss = require('postcss');
+import test from 'ava';
+import postcss from 'postcss';
 
-const syntax = require('../lib');
+import syntax from '../lib/index.js';
 
 const { parser } = syntax;
+const __dirname = import.meta.dirname;
 
 // silence the rediculously verbose "You did not set any plugins, parser, or
 // stringifier" warnings in PostCSS.

test/package.json

@@ -0,0 +1 @@
+{"type": "module", "//": "delete this file when we migrate to ESM"}

test/parser/comments.test.js

@@ -1,7 +1,9 @@
-const test = require('ava');
-const Comment = require('postcss/lib/comment');
+import test from 'ava';
+import Comment from 'postcss/lib/comment';
 
-const { parse, nodeToString } = require('../../lib');
+import syntax from '../../lib/index.js';
+
+const { parse, nodeToString } = syntax;
 
 test('inline comment', (t) => {
   const less = '// batman';

test/parser/declaration.test.js

@@ -1,6 +1,8 @@
-const test = require('ava');
+import test from 'ava';
 
-const { parse, nodeToString } = require('../../lib');
+import syntax from '../../lib/index.js';
+
+const { parse, nodeToString } = syntax;
 
 test('should allow case-insensitive !important (#89)', (t) => {
   const less = 'a{k: v !IMPORTANT;}';

test/parser/escaping.test.js

@@ -1,6 +1,8 @@
-const test = require('ava');
+import test from 'ava';
 
-const { parse, nodeToString } = require('../../lib');
+import syntax from '../../lib/index.js';
+
+const { parse, nodeToString } = syntax;
 
 test('parses escaped string', (t) => {
   const less = `

test/parser/extend.test.js

@@ -1,6 +1,8 @@
-const test = require('ava');
+import test from 'ava';
 
-const { parse, nodeToString } = require('../../lib');
+import syntax from '../../lib/index.js';
+
+const { parse, nodeToString } = syntax;
 
 test('inline :extend()', (t) => {
   const less = '.a:extend(.b) {color: red;}';

test/parser/function.test.js

@@ -1,6 +1,8 @@
-const test = require('ava');
+import test from 'ava';
 
-const { parse, nodeToString } = require('../../lib');
+import syntax from '../../lib/index.js';
+
+const { parse, nodeToString } = syntax;
 
 test('each (#121)', (t) => {
   const params = `(@colors, {

test/parser/import.test.js

@@ -1,7 +1,9 @@
-const test = require('ava');
-const AtRule = require('postcss/lib/at-rule');
+import test from 'ava';
+import AtRule from 'postcss/lib/at-rule';
 
-const { parse, nodeToString } = require('../../lib');
+import syntax from '../../lib/index.js';
+
+const { parse, nodeToString } = syntax;
 
 test('@at-rules != @imports', async (t) => {
   const less = '@const "foo.less";';

test/parser/interpolation.test.js

@@ -1,8 +1,10 @@
 /* eslint no-useless-escape: off */
 
-const test = require('ava');
+import test from 'ava';
 
-const { parse } = require('../../lib');
+import syntax from '../../lib/index.js';
+
+const { parse } = syntax;
 
 test('parses interpolation', (t) => {
   const root = parse('@{selector}:hover { @{prop}-size: @{color} }');

test/parser/mixins.test.js

@@ -1,6 +1,8 @@
-const test = require('ava');
+import test from 'ava';
 
-const { parse, nodeToString } = require('../../lib');
+import syntax from '../../lib/index.js';
+
+const { parse, nodeToString } = syntax;
 
 test('basic', (t) => {
   const params = '(  1, 2, 3; something, else; ...) when (@mode=huge)';

test/parser/params.test.js

@@ -1,6 +1,8 @@
-const test = require('ava');
+import test from 'ava';
 
-const { parse, nodeToString } = require('../../lib');
+import syntax from '../../lib/index.js';
+
+const { parse, nodeToString } = syntax;
 
 test('should not assign parameters for pseudo-selectors (#56)', (t) => {
   const less = '.test2:not(.test3) {}';

test/parser/postcss-sanity.test.js

@@ -1,7 +1,13 @@
-const test = require('ava');
+import { createRequire } from 'node:module';
+
+import test from 'ava';
+
+import syntax from '../../lib/index.js';
+
+const require = createRequire(import.meta.url);
 const { eachTest, jsonify } = require('postcss-parser-tests');
 
-const { nodeToString, parse, stringify } = require('../../lib');
+const { nodeToString, parse, stringify } = syntax;
 
 eachTest((name, code, json) => {
   // Skip comments.css, because we have an extended Comment node

test/parser/variables.test.js

@@ -1,7 +1,9 @@
-const test = require('ava');
-const AtRule = require('postcss/lib/at-rule');
+import test from 'ava';
+import AtRule from 'postcss/lib/at-rule';
 
-const { parse, nodeToString } = require('../../lib');
+import syntax from '../../lib/index.js';
+
+const { parse, nodeToString } = syntax;
 
 test('numeric', (t) => {
   const less = '@var: 1;';

test/postcss.test.js

@@ -1,8 +1,8 @@
-const test = require('ava');
-const postcss = require('postcss');
-const CssSyntaxError = require('postcss/lib/css-syntax-error');
+import test from 'ava';
+import postcss from 'postcss';
+import CssSyntaxError from 'postcss/lib/css-syntax-error';
 
-const syntax = require('../lib');
+import syntax from '../lib/index.js';
 
 const { parser } = syntax;
 

test/stringify.test.js

@@ -1,6 +1,6 @@
-const test = require('ava');
+import test from 'ava';
 
-const syntax = require('../lib');
+import syntax from '../lib/index.js';
 
 const { parse, nodeToString } = syntax;