testaro 28.0.0 → 28.0.2

package/README.md

@@ -284,6 +284,8 @@ If a tool has the option to be used without itemization and is being so used, th
 This standard format reflects some judgments. For example:
 - The `ordinalSeverity` property of an instance may have required interpretation. Tools may report severity, certainty, priority, or some combination of those. They may use ordinal or metric quantifications. If they quantify ordinally, their scales may have more or fewer than 4 ranks. Testaro coerces each tool’s severity, certainty, and/or priority classification into a 4-rank ordinal classification. This classification is deemed to express the most common pattern among the tools.
 - The `tagName` property of an instance may not always be obvious, because in some cases the rule being tested for requires a relationship among more than one element (e.g., “An X element may not have a Y element as its parent”).
+- The `ruleID` property of an instance is a matching rule if the tool issues a message but no rule identifier for each instance. The `nuVal` tool does this. In this case, Testaro is classifying the messages into rules.
+- The `ruleID` property of an instance may reclassify tool rules. For example, if a tool rule covers multiple situations that are dissimilar (such as having dissimilar user impacts), that rule may be split into multiple rules with distinct `ruleID` properties.
 
 You are not dependent on the judgments incorporated into the standard format, because Testaro can give you the original reports from the tools.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "testaro",
-  "version": "28.0.0",
+  "version": "28.0.2",
   "description": "Run 650 web accessibility tests from 8 tools and get a standardized report",
   "main": "index.js",
   "scripts": {

package/procs/aslint.js

@@ -9,14 +9,13 @@ const options = {
   reportFormat: {JSON: true},
   watchDomChanges: false
 };
-const bundleEl = document.getElementById('aslintBundle');
 window
 .aslint
 .config(options)
 .addListener('onValidatorStarted', function () {
   console.log('@ Validator started');
 })
-.addListener('onValidatorComplete', function (error, report) {
+.addListener('onValidatorComplete', function () {
   console.log('@ Validator Complete');
 })
 .addFilter('onBeforeRuleReport', function (report) {
@@ -42,7 +41,7 @@ window
           .html
           .replace(/&lt;/g, '<')
           .replace(/&gt;/g, '>')
-          .replace(/&quot;/g, '\"')
+          .replace(/&quot;/g, '"')
           .replace(/&#x3D;/g, '=')
           .replace(/&#x2F;/g, '/')
           .replace(/&#39;/g, '\'')
@@ -55,7 +54,7 @@ window
           .description
           .replace(/&lt;/g, '<')
           .replace(/&gt;/g, '>')
-          .replace(/&quot;/g, '\"')
+          .replace(/&quot;/g, '"')
           .replace(/&#x3D;/g, '=')
           .replace(/&#x2F;/g, '/')
           .replace(/&amp;(?:amp;)*/g, '&')

package/procs/standardize.js

@@ -268,6 +268,10 @@ const convert = (toolName, result, standardResult) => {
       const ruleResults = result.rules[ruleID].results;
       if (ruleResults && ruleResults.length) {
         ruleResults.forEach(ruleResult => {
+          const what = ruleResult.message.actual.description;
+          if (ruleID === 'misused-required-attribute' && what.includes('not needed')) {
+            ruleID = 'misused-required-attributeR';
+          }
           const {issueType} = result.rules[ruleID];
           const xpath = ruleResult.element && ruleResult.element.xpath || '';
           const tagName = xpath && xpath.replace(/^.*\//, '').replace(/[^-\w].*$/, '').toUpperCase()
@@ -279,7 +283,7 @@ const convert = (toolName, result, standardResult) => {
             : '';
           const instance = {
             ruleID,
-            what: ruleResult.message.actual.description,
+            what,
             ordinalSeverity: ['warning', 0, 0, 'error'].indexOf(issueType),
             tagName,
             id,