testilo 44.2.0 → 44.2.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "testilo",
-  "version": "44.2.0",
+  "version": "44.2.2",
   "description": "Prepares Testaro jobs and processes Testaro reports",
   "main": "call.js",
   "scripts": {

package/procs/score/tic.js

@@ -1376,6 +1376,11 @@ exports.issues = {
           variable: false,
           quality: 1,
           what: 'img element with alt="" has a role attribute'
+        },
+        'An img element with a role attribute must not have an alt attribute whose value is the empty string.': {
+          variable: false,
+          quality: 1,
+          what: 'img element with a role attribute has alt=""'
         }
       },
       nuVnu: {
@@ -1383,6 +1388,11 @@ exports.issues = {
           variable: false,
           quality: 1,
           what: 'img element with alt="" has a role attribute'
+        },
+        'An img element with a role attribute must not have an alt attribute whose value is the empty string.': {
+          variable: false,
+          quality: 1,
+          what: 'img element with a role attribute has alt=""'
         }
       },
       qualWeb: {
@@ -4040,6 +4050,16 @@ exports.issues = {
           variable: false,
           quality: 1,
           what: 'Element is script and has a src attribute but its type is not empty, a JS MIME type, or module'
+        },
+        'A script element with a type attribute whose value is neither a JavaScript MIME type, module, importmap, nor speculationrules (i.e., a data block) must not have a defer attribute.': {
+          variable: false,
+          quality: 1,
+          what: 'Element is not eligible for a defer attribute but has one'
+        },
+        'A script element with a type attribute whose value is neither a JavaScript MIME type, module, importmap, nor speculationrules (i.e., a data block) must not have an async attribute.': {
+          variable: false,
+          quality: 1,
+          what: 'Element is not eligible for an async attribute but has one'
         }
       },
       nuVnu: {
@@ -4057,6 +4077,38 @@ exports.issues = {
           variable: false,
           quality: 1,
           what: 'Element is script and has a src attribute but its type is not empty, a JS MIME type, or module'
+        },
+        'A script element with a type attribute whose value is neither a JavaScript MIME type, module, importmap, nor speculationrules (i.e., a data block) must not have a defer attribute.': {
+          variable: false,
+          quality: 1,
+          what: 'Element is not eligible for a defer attribute but has one'
+        },
+        'A script element with a type attribute whose value is neither a JavaScript MIME type, module, importmap, nor speculationrules (i.e., a data block) must not have an async attribute.': {
+          variable: false,
+          quality: 1,
+          what: 'Element is not eligible for an async attribute but has one'
+        }
+      }
+    }
+  },
+  specRulesScriptBad: {
+    summary: 'speculation rules script element invalid',
+    why: 'Document navigation performs poorly',
+    wcag: '1.3.1',
+    weight: 1,
+    tools: {
+      nuVal: {
+        'The href_matches property in a document rule must be a string.': {
+          variable: false,
+          quality: 1,
+          what: 'Element is script with type=speculationrules but its href_matches value is not a string'
+        }
+      },
+      nuVnu: {
+        'The href_matches property in a document rule must be a string.': {
+          variable: false,
+          quality: 1,
+          what: 'Element is script with type=speculationrules but its href_matches value is not a string'
         }
       }
     }
@@ -4515,35 +4567,10 @@ exports.issues = {
           quality: 1,
           what: 'Page includes more than one autofocus attribute'
         },
-        'A link element with a sizes attribute must have a rel attribute that contains the value icon or the value apple-touch-icon or the value apple-touch-icon-precomposed.': {
-          variable: false,
-          quality: 1,
-          what: 'link element has a sizes attribute but no icon-type rel attribute'
-        },
         'An input element with a type attribute whose value is hidden must not have any aria-* attributes.': {
           variable: false,
           quality: 1,
           what: 'hidden-type input element has an ARIA attribute'
-        },
-        'The sizes attribute may be specified only if the srcset attribute is also present.': {
-          variable: false,
-          quality: 1,
-          what: 'Element has a sizes attribute but no srcset attribute'
-        },
-        'The sizes attribute must only be specified if the srcset attribute is also specified.': {
-          variable: false,
-          quality: 1,
-          what: 'Element has a sizes attribute but no srcset attribute'
-        },
-        'When the srcset attribute has any image candidate string with a width descriptor, the sizes attribute must also be present.': {
-          variable: false,
-          quality: 1,
-          what: 'Element with a srcset attribute with a width has no sizes attribute'
-        },
-        'When the srcset attribute has any image candidate string with a width descriptor, the sizes attribute must also be specified.': {
-          variable: false,
-          quality: 1,
-          what: 'Element with a srcset attribute with a width has no valid sizes attribute'
         }
       },
       nuVnu: {
@@ -4582,15 +4609,37 @@ exports.issues = {
           quality: 1,
           what: 'Page includes more than one autofocus attribute'
         },
-        'A link element with a sizes attribute must have a rel attribute that contains the value icon or the value apple-touch-icon or the value apple-touch-icon-precomposed.': {
+        'An input element with a type attribute whose value is hidden must not have any aria-* attributes.': {
           variable: false,
           quality: 1,
-          what: 'link element has a sizes attribute but no icon-type rel attribute'
+          what: 'hidden-type input element has an ARIA attribute'
+        }
+      },
+      wax: {
+        'Elements must only use allowed ARIA attributes': {
+          variable: false,
+          quality: 1,
+          what: 'Element has an ARIA attribute that is not allowed'
         },
-        'An input element with a type attribute whose value is hidden must not have any aria-* attributes.': {
+        'Ensure ARIA attributes used are permitted for the element\'s role.': {
           variable: false,
           quality: 1,
-          what: 'hidden-type input element has an ARIA attribute'
+          what: 'Element has an ARIA attribute that is invalid for the role of the element'
+        }
+      }
+    }
+  },
+  sizesAttributeBad: {
+    summary: 'sizes attribute invalid',
+    why: 'Item behaves improperly',
+    wcag: '1.3.1',
+    weight: 4,
+    tools: {
+      nuVal: {
+        'A link element with a sizes attribute must have a rel attribute that contains the value icon or the value apple-touch-icon or the value apple-touch-icon-precomposed.': {
+          variable: false,
+          quality: 1,
+          what: 'link element has a sizes attribute but no icon-type rel attribute'
         },
         'The sizes attribute may be specified only if the srcset attribute is also present.': {
           variable: false,
@@ -4611,20 +4660,45 @@ exports.issues = {
           variable: false,
           quality: 1,
           what: 'Element with a srcset attribute with a width has no valid sizes attribute'
+        },
+        'The sizes attribute value starting with auto is only valid for lazy-loaded images. Add loading=lazy to this element.': {
+          variable: false,
+          quality: 1,
+          what: 'Element with a sizes=auto… attribute has no loading=lazy attribute'
         }
       },
-      wax: {
-        'Elements must only use allowed ARIA attributes': {
+      nuVnu: {
+        'A link element with a sizes attribute must have a rel attribute that contains the value icon or the value apple-touch-icon or the value apple-touch-icon-precomposed.': {
           variable: false,
           quality: 1,
-          what: 'Element has an ARIA attribute that is not allowed'
+          what: 'link element has a sizes attribute but no icon-type rel attribute'
         },
-        'Ensure ARIA attributes used are permitted for the element\'s role.': {
+        'The sizes attribute may be specified only if the srcset attribute is also present.': {
           variable: false,
           quality: 1,
-          what: 'Element has an ARIA attribute that is invalid for the role of the element'
+          what: 'Element has a sizes attribute but no srcset attribute'
+        },
+        'The sizes attribute must only be specified if the srcset attribute is also specified.': {
+          variable: false,
+          quality: 1,
+          what: 'Element has a sizes attribute but no srcset attribute'
+        },
+        'When the srcset attribute has any image candidate string with a width descriptor, the sizes attribute must also be present.': {
+          variable: false,
+          quality: 1,
+          what: 'Element with a srcset attribute with a width has no sizes attribute'
+        },
+        'When the srcset attribute has any image candidate string with a width descriptor, the sizes attribute must also be specified.': {
+          variable: false,
+          quality: 1,
+          what: 'Element with a srcset attribute with a width has no valid sizes attribute'
+        },
+        'The sizes attribute value starting with auto is only valid for lazy-loaded images. Add loading=lazy to this element.': {
+          variable: false,
+          quality: 1,
+          what: 'Element with a sizes=auto… attribute has no loading=lazy attribute'
         }
-      },
+      }
     }
   },
   attributeValueBad: {
@@ -4703,6 +4777,28 @@ exports.issues = {
       }
     }
   },
+  attributeValueRisk: {
+    summary: 'attribute value bad?',
+    why: 'Item may behave improperly',
+    wcag: '4.1.2',
+    weight: 1,
+    tools: {
+      nuVal: {
+        '^Potentially bad value .+ for attribute .+ on element .+Typo for .+\?.*$': {
+          variable: true,
+          quality: 1,
+          what: 'Attribute value may be a typographical error'
+        }
+      },
+      nuVnu: {
+        '^Potentially bad value .+ for attribute .+ on element .+Typo for .+\?.*$': {
+          variable: true,
+          quality: 1,
+          what: 'Attribute value may be a typographical error'
+        }
+      }
+    }
+  },
   attributeMissing: {
     summary: 'attribute missing',
     why: 'Item behaves improperly',
@@ -6353,6 +6449,11 @@ exports.issues = {
           variable: false,
           quality: 1,
           what: 'Page contains more than 1 h1 element'
+        },
+        'Consider using the h1 element as a top-level heading only — or else use the headingoffset attribute (otherwise, all h1 elements are treated as top-level headings by many screen readers and other tools).': {
+          variable: false,
+          quality: 1,
+          what: 'Page contains more than 1 h1 element'
         }
       },
       nuVnu: {
@@ -6360,6 +6461,11 @@ exports.issues = {
           variable: false,
           quality: 1,
           what: 'Page contains more than 1 h1 element'
+        },
+        'Consider using the h1 element as a top-level heading only — or else use the headingoffset attribute (otherwise, all h1 elements are treated as top-level headings by many screen readers and other tools).': {
+          variable: false,
+          quality: 1,
+          what: 'Page contains more than 1 h1 element'
         }
       },
       wave: {

package/procs/score/tsp.js

@@ -51,7 +51,7 @@ const issueCountWeight = 10;
 */
 const maxWeight = 30;
 
-// 3. Tool
+// 2. Tool
 /*
   Severity: amount added to each raw tool score by each violation of a rule with ordinal severity 0
   through 3.
@@ -60,17 +60,17 @@ const severityWeights = [1, 2, 3, 4];
 // Final: multiplier of the raw tool score to obtain the final tool score.
 const toolWeight = 0.1;
 
-// 4. Element
+// 3. Element
 // Multiplier of the count of elements with at least 1 rule violation.
 const elementWeight = 2;
 
-// 5. Prevention
+// 4. Prevention
 // Each tool prevention by the page.
 const preventionWeight = 300;
 // Each prevention of a Testaro rule test by the page.
 const testaroRulePreventionWeight = 30;
 
-// 6. Log
+// 5. Log
 // Multipliers of log values to obtain the log score.
 const logWeights = {
   logCount: 0.1,
@@ -81,7 +81,7 @@ const logWeights = {
   visitRejectionCount: 2
 };
 
-// 7. Latency
+// 6. Latency
 // Normal latency (11 visits [1 per tool], with 2 seconds per visit).
 const normalLatency = 22;
 // Total latency exceeding normal, in seconds.
@@ -89,7 +89,7 @@ const latencyWeight = 2;
 
 // RULE CONSTANTS
 
-// Initialize a table of issue-classified tool rules.
+// Initialize a directory of issue-classified tool rules.
 const issueIndex = {};
 // Initialize an array of variably named tool rules.
 const issueMatcher = [];
@@ -100,7 +100,7 @@ Object.keys(issues).forEach(issueName => {
     // For each of those rules:
     Object.keys(issues[issueName].tools[toolName]).forEach(ruleID => {
       issueIndex[toolName] ??= {};
-      // Add it to the table of tool rules.
+      // Add it to the directory of tool rules.
       issueIndex[toolName][ruleID] = issueName;
       // If it is variably named:
       if (issues[issueName].tools[toolName][ruleID].variable) {
@@ -203,38 +203,35 @@ exports.scorer = report => {
                 return patternRE.test(ruleID);
               });
             }
-            // If the rule has an ID:
+            // If the instance rule has an ID:
             if (canonicalRuleID) {
               // Get the issue of the rule.
               const issueName = issueIndex[which][canonicalRuleID];
-              // If the rule ID belongs to a non-ignorable issue:
+              // If the issue is non-ignorable:
               if (issueName !== 'ignorable') {
-                // Add the instance to the issue details of the score data.
-                if (! details.issue[issueName]) {
-                  details.issue[issueName] = {
-                    summary: issues[issueName].summary,
-                    wcag: issues[issueName].wcag || '',
-                    score: 0,
-                    maxCount: 0,
-                    weight: issues[issueName].weight,
-                    countLimit: issues[issueName].max,
-                    instanceCounts: {},
-                    tools: {}
-                  };
-                  if (! details.issue[issueName].countLimit) {
-                    delete details.issue[issueName].countLimit;
-                  }
-                }
-                if (! details.issue[issueName].tools[which]) {
-                  details.issue[issueName].tools[which] = {};
-                }
-                if (! details.issue[issueName].instanceCounts[which]) {
-                  details.issue[issueName].instanceCounts[which] = 0;
+                // Initialize the issue details if necessary.
+                details.issue[issueName] ??= {
+                  summary: issues[issueName].summary,
+                  wcag: issues[issueName].wcag || '',
+                  score: 0,
+                  maxCount: 0,
+                  weight: issues[issueName].weight,
+                  countLimit: issues[issueName].max,
+                  instanceCounts: {},
+                  elementXPaths: [],
+                  tools: {}
+                };
+                const issueDetails = details.issue[issueName];
+                if (! issueDetails.countLimit) {
+                  delete issueDetails.countLimit;
                 }
-                details.issue[issueName].instanceCounts[which] += count;
-                if (! details.issue[issueName].tools[which][canonicalRuleID]) {
+                issueDetails.tools[which] ??= {};
+                issueDetails.instanceCounts[which] ??= 0;
+                // Add data from the instance to the issue details.
+                issueDetails.instanceCounts[which] += count;
+                if (! issueDetails.tools[which][canonicalRuleID]) {
                   const ruleData = issues[issueName].tools[which][canonicalRuleID];
-                  details.issue[issueName].tools[which][canonicalRuleID] = {
+                  issueDetails.tools[which][canonicalRuleID] = {
                     quality: ruleData.quality,
                     what: ruleData.what,
                     complaints: {
@@ -266,7 +263,7 @@ exports.scorer = report => {
                 issuePaths[issueName] ??= new Set();
                 // If the element has a path ID:
                 if (pathID) {
-                  // Ensure that it is in the issue-specific set of paths.
+                  // Ensure that it is in the issue-specific set of XPaths.
                   issuePaths[issueName].add(pathID);
                 }
               }