@enact/ui-test-utils 4.0.1 → 4.0.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@enact/ui-test-utils",
-  "version": "4.0.1",
+  "version": "4.0.3",
   "description": "UI Testing for the Enact framework",
   "repository": "https://github.com/enactjs/ui-test-utils",
   "type": "module",
@@ -23,38 +23,37 @@
   "author": "Roy Sutton <roy.sutton@lge.com>",
   "license": "Apache-2.0",
   "engines": {
-    "node": "^20.12.0 || ^22.0.0 || >=24.0.0"
+    "node": "^20.19.0 || ^22.0.0 || >=24.0.0"
   },
   "publishConfig": {
     "access": "public"
   },
   "dependencies": {
-    "@wdio/cli": "^9.21.1",
-    "@wdio/dot-reporter": "^9.20.0",
-    "@wdio/local-runner": "^9.21.0",
-    "@wdio/mocha-framework": "^9.21.0",
-    "@wdio/spec-reporter": "^9.20.0",
-    "@wdio/static-server-service": "^9.20.0",
-    "@wdio/visual-service": "^9.0.2",
+    "@wdio/cli": "^9.27.2",
+    "@wdio/dot-reporter": "^9.27.2",
+    "@wdio/local-runner": "^9.27.2",
+    "@wdio/mocha-framework": "^9.27.2",
+    "@wdio/spec-reporter": "^9.27.2",
+    "@wdio/static-server-service": "^9.27.2",
+    "@wdio/visual-service": "^9.2.3",
     "chalk": "^5.6.2",
     "cross-spawn": "^7.0.6",
-    "expect-webdriverio": "^5.5.0",
-    "fs-extra": "^11.3.2",
+    "expect-webdriverio": "^5.6.8",
+    "fs-extra": "^11.3.5",
     "minimist": "^1.2.8",
-    "query-string": "^7.1.3",
-    "ramda": "^0.31.3",
-    "readdirp": "^3.6.0",
+    "query-string": "^9.4.0",
+    "ramda": "^0.32.0",
+    "readdirp": "^5.0.0",
     "wdio-docker-service": "^3.2.1",
-    "webdriverio": "^9.21.0"
+    "webdriverio": "^9.27.2"
   },
   "devDependencies": {
-    "eslint": "^9.39.1",
-    "eslint-config-enact": "^5.0.3",
-    "globals": "^16.5.0"
+    "eslint": "^9.39.4",
+    "eslint-config-enact": "^5.1.2",
+    "globals": "^17.6.0"
   },
   "overrides": {
-    "cross-spawn": "$cross-spawn",
-    "express": "^4.21.2",
-    "ws": "^8.18.0"
+    "serialize-javascript": "^7.0.4",
+    "simple-xml-to-json": "1.2.4"
   }
 }

package/screenshot/utils/confHelpers.js

@@ -32,6 +32,7 @@ const baselineRelativePath = 'screenshots/reference';
 const screenshotRelativePath = 'screenshots/screen';
 const baselineFolder = path.join(distPath, baselineRelativePath);
 const screenshotFolder = path.join(distPath, screenshotRelativePath);
+const pendingFailuresFolder = path.join(distPath, 'pending-failures');
 
 const generateReferenceName = getScreenshotName(baselineFolder);
 
@@ -55,6 +56,12 @@ function onPrepare () {
 	global.sessionFailures = new Map();
 	global.failedSessions = new Set();
 
+	// Clear any leftover pending-failure files from a previous run
+	if (fs.existsSync(pendingFailuresFolder)) {
+		fs.rmSync(pendingFailuresFolder, {recursive: true, force: true});
+	}
+	fs.mkdirSync(pendingFailuresFolder, {recursive: true});
+
 	if (!fs.existsSync('tests/screenshot/dist/screenshots/reference')) {
 		console.log('No reference screenshots found, creating new references!');
 	}
@@ -263,38 +270,30 @@ async function afterTest (testData, _context, {error, passed}) {
 			});
 		}
 
