@brightspace-ui/testing 0.28.0 → 0.28.1

package/package.json

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

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
 			});
 		});
 	}