testaro 57.3.0 → 57.4.0

package/package.json

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

package/procs/doTestAct.js

@@ -54,7 +54,7 @@ const doTestAct = async () => {
   // Get the saved report.
   const reportJSON = await fs.readFile(reportPath, 'utf8');
   const report = JSON.parse(reportJSON);
-  // Get the act.
+  // Get a reference to the act in the report.
   const act = report.acts[actIndex];
   // Get the tool name.
   const {which} = act;
@@ -94,7 +94,7 @@ const doTestAct = async () => {
           act.data = actReport.data;
           act.result = actReport.result;
           // If the tool reported that the page prevented testing:
-          if (actReport.data && actReport.data.prevented) {
+          if (act.data && act.data.prevented) {
             // Add prevention data to the job data.
             report.jobData.preventions[which] = act.data.error;
           }
@@ -118,8 +118,19 @@ const doTestAct = async () => {
     }
     // Otherwise, i.e. if the page does not exist:
     else {
+      // Add data to the act.
+      act.data ??= {};
+      act.data.prevented = true;
+      act.data.error = 'No page';
+      // Add prevention data to the job data.
+      report.jobData.preventions[which] = act.data.error;
+      // Save the revised report.
+      const reportJSON = JSON.stringify(report);
+      await fs.writeFile(reportPath, reportJSON);
       // Report this.
-      process.send('ERROR: No page');
+      const message = 'ERROR: No page';
+      console.log(message);
+      process.send(message);
     }
   }
 };

package/run.js

@@ -174,7 +174,7 @@ const goTo = async (report, page, url, timeout, waitUntil) => {
     // Otherwise, if the response status was rejection of excessive requests:
     else if (httpStatus === 429) {
       // Return this.
-      console.log(`Visit to ${url} prevented by request frequency restriction (status 429)`);
+      console.log(`ERROR: Visit to ${url} prevented by request frequency limit (status 429)`);
       return {
         success: false,
         error: 'status429'
@@ -633,8 +633,9 @@ const doActs = async (report) => {
         // If this failed:
         if (page.prevented) {
           // Add this to the act.
-          act.prevented = true;
-          act.error = page.error || '';
+          act.data ??= {};
+          act.data.prevented = true;
+          act.data.error = page.error || '';
         }
       }
       // Otherwise, if the act is a test act:
@@ -708,8 +709,9 @@ const doActs = async (report) => {
               // If this failed:
               if (page.prevented) {
                 // Add this to the act.
-                act.prevented = true;
-                act.error = page.error || '';
+                act.data ??= {};
+                act.data.prevented = true;
+                act.data.error = page.error || '';
               }
               // Otherwise, i.e. if it succeeded:
               else {
@@ -782,7 +784,7 @@ const doActs = async (report) => {
             act.result.url = page.url();
             // If a prohibited redirection occurred:
             if (response.exception === 'badRedirection') {
-              // Report this and abort the job.
+              // Report this.
               addError(true, false, report, actIndex, 'ERROR: Navigation illicitly redirected');
             }
           }

package/tests/ed11y.js

@@ -187,10 +187,12 @@ exports.reporter = async (page, report, actIndex, timeLimit) => {
       };
     }
     // Return the data and result, discarding the separate element data.
+    const data = {};
+    if (result.prevented) {
+      data.prevented = true;
+    }
     return {
-      data: {
-        prevented: result.prevented
-      },
+      data,
       result
     };
   }
@@ -198,12 +200,8 @@ exports.reporter = async (page, report, actIndex, timeLimit) => {
   else {
     // Return this.
     return {
-      data: {
-        prevented: false
-      },
-      result: {
-        prevented: false,
-      }
+      data: {},
+      result: {}
     }
   }
 };