-		if (!passed) {
-			// Track failed tests to avoid duplicate logging during retries
-			if (!global.loggedFailures) {
-				global.loggedFailures = new Set();
-			}
+		// Use a stable filename derived from the test identifier so the worker can find and
+		// delete the right file on a passing retry, even across different worker processes.
+		const testIdentifier = testData.title + '::' + fileName;
+		const pendingKey = cryptoModule.createHash('md5').update(testIdentifier).digest('hex');
+		const pendingFile = path.join(pendingFailuresFolder, `${pendingKey}.json`);
 
-			const testIdentifier = testData.title + '::' + fileName;
-
-			// Only log if we haven't already logged this test failure
-			if (!global.loggedFailures.has(testIdentifier)) {
-				global.loggedFailures.add(testIdentifier);
-
-				const screenPath = path.join(screenshotRelativePath, 'actual', fileName);
-				const diffPath = path.join(screenshotRelativePath, 'diff', fileName);
-				fs.open(failedScreenshotFilename, 'a', (err, fd) => {
-					if (err) {
-						console.error('Unable to create failed test log file!');
-					} else {
-						const title = testData.title.replace(/~\//g, '/');
-						const {params, url} = testData.context;
-						const output = {title, diffPath, referencePath, screenPath, params, url};
-						fs.appendFile(fd, `${JSON.stringify(output)},`, 'utf8', () => {
-							fs.close(fd);
-						});
-					}
-				});
+		if (!passed) {
+			// Track pending failed tests to avoid duplicate logging during retries
+			const screenPath = path.join(screenshotRelativePath, 'actual', fileName);
+			const diffPath = path.join(screenshotRelativePath, 'diff', fileName);
+			const title = testData.title.replace(/~\//g, '/');
+			const {params, url} = testData.context;
+			const output = {title, diffPath, referencePath, screenPath, params, url};
+			try {
+				fs.writeFileSync(pendingFile, JSON.stringify(output), 'utf8');
+			} catch (writeErr) {
+				console.error(`Unable to stage failure for "${title}": ${writeErr.message}`);
 			}
 		} else {
-			// Test passed on retry - remove from logged failures
-			if (global.loggedFailures) { // eslint-disable-line no-lonely-if
-				const testIdentifier = testData.title + '::' + fileName;
-				global.loggedFailures.delete(testIdentifier);
+			// Test passed (possibly on retry) — remove the staged failure so it won't be reported
+			try {
+				fs.unlinkSync(pendingFile);
+			} catch (e) {
+				// test passed on first attempt — nothing to do
 			}
 		}
 	}
@@ -303,6 +302,20 @@ async function afterTest (testData, _context, {error, passed}) {
 }
 
 function onComplete () {
+	// Write all tests that ultimately failed
+	if (fs.existsSync(pendingFailuresFolder)) {
+		const pendingFiles = fs.readdirSync(pendingFailuresFolder).filter(f => f.endsWith('.json'));
+		for (const file of pendingFiles) {
+			try {
+				const raw = fs.readFileSync(path.join(pendingFailuresFolder, file), 'utf8');
+				fs.appendFileSync(failedScreenshotFilename, `${raw},`, 'utf8');
+			} catch (e) {
+				console.error(`Failed to flush pending failure ${file}: ${e.message}`);
+			}
+		}
+		fs.rmSync(pendingFailuresFolder, {recursive: true, force: true});
+	}
+
 	const {size: newSize} = fs.statSync(newScreenshotFilename),
 		{size: failedSize} = fs.statSync(failedScreenshotFilename);
 

package/src/build-apps.js

@@ -3,18 +3,20 @@ import path from 'path';
 let chalk;
 import spawn from 'cross-spawn';
 import fs from 'fs-extra';
-import readdirp from 'readdirp';
+import {readdirpPromise} from 'readdirp';
 import * as url from 'url';
 
 const env = {
 	ILIB_BASE_PATH: '/framework/ilib',
 	ILIB_ASSET_CREATE: 'false',
 	SIMPLE_CSS_IDENT: 'true',
-	BROWSERSLIST: 'Chrome 79'
+	BROWSERSLIST: 'Chrome 132'
 };
 
 function findViews (base) {
-	return readdirp.promise(path.join('tests', base, 'apps'), {fileFilter: '*-View.js'});
+	return readdirpPromise(path.join('tests', base, 'apps'), {
+		fileFilter: (entry) => entry.basename.endsWith('-View.js')
+	});
 }
 
 // eslint-disable-next-line no-shadow

package/utils/Page.js

@@ -12,7 +12,7 @@ export class Page {
 
 	async open (appPath, urlExtra = '?locale=en-US') {
 		await browser.execute(function () {
-			document.body.innerHTML = '';
+			document.body.textContent = '';
 		});
 
 		this._url = `/${appPath}/${urlExtra}`;

package/.travis.yml

@@ -1,13 +0,0 @@
-dist: jammy
-language: node_js
-node_js:
-    - 21
-before_install:
-    - sudo apt-get update
-    - sudo apt-get install build-essential libcairo2-dev libpango1.0-dev libjpeg-dev libgif-dev librsvg2-dev
-install:
-    - npm install
-script:
-    - echo -e "\x1b\x5b35;1m*** Linting...\x1b\x5b0m"
-    - npm run lint -- --report-unused-disable-directives --max-warnings 0
-    - echo -e "\x1b\x5b35;1m*** Linting complete...\x1b\x5b0m"