testaro 57.1.0 → 57.2.0

package/actSpecs.js

@@ -147,7 +147,7 @@ exports.actSpecs = {
       {
         which: [true, 'string', 'isTest', 'tool name'],
         launch: [false, 'object', '', 'new target, browserID, and/or what, if any'],
-        rules: [false, 'array', 'areStrings', 'rule IDs or (for nuVal) specifications, if not all']
+        rules: [false, 'array', 'areStrings', 'rule IDs or (for testaro or nuVal) specifications, if not all']
       }
     ],
     text: [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "testaro",
-  "version": "57.1.0",
+  "version": "57.2.0",
   "description": "Run 1000 web accessibility tests from 11 tools and get a standardized report",
   "main": "index.js",
   "scripts": {

package/procs/target.js

@@ -44,7 +44,7 @@ exports.isTooSmall = async (loc, min) => {
       // Get its styles.
       const styleDec = window.getComputedStyle(el);
       const displayStyle = styleDec.display;
-      // If it is hidden:
+      // If it is hidden (which should be impossible because of the selector):
       if (displayStyle === 'none') {
         // Exempt it.
         return null;
@@ -67,22 +67,29 @@ exports.isTooSmall = async (loc, min) => {
         const elDims = getDims(el);
         // If either dimension is too small:
         if (elDims.some(dim => dim < min)) {
-          // Get the parent of the target.
+          // Get the parent element of the target.
           const elParent = el.parentElement;
-          // Get the child elements of the parent, including the target.
-          const elGeneration = Array.from(elParent.children);
-          // If the target has no siblings of its type:
-          if (elGeneration.filter(peer => peer.tagName === tagName).length === 1) {
-            // Get the width and height of the parent.
-            const parentDims = getDims(elParent);
-            // For each dimension of the target:
-            elDims.forEach((elDim, index) => {
-              // If it is too small and smaller than that of the parent:
-              if (elDim < min && parentDims[index] > elDim) {
-                // Replace it with the dimension of the parent.
-                elDims[index] = parentDims[index];
-              }
-            });
+          // If the parent element exists:
+          if (elParent) {
+            // Get the child elements of the parent, including the target.
+            const elGeneration = Array.from(elParent.children);
+            // If the target has no siblings of its type:
+            if (elGeneration.filter(peer => peer.tagName === tagName).length === 1) {
+              // Get the width and height of the parent.
+              const parentDims = getDims(elParent);
+              // For each dimension of the target:
+              elDims.forEach((elDim, index) => {
+                // If it is too small and smaller than that of the parent:
+                if (elDim < min && parentDims[index] > elDim) {
+                  // Replace it with the dimension of the parent (a substitute for distance).
+                  elDims[index] = parentDims[index];
+                }
+              });
+            }
+          }
+          // Otherwise, i.e. if it does not exist:
+          else {
+            console.log(`WARNING: Target ${tagName} (${el.innerHTML}) has no parent element`);
           }
         }
         // Get whether it is too small.

package/run.js

@@ -1414,11 +1414,14 @@ exports.doJob = async job => {
       }
     });
     // Perform the acts and get a report.
+    console.log('Performing the job acts');
     report = await doActs(report, 0, null);
     // Add the end time and duration to the report.
     const endTime = new Date();
     report.jobData.endTime = nowString();
-    report.jobData.elapsedSeconds =  Math.floor((endTime - startTime) / 1000);
+    const elapsedSeconds = Math.floor((endTime - startTime) / 1000);
+    report.jobData.elapsedSeconds =  elapsedSeconds;
+    console.log(`Elapsed seconds: ${elapsedSeconds}`);
     // Consolidate and sort the tool times.
     const {toolTimes} = report.jobData;
     const toolTimeData = Object

package/tests/wax.js

@@ -36,7 +36,7 @@ const waxDev = {runWax};
 // FUNCTIONS
 
 // Conducts and reports the WAX tests.
-exports.reporter = async (page, report, actIndex, timeLimit) => {
+exports.reporter = async (page, report, actIndex) => {
   // Initialize the act report.
   let data = {};
   let result = {};
@@ -56,17 +56,43 @@ exports.reporter = async (page, report, actIndex, timeLimit) => {
       data.prevented = true;
       data.error = actReport;
     }
-    // Otherwise, if it failed with an object report:
-    else if (typeof actReport === 'object' && actReport.responseCode === 500) {
-      // Report this.
-      data.prevented = true;
-      data.error = actReport.message || 'response status code 500';
+    // Otherwise, if the report is an array:
+    else if (Array.isArray(actReport)) {
+      // If it is an error report:
+      if (actReport.length === 1 && typeof actReport[0] === 'object' && actReport[0].error) {
+        // Report this.
+        data.prevented = true;
+        const {error} = actReport[0];
+        data.error = error;
+        console.log(`ERROR running wax: ${error}`);
+      }
+      // Otherwise, i.e. if it is a successful report:
+      else {
+        // Populate the act report.
+        result = {
+          violations: actReport
+        }
+      }
     }
-    // Otherwise, i.e. if WAX succeeded:
-    else {
-      // Populate the act report.
-      result = {
-        violations: actReport
+    // Otherwise, if the report is a non-array object:
+    else if (typeof actReport === 'object') {
+      // If the response status was a system error:
+      if(actReport.responseCode === 500) {
+        // Report this.
+        data.prevented = true;
+        data.error = actReport.message || 'response status code 500';
+      }
+      // Otherwise, if there was an error:
+      else if (actReport.error) {
+        // Report this.
+        data.prevented = true;
+        data.error = actReport.error;
+      }
+      // In any other case:
+      else {
+        // Report a prevention.
+        data.prevented = true;
+        data.error = 'wax failure';
       }
     }
     // Return the results.