testaro 33.1.3 → 34.0.0

package/README.md

@@ -424,7 +424,7 @@ When you include a `rules` property, you limit the tests of the tool that are pe
 
 The `nuVal`, `qualWeb`, and `testaro` tools require specific formats for the `rules` property. Those formats are described below in the sections about those tools.
 
-The `aslint` and `ed11y` tools do not yet allow rule specification.
+The `ed11y` tool does not yet allow rule specification.
 
 ###### Examples
 

package/package.json

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

package/tests/aslint.js

@@ -43,7 +43,7 @@ exports.reporter = async (page, options) => {
     `${__dirname}/../node_modules/aslint-testaro/aslint.bundle.js`, 'utf8'
   );
   // Get the nonce, if any.
-  const {report} = options;
+  const {act, report} = options;
   const {jobData} = report;
   const scriptNonce = jobData && jobData.lastScriptNonce;
   // Inject the ASLint bundle and runner into the page.
@@ -94,10 +94,16 @@ exports.reporter = async (page, options) => {
     const actReport = await reportLoc.textContent();
     // Populate the act report.
     result = JSON.parse(actReport);
-    // Delete irrelevant properties from the tool report details.
+    // If any rules were reported violated:
     if (result.rules) {
+      // For each such rule:
       Object.keys(result.rules).forEach(ruleID => {
-        if (['passed', 'skipped'].includes(result.rules[ruleID].status.type)) {
+        // If the rule was passed or skipped or rules to be tested were specified and exclude it:
+        if (
+          ['passed', 'skipped'].includes(result.rules[ruleID].status.type)
+          || act.rules && ! act.rules.includes(ruleID)
+        ) {
+          // Delete the rule report.
           delete result.rules[ruleID];
         }
       });