Address PR feedback.

Author: johnbillion

Gruntfile.js

@@ -2168,8 +2168,8 @@ module.exports = function(grunt) {
 	grunt.registerTask( 'qunit', 'Runs QUnit tests.', function() {
 		var done = this.async();
 		grunt.util.spawn( {
-			cmd: 'node_modules/.bin/playwright',
-			args: [ 'test', '--config', 'tests/qunit/playwright.config.js' ],
+			cmd: 'npx',
+			args: [ 'playwright', 'test', '--config', 'tests/qunit/playwright.config.js' ],
 			opts: { stdio: 'inherit' }
 		}, function( error, result, code ) {
 			if ( code !== 0 ) {

tests/qunit/playwright.config.js

@@ -12,7 +12,8 @@ module.exports = defineConfig( {
 	workers: 1,
 	use: {
 		headless: true,
-		channel: 'chrome',
+		/* This avoids the need to run `npx playwright install` in CI. */
+		channel: process.env.CI ? 'chrome' : undefined,
 	},
 	reporter: process.env.CI ? 'github' : 'list',
 } );

tests/qunit/qunit.js

@@ -1,12 +1,24 @@
 const { test, expect } = require( '@playwright/test' );
+const fs = require( 'fs' );
 const path = require( 'path' );
-const glob = require( 'fast-glob' );
+const { pathToFileURL } = require( 'url' );
+
+function getHtmlFiles( dir ) {
+	const entries = fs.readdirSync( dir, { withFileTypes: true } );
+
+	return entries.flatMap( ( entry ) => {
+		const entryPath = path.join( dir, entry.name );
+
+		if ( entry.isDirectory() ) {
+			return getHtmlFiles( entryPath );
+		}
+
+		return entry.isFile() && entry.name.endsWith( '.html' ) ? [ entryPath ] : [];
+	} );
+}
 
 const qunitDir = path.resolve( __dirname );
-const htmlFiles = glob.sync( [ '**/*.html' ], {
-	cwd: qunitDir,
-	absolute: true,
-} );
+const htmlFiles = getHtmlFiles( qunitDir );
 
 for ( const file of htmlFiles ) {
 	const name = path.relative( qunitDir, file );
@@ -51,7 +63,7 @@ for ( const file of htmlFiles ) {
 		} );
 
 		// Navigate to the test file.
-		await page.goto( 'file://' + file, { waitUntil: 'domcontentloaded' } );
+		await page.goto( pathToFileURL( file ).href, { waitUntil: 'domcontentloaded' } );
 
 		// Wait for QUnit to complete.
 		const results = await page.evaluate( () => window.__qunitResults );