#64894 Switch from Puppeteer to Playwright for the QUnit test runner

.github/workflows/reusable-javascript-tests.yml

@@ -5,12 +5,6 @@ name: JavaScript tests
 
 on:
   workflow_call:
-    inputs:
-      disable-apparmor:
-        description: 'Whether to disable AppArmor.'
-        required: false
-        type: 'boolean'
-        default: false
 
 # Disable permissions for all available scopes by default.
 # Any needed permissions should be configured at the job level.
@@ -55,15 +49,6 @@ jobs:
       - name: Install npm Dependencies
         run: npm ci
 
-      # Older branches using outdated versions of Puppeteer fail on newer versions of the `ubuntu-24` image.
-      # This disables AppArmor in order to work around those failures.
-      #
-      # See https://issues.chromium.org/issues/373753919
-      # and https://chromium.googlesource.com/chromium/src/+/main/docs/security/apparmor-userns-restrictions.md
-      - name: Disable AppArmor
-        if: ${{ inputs.disable-apparmor }}
-        run: echo 0 | sudo tee /proc/sys/kernel/apparmor_restrict_unprivileged_userns
-
       - name: Run QUnit tests
         run: npm run grunt qunit:compiled
 

Gruntfile.js

@@ -115,7 +115,6 @@ module.exports = function(grunt) {
 			'cssmin',
 			'imagemin',
 			'jshint',
-			'qunit',
 			'uglify',
 			'watch'
 		],
@@ -906,7 +905,8 @@ module.exports = function(grunt) {
 				src: [
 					'tests/qunit/**/*.js',
 					'!tests/qunit/vendor/*',
-					'!tests/qunit/editor/**'
+					'!tests/qunit/qunit.js',
+					'!tests/qunit/playwright.config.js'
 				],
 				options: grunt.file.readJSON( 'tests/qunit/.jshintrc' )
 			},
@@ -1010,12 +1010,6 @@ module.exports = function(grunt) {
 				}
 			}
 		},
-		qunit: {
-			files: [
-				'tests/qunit/**/*.html',
-				'!tests/qunit/editor/**'
-			]
-		},
 		phpunit: {
 			'default': {
 				args: ['--verbose', '-c', 'phpunit.xml.dist']
@@ -1540,8 +1534,7 @@ module.exports = function(grunt) {
 			},
 			test: {
 				files: [
-					'tests/qunit/**',
-					'!tests/qunit/editor/**'
+					'tests/qunit/**'
 				],
 				tasks: ['qunit']
 			}
@@ -2172,6 +2165,20 @@ module.exports = function(grunt) {
 		}, this.async());
 	});
 
+	grunt.registerTask( 'qunit', 'Runs QUnit tests.', function() {
+		var done = this.async();
+		grunt.util.spawn( {
+			cmd: 'npx',
+			args: [ 'playwright', 'test', '--config', 'tests/qunit/playwright.config.js' ],
+			opts: { stdio: 'inherit' }
+		}, function( error, result, code ) {
+			if ( code !== 0 ) {
+				grunt.fail.warn( 'QUnit tests failed.' );
+			}
+			done();
+		} );
+	} );
+
 	grunt.registerTask( 'qunit:compiled', 'Runs QUnit tests on compiled as well as uncompiled scripts.',
 		['build', 'copy:qunit', 'qunit']
 	);

package.json

@@ -53,7 +53,6 @@
 		"grunt-contrib-cssmin": "~5.0.0",
 		"grunt-contrib-imagemin": "~4.0.0",
 		"grunt-contrib-jshint": "3.2.0",
-		"grunt-contrib-qunit": "~10.1.1",
 		"grunt-contrib-uglify": "~5.2.2",
 		"grunt-contrib-watch": "~1.1.0",
 		"grunt-file-append": "0.0.7",

tests/qunit/playwright.config.js

@@ -0,0 +1,19 @@
+/**
+ * Playwright configuration for QUnit tests.
+ */
+const path = require( 'path' );
+const { defineConfig } = require( '@playwright/test' );
+
+module.exports = defineConfig( {
+	testDir: __dirname,
+	outputDir: path.join( __dirname, '..', '..', 'artifacts', 'test-results' ),
+	testMatch: 'qunit.js',
+	timeout: 30_000,
+	workers: 1,
+	use: {
+		headless: true,
+		/* This avoids the need to run `npx playwright install` in CI. */
+		channel: process.env.CI ? 'chrome' : undefined,
+	},
+	reporter: process.env.CI ? 'github' : 'list',
+} );