testilo 41.5.6 → 41.5.8

package/package.json

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

package/procs/score/tic49.js

@@ -251,6 +251,11 @@ exports.issues = {
           quality: 0,
           what: 'h1-h6 may fail to be used to identify headings [speculative]'
         },
+        'QW-BP2': {
+          variable: false,
+          quality: 1,
+          what: 'Image text alternative is not concise'
+        },
         'QW-BP17': {
           variable: false,
           quality: 1,
@@ -320,6 +325,11 @@ exports.issues = {
           variable: false,
           quality: 1,
           what: 'ol, ul or dl may fail to be used for a list or group of links [speculative]'
+        },
+        'QW-WCAG-T35': {
+          variable: false,
+          quality: 1,
+          what: 'Several elements have this id attribute value [invalid]'
         }
       }
     }
@@ -409,11 +419,6 @@ exports.issues = {
           variable: false,
           quality: 1,
           what: 'id attribute value is not unique'
-        },
-        'QW-WCAG-T35': {
-          variable: false,
-          quality: 1,
-          what: 'Several elements have this id attribute value'
         }
       }
     }
@@ -821,14 +826,14 @@ exports.issues = {
         'QW-WCAG-T8': {
           variable: false,
           quality: 1,
-          what: 'Text alternative is not an alternative'
+          what: 'Text alternative is suspect'
         }
       },
       wave: {
         alt_suspicious: {
           variable: false,
           quality: 1,
-          what: 'Image text alternative is suspicious'
+          what: 'Image text alternative is suspect'
         }
       }
     }
@@ -900,7 +905,7 @@ exports.issues = {
     weight: 4,
     tools: {
       nuVal: {
-        '^CSS: padding[-a-z]*: .+ negative values are not allowed\.$': {
+        '^CSS: padding[-a-z]*: .+ negative values are not allowed.*$': {
           variable: true,
           quality: 1,
           what: 'One of the CSS padding values is negative'
@@ -915,7 +920,7 @@ exports.issues = {
     weight: 4,
     tools: {
       nuVal: {
-        '^CSS: gap: .+ negative values are not allowed\.$': {
+        '^CSS: gap: .+ negative values are not allowed.*$': {
           variable: true,
           quality: 1,
           what: 'CSS gap value is negative'
@@ -1001,13 +1006,6 @@ exports.issues = {
           what: 'Linked img alt value longer than 160 characters'
         }
       },
-      qualWeb: {
-        'QW-BP2': {
-          variable: false,
-          quality: 1,
-          what: 'Image text alternative is not concise'
-        }
-      },
       wave: {
         alt_long: {
           variable: false,
@@ -2284,11 +2282,6 @@ exports.issues = {
         }
       },
       qualWeb: {
-        'QW-WCAG-T10': {
-          variable: false,
-          quality: 1,
-          what: 'Adjacent image and text links for the same resource are not combined'
-        },
         'QW-BP13': {
           variable: false,
           quality: 1,
@@ -2319,6 +2312,21 @@ exports.issues = {
       }
     }
   },
+  linkNameRisk: {
+    summary: 'image link text suspect',
+    why: 'Keyboard-only user may misunderstand the effect of following a link',
+    wcag: '2.4.4',
+    weight: 1,
+    tools: {
+      qualWeb: {
+        'QW-WCAG-T10': {
+          variable: false,
+          quality: 0.8,
+          what: 'Link name repeats its image name and/or that name is suspect'
+        }
+      }
+    }
+  },
   pageNewWindow: {
     summary: 'page immediately opens window',
     why: 'User promised a document gets another document, too',
@@ -3071,7 +3079,7 @@ exports.issues = {
     weight: 4,
     tools: {
       nuVal: {
-        '^Saw a start tag [a-z]+\.$': {
+        '^Saw a start tag [a-z]+.*$': {
           variable: true,
           quality: 1,
           what: 'Element does not exist in HTML'
@@ -5723,7 +5731,7 @@ exports.issues = {
           quality: 1,
           what: 'Table row has a column count larger than that of the first row'
         },
-        '^Table column \d+ established by element td has no cells beginning in it\.$': {
+        '^Table column \d+ established by element td has no cells beginning in it.*$': {
           variable: true,
           quality: 1,
           what: 'Element is td but the prior cells in its table column do not exist'
@@ -8816,7 +8824,7 @@ exports.issues = {
           quality: 1,
           what: 'Space is misused as a CSS operator'
         },
-        '^CSS: .+: , is an incorrect operator\.$': {
+        '^CSS: .+: , is an incorrect operator.*$': {
           variable: true,
           quality: 1,
           what: 'Comma is misused as a CSS operator'
@@ -8886,7 +8894,7 @@ exports.issues = {
           quality: 1,
           what: 'Invalid closing tag'
         },
-        '^End tag [a-z]+\.$': {
+        '^End tag [a-z]+.*$': {
           variable: true,
           quality: 1,
           what: 'Closing tag of an ineligible element'
@@ -9121,7 +9129,7 @@ exports.issues = {
           quality: 1,
           what: 'Invalid element name'
         },
-        '^Quote . in attribute name\. Probable cause: Matching quote missing somewhere earlier\.$': {
+        '^Quote . in attribute name\\. Probable cause: Matching quote missing somewhere earlier.*$': {
           variable: true,
           quality: 1,
           what: 'Attribute name includes an apostrophe or double quotation mark'

package/procs/score/tsp49.js

@@ -37,6 +37,7 @@ const {issues} = require('./tic49');
 const scoreProcID = 'tsp49';
 
 // WEIGHT CONSTANTS
+
 // How much is added to the page score by each component.
 
 // 1. Issue
@@ -117,9 +118,9 @@ exports.scorer = report => {
   // If there are any acts in the report:
   const {acts} = report;
   if (Array.isArray(acts) && acts.length) {
-    // If any of them are test acts:
     const testActs = acts.filter(act => act.type === 'test');
     const testTools = new Set(testActs.map(act => act.which));
+    // If any of them are test acts:
     if (testActs.length) {
       // Initialize the score data.
       const score = {
@@ -402,7 +403,7 @@ exports.scorer = report => {
       + summary.prevention
       + summary.log
       + summary.latency;
-      // Add the score to the report or replace the existing score of the report.
+      // Add a, or replace the, score property of the report.
       report.score = score;
     }
     // Otherwise, i.e. if none of them is a test act: