@brightspace-ui/testing 0.27.2 → 0.28.1

package/README.md

@@ -351,7 +351,7 @@ TODO: note about leveraging @web/test-runner
 
 TODO
 
-## Developing & Contributing
+## Developing and Contributing
 
 After cloning the repo, run `npm install` to install its dependencies.
 
@@ -363,7 +363,7 @@ To run the full suite of tests:
 npm test
 ```
 
-Alternately, tests can be selectively run:
+Alternatively, tests can be selectively run:
 
 ```shell
 # binary unit tests
@@ -379,7 +379,7 @@ npm run test:server
 npm run test:vdiff
 ```
 
-### Versioning & Releasing
+### Versioning and Releasing
 
 This repo is configured to use `semantic-release`. Commits prefixed with `fix:` and `feat:` will trigger patch and minor releases when merged to `main`.
 

package/bin/d2l-test-runner.js

@@ -29,6 +29,7 @@ if (cli.subcommand === 'vdiff') {
 }
 
 async function runTests() {
+	runner.install();
 	const options = await runner.getOptions(argv);
 	await runner.start(options);
 }

package/package.json

@@ -2,7 +2,7 @@
   "name": "@brightspace-ui/testing",
   "description": "Utilities for testing front-end components and applications",
   "type": "module",
-  "version": "0.27.2",
+  "version": "0.28.1",
   "repository": "https://github.com/BrightspaceUI/testing.git",
   "scripts": {
     "lint": "eslint . --ext .js",

package/src/server/cli/test-runner.js

@@ -2,6 +2,7 @@ import { ConfigLoaderError, readConfig } from '@web/config-loader';
 import { DEFAULT_VDIFF_SLOW, WTRConfig } from '../wtr-config.js';
 import commandLineArgs from 'command-line-args';
 import commandLineUsage from 'command-line-usage';
+import { execSync } from 'node:child_process';
 import process from 'node:process';
 import { startTestRunner } from '@web/test-runner';
 
@@ -196,7 +197,12 @@ async function getTestRunnerOptions(argv = []) {
 	};
 }
 
+function installDeps() {
+	execSync('npx playwright install-deps', { stdio: 'pipe' });
+}
+
 export const runner = {
 	getOptions: getTestRunnerOptions,
+	install: installDeps,
 	start: startTestRunner
 };

package/src/server/report/result.js

@@ -195,11 +195,14 @@ export function renderBrowserResults(browser, tests, options) {
 			return acc;
 		}
 
-		let status = STATUS_TYPE.WARNING;
-		if (resultData.duration > resultData.info.slowDuration) {
-			status = STATUS_TYPE.ERROR;
-		} else if (resultData.duration < (resultData.info.slowDuration / 2)) {
-			status = STATUS_TYPE.NORMAL;
+		let status = STATUS_TYPE.NORMAL;
+		if (resultData.info) {
+			status = STATUS_TYPE.WARNING;
+			if (resultData.duration > resultData.info.slowDuration) {
+				status = STATUS_TYPE.ERROR;
+			} else if (resultData.duration < (resultData.info.slowDuration / 2)) {
+				status = STATUS_TYPE.NORMAL;
+			}
 		}
 
 		return acc.push(html`

package/src/server/visual-diff-reporter.js

@@ -70,6 +70,11 @@ function flattenResults(session, browserData, fileData) {
 		tests.forEach(t => {
 			const testName = `${prefix}${t.name}`;
 			const testKey = testName.replaceAll(' > ', ' ');
+			const info = getTestInfo(session, testKey);
+
+			// tests missing info but with no error were skipped via grep, so exclude them
+			if (!info && !t.error) return;
+
 			if (!fileData.tests.has(testName)) {
 				fileData.tests.set(testName, {
 					name: testName,
@@ -87,7 +92,7 @@ function flattenResults(session, browserData, fileData) {
 				duration: t.duration,
 				error: t.error?.message,
 				passed: t.passed,
-				info: getTestInfo(session, testKey)
+				info: info
 			});
 		});
 	}