testilo 21.2.4 → 21.4.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "testilo",
-  "version": "21.2.4",
+  "version": "21.4.0",
   "description": "Prepares and processes Testaro reports",
   "main": "aim.js",
   "scripts": {

package/procs/score/tic38.js

@@ -2736,6 +2736,21 @@ exports.issues = {
       }
     }
   },
+  itemIDBad: {
+    summary: 'itemid invalid',
+    why: 'User cannot get help to identify a referent',
+    wcag: '1.3.1',
+    weight: 4,
+    tools: {
+      nuVal: {
+        'The itemid attribute must not be specified on elements that do not have both an itemscope attribute and an itemtype attribute specified.': {
+          variable: false,
+          quality: 1,
+          what: 'Element has an itemid attribute without both an itemscope and an itemtype attribute'
+        }
+      }
+    }
+  },
   itemTypeBad: {
     summary: 'itemtype invalid',
     why: 'User cannot get help on the definition of a term',
@@ -2898,6 +2913,11 @@ exports.issues = {
           variable: false,
           quality: 1,
           what: 'Table structure element specifies an explicit role within the table container'
+        },
+        aria_child_valid: {
+          variable: false,
+          quality: 1,
+          what: 'Child element has a role not allowed for the role of the parent'
         }
       },
       nuVal: {
@@ -3020,11 +3040,6 @@ exports.issues = {
           variable: false,
           quality: 1,
           what: 'ARIA attribute is invalid for the role of its element'
-        },
-        aria_attribute_value_valid: {
-          variable: false,
-          quality: 1,
-          what: 'Value of an attribute on the element is not valid'
         }
       },
       nuVal: {
@@ -3043,16 +3058,6 @@ exports.issues = {
           quality: 1,
           what: 'Attribute not allowed on this element'
         },
-        '^Bad value .* for attribute .+ on element .+$': {
-          variable: true,
-          quality: 1,
-          what: 'Attribute on this element has an invalid value'
-        },
-        '^Bad value .+ for the attribute .+$': {
-          variable: true,
-          quality: 1,
-          what: 'Attribute has an invalid value'
-        },
         '^Attribute .+ not allowed here.*$': {
           variable: true,
           quality: 1,
@@ -3063,11 +3068,6 @@ exports.issues = {
           quality: 1,
           what: 'Attribute is invalidly nonserializable'
         },
-        '^Bad value  for attribute .+ on element .+: Must not be empty.*$': {
-          variable: true,
-          quality: 1,
-          what: 'Attribute has an invalidly empty value'
-        },
         '^Attribute .+ is only allowed when .+$': {
           variable: true,
           quality: 1,
@@ -3098,15 +3098,57 @@ exports.issues = {
           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'
+        }
+      }
+    }
+  },
+  attributeValueBad: {
+    summary: 'attribute value invalid',
+    why: 'Item behaves improperly',
+    wcag: '1.3.1',
+    weight: 4,
+    tools: {
+      ibm: {
+        aria_attribute_value_valid: {
+          variable: false,
+          quality: 1,
+          what: 'Value of an attribute on the element is not valid'
+        }
+      },
+      nuVal: {
+        '^Bad value .* for attribute .+ on element .+$': {
+          variable: true,
+          quality: 1,
+          what: 'Attribute on this element has an invalid value'
+        },
+        '^Bad value .+ for the attribute .+$': {
+          variable: true,
+          quality: 1,
+          what: 'Attribute has an invalid value'
+        },
+        '^Bad value  for attribute .+ on element .+: Must not be empty.*$': {
+          variable: true,
+          quality: 1,
+          what: 'Attribute has an invalidly empty value'
+        },
         '^Bad value  for attribute (?:width|height) on element img: The empty string is not a valid non-negative integer.*$': {
           variable: true,
           quality: 1,
           what: 'Attribute has an empty value'
         },
-        '^.+ in an unquoted attribute value. Probable causes: Attributes running together or a URL query string in an unquoted attribute value.*$': {
-          variable: true,
+        'A script element with a defer attribute must not have a type attribute with the value module.': {
+          variable: false,
           quality: 1,
-          what: 'Attribute has a value containing invalid punctuation'
+          what: 'script element with a defer attribute has type="module"'
         }
       }
     }
@@ -3132,11 +3174,6 @@ exports.issues = {
         }
       },
       nuVal: {
-        '^Element .+ is missing required attribute .+$': {
-          variable: true,
-          quality: 1,
-          what: 'Element is missing a required attribute'
-        },
         '^Element image is missing required attribute (?:height|width).*$': {
           variable: true,
           quality: 1,
@@ -3156,6 +3193,11 @@ exports.issues = {
           variable: false,
           quality: 1,
           what: 'source or img element is missing a media or type attribute'
+        },
+        '^Element .+ is missing required attribute .+$': {
+          variable: true,
+          quality: 1,
+          what: 'Element is missing a required attribute'
         }
       }
     }
@@ -4064,7 +4106,7 @@ exports.issues = {
       }
     }
   },
-  docType: {
+  docTypeMissing: {
     summary: 'DOCTYPE missing',
     why: 'Browser processes the document improperly',
     wcag: '1.3.1',
@@ -4087,6 +4129,22 @@ exports.issues = {
       }
     }
   },
+  docTypeBad: {
+    summary: 'DOCTYPE invalid',
+    why: 'Browser processes the document improperly',
+    wcag: '1.3.1',
+    weight: 3,
+    max: 1,
+    tools: {
+      nuVal: {
+        'Almost standards mode doctype. Expected <!DOCTYPE html>.': {
+          variable: false,
+          quality: 1,
+          what: 'document type declaration differs from <!DOCTYPE html>'
+        }
+      }
+    }
+  },
   pageTitleBad: {
     summary: 'page title invalid',
     why: 'Browser processes the document improperly',
@@ -5256,6 +5314,13 @@ exports.issues = {
           quality: 1,
           what: 'None of the cells in the table is a header'
         }
+      },
+      ibm: {
+        table_headers_exists: {
+          variable: false,
+          quality: 1,
+          what: 'No cell in the table is a th element or has a scope or headers attribute'
+        }
       }
     }
   },
@@ -5610,6 +5675,11 @@ exports.issues = {
           quality: 1,
           what: 'descendant of an element with a link role has a tabindex attribute'
         },
+        'An element with the attribute tabindex must not appear as a descendant of the button element.': {
+          variable: false,
+          quality: 1,
+          what: 'descendant of a button element has a tabindex attribute'
+        },
         'An element with the attribute tabindex must not appear as a descendant of an element with the attribute role=button.': {
           variable: false,
           quality: 1,
@@ -7965,24 +8035,102 @@ exports.issues = {
       }
     }
   },
-  parseError: {
-    summary: 'code invalid',
-    why: 'Document contains invalid code',
+  characterBad: {
+    summary: 'invalid character',
+    why: 'Invalid character makes the document behave correctly',
     wcag: '4.1',
     weight: 3,
     tools: {
-      ibm: {
-        aria_child_valid: {
+      nuVal: {
+        '^Bad value [^\ufffd]+ Tab, new line or carriage return found.*$': {
+          variable: true,
+          quality: 1,
+          what: 'Attribute value contains an illegal spacing character'
+        },
+        '^Bad character . after <. Probable cause: Unescaped <. Try escaping it as &lt;.*$': {
+          variable: true,
+          quality: 1,
+          what: 'Left angle bracket is followed by an invalid character'
+        },
+        '^Saw .+ when expecting an attribute name. Probable cause: (?:.+ missing|Missing .+) immediately before.*$': {
+          variable: true,
+          quality: 1,
+          what: 'Invalid character appears where an attribute name must appear'
+        },
+        '^Bad element name .*: Code point .* is not allowed*$': {
+          variable: true,
+          quality: 1,
+          what: 'Element name contains an invalid character'
+        },
+        '^Bad value .* for attribute href on element .+: Illegal character in path segment: .+ is not allowed.*$': {
+          variable: true,
+          quality: 1,
+          what: 'href attribute path value contains an invalid character in a segment'
+        },
+        '^Bad value .* for attribute src on element .+: Illegal character in path segment: .+ is not allowed.*$': {
+          variable: true,
+          quality: 1,
+          what: 'src attribute path value contains an invalid character in a segment'
+        },
+        '^Bad value .* for attribute href on element .+: Illegal character in query: .+ is not allowed.*$': {
+          variable: true,
+          quality: 1,
+          what: 'href attribute query value contains an invalid character'
+        },
+        '^Bad value .* for attribute src on element .+: Illegal character in query: .+ is not allowed.*$': {
+          variable: true,
+          quality: 1,
+          what: 'src attribute query value contains an invalid character'
+        },
+        '^Bad value .+ for attribute src on element .+: Tab, new line or carriage return found.*$': {
+          variable: true,
+          quality: 1,
+          what: 'src attribute value contains a tab, newline, or return character'
+        },
+        'Non-space character inside noscript inside head.': {
           variable: false,
           quality: 1,
-          what: 'Child element has a role not allowed for the role of the parent'
+          what: 'noscript element inside the head element has a nonspace text-node child'
+        },
+        '^.+ in an unquoted attribute value. Probable causes: Attributes running together or a URL query string in an unquoted attribute value.*$': {
+          variable: true,
+          quality: 1,
+          what: 'Attribute has a value containing invalid punctuation'
+        }
+      }
+    }
+  },
+  textContentBad: {
+    summary: 'element text content invalid',
+    why: 'User may be unable to read all the document text',
+    wcag: '4.1',
+    weight: 3,
+    tools: {
+      nuVal: {
+        'Text run is not in Unicode Normalization Form C.': {
+          variable: false,
+          quality: 1,
+          what: 'Text run is not in Unicode Normalization Form C.'
         },
-        Rpt_Aria_InvalidTabindexForActivedescendant: {
+        '^The text content of element .+ was not in the required format: Expected .+ but found .+ instead.*$': {
+          variable: true,
+          quality: 1,
+          what: 'Element has text content with invalid format'
+        },
+        'The text content of element time was not in the required format: The literal did not satisfy the time-datetime format.': {
           variable: false,
           quality: 1,
-          what: 'Element with an aria-activedescendant attribute has no nonpositive tabindex attribute'
+          what: 'time element has text content that is not in the time-datetime format'
         }
-      },
+      }
+    }
+  },
+  parseError: {
+    summary: 'code invalid',
+    why: 'Invalid code in the document may prevent a helper from working',
+    wcag: '4.1',
+    weight: 3,
+    tools: {
       nuVal: {
         '^End tag .+ did not match the name of the current open element (.+).*$': {
           variable: true,
@@ -7999,26 +8147,11 @@ exports.issues = {
           quality: 1,
           what: 'No space between attributes'
         },
-        '^Bad value [^\ufffd]+ Tab, new line or carriage return found.*$': {
-          variable: true,
-          quality: 1,
-          what: 'Attribute value contains an illegal spacing character'
-        },
         'Saw <?. Probable cause: Attempt to use an XML processing instruction in HTML. (XML processing instructions are not supported in HTML.)': {
           variable: false,
           quality: 1,
           what: 'Left angle bracket is followed by a question mark'
         },
-        '^Bad character . after <. Probable cause: Unescaped <. Try escaping it as &lt;.*$': {
-          variable: true,
-          quality: 1,
-          what: 'Left angle bracket is followed by an invalid character'
-        },
-        'Almost standards mode doctype. Expected <!DOCTYPE html>.': {
-          variable: false,
-          quality: 1,
-          what: 'document type declaration differs from <!DOCTYPE html>'
-        },
         '^The aria-hidden attribute must not be specified on the .+ element.*$': {
           variable: true,
           quality: 1,
@@ -8044,11 +8177,6 @@ exports.issues = {
           quality: 1,
           what: 'Comment is nested within a comment'
         },
-        '^Saw .+ when expecting an attribute name. Probable cause: (?:.+ missing|Missing .+) immediately before.*$': {
-          variable: true,
-          quality: 1,
-          what: 'Invalid character appears where an attribute name must appear'
-        },
         'The document is not mappable to XML 1.0 due to two consecutive hyphens in a comment.': {
           variable: false,
           quality: 1,
@@ -8069,11 +8197,6 @@ exports.issues = {
           quality: 1,
           what: 'Invalid element name'
         },
-        '^Bad element name .*: Code point .* is not allowed*$': {
-          variable: true,
-          quality: 1,
-          what: 'Element name contains an invalid character'
-        },
         '^Forbidden code point U+.+$': {
           variable: true,
           quality: 1,
@@ -8084,11 +8207,6 @@ exports.issues = {
           quality: 1,
           what: 'Encoding declaration disagrees with the actual encoding of the page'
         },
-        'Text run is not in Unicode Normalization Form C.': {
-          variable: false,
-          quality: 1,
-          what: 'Text run is not in Unicode Normalization Form C.'
-        },
         'Quote \" in attribute name. Probable cause: Matching quote missing somewhere earlier.': {
           variable: false,
           quality: 1,
@@ -8099,41 +8217,11 @@ exports.issues = {
           quality: 1,
           what: 'script element has an async attribute but has no src or value=module attribute'
         },
-        '^Bad value .* for attribute href on element .+: Illegal character in path segment: .+ is not allowed.*$': {
-          variable: true,
-          quality: 1,
-          what: 'href attribute path value contains an invalid character in a segment'
-        },
-        '^Bad value .* for attribute src on element .+: Illegal character in path segment: .+ is not allowed.*$': {
-          variable: true,
-          quality: 1,
-          what: 'src attribute path value contains an invalid character in a segment'
-        },
-        '^Bad value .* for attribute href on element .+: Illegal character in query: .+ is not allowed.*$': {
-          variable: true,
-          quality: 1,
-          what: 'href attribute query value contains an invalid character'
-        },
-        '^Bad value .* for attribute src on element .+: Illegal character in query: .+ is not allowed.*$': {
-          variable: true,
-          quality: 1,
-          what: 'src attribute query value contains an invalid character'
-        },
-        '^Bad value .+ for attribute src on element .+: Tab, new line or carriage return found.*$': {
-          variable: true,
-          quality: 1,
-          what: 'src attribute value contains a tab, newline, or return character'
-        },
         '^Text not allowed in element .+ in this context.*$': {
           variable: true,
           quality: 1,
           what: 'Element contains text, which is not allowed here'
         },
-        'Element source is missing required attribute srcset.': {
-          variable: false,
-          quality: 1,
-          what: 'source element has no srcset attribute'
-        },
         '^The .+ element must not appear as a descendant of the .+ element.*$': {
           variable: true,
           quality: 1,
@@ -8149,41 +8237,11 @@ exports.issues = {
           quality: 1,
           what: 'option element has a nonempty value'
         },
-        '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 text content of element .+ was not in the required format: Expected .+ but found .+ instead.*$': {
-          variable: true,
-          quality: 1,
-          what: 'Element has text content with invalid format'
-        },
         'Element script must not have attribute charset unless attribute src is also specified.': {
           variable: false,
           quality: 1,
           what: 'script element has a charset attribute but no src attribute'
         },
-        'The text content of element time was not in the required format: The literal did not satisfy the time-datetime format.': {
-          variable: false,
-          quality: 1,
-          what: 'time element has text content that is not in the time-datetime format'
-        },
-        'A script element with a defer attribute must not have a type attribute with the value module.': {
-          variable: false,
-          quality: 1,
-          what: 'script element with a defer attribute has type="module"'
-        },
-        'Non-space character inside noscript inside head.': {
-          variable: false,
-          quality: 1,
-          what: 'noscript element inside the head element has a nonspace text-node child'
-        },
         '^java.util.concurrent.TimeoutException: Idle timeout expired: .+ ms.*$': {
           variable: true,
           quality: 1,
@@ -8221,9 +8279,9 @@ exports.issues = {
       }
     }
   },
-  encodingBad: {
+  encodingPrivate: {
     summary: 'text in Private Use Area',
-    why: 'Document contains invalid code',
+    why: 'User cannot read all of the text',
     wcag: '3.1.3',
     weight: 4,
     max: 1,

package/procs/score/tsp38-old.js

@@ -0,0 +1,300 @@
+/*
+  tsp37
+  Testilo score proc 37
+
+  Computes target score data and adds them to a ts37 report.
+*/
+
+// IMPORTS
+
+const {issues} = require('./tic38');
+
+// CONSTANTS
+
+// ID of this proc.
+const scoreProcID = 'tsp38';
+// Latency weight (how much each second of excess latency adds to the score).
+const latencyWeight = 1;
+// Normal latency (6 visits, with 1.5 second per visit).
+const normalLatency = 9;
+// Prevention weight (how much each prevention adds to the score).
+const preventionWeight = 300;
+// Maximum instance count addition weight (divisor of max).
+const maxWeight = 30;
+// Other weights.
+const severityWeights = [1, 2, 3, 4];
+const toolWeight = 0.1;
+const logWeights = {
+  logCount: 0.1,
+  logSize: 0.002,
+  errorLogCount: 0.2,
+  errorLogSize: 0.004,
+  prohibitedCount: 3,
+  visitRejectionCount: 2
+};
+// Initialize a table of tool rules.
+const issueIndex = {};
+// Initialize an array of variably named tool rules.
+const issueMatcher = [];
+// For each issue:
+Object.keys(issues).forEach(issueName => {
+  // For each tool with rules belonging to that issue:
+  Object.keys(issues[issueName].tools).forEach(toolName => {
+    // For each of those rules:
+    Object.keys(issues[issueName].tools[toolName]).forEach(ruleID => {
+      // Add it to the table of tool rules.
+      if (! issueIndex[toolName]) {
+        issueIndex[toolName] = {};
+      }
+      issueIndex[toolName][ruleID] = issueName;
+      // If it is variably named:
+      if (issues[issueName].tools[toolName][ruleID].variable) {
+        // Add it to the array of variably named tool rules.
+        issueMatcher.push(ruleID);
+      }
+    })
+  });
+});
+
+// FUNCTIONS
+
+// Scores a report.
+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');
+    if (testActs.length) {
+      // Initialize the score data.
+      const score = {
+        scoreProcID,
+        weights: {
+          severities: severityWeights,
+          tool: toolWeight,
+          log: logWeights,
+          latency: latencyWeight,
+          prevention: preventionWeight,
+          maxInstanceCount: maxWeight
+        },
+        normalLatency,
+        summary: {
+          total: 0,
+          issue: 0,
+          solo: 0,
+          tool: 0,
+          prevention: 0,
+          log: 0,
+          latency: 0
+        },
+        details: {
+          severity: {
+            total: [0, 0, 0, 0],
+            byTool: {}
+          },
+          prevention: {},
+          issue: {},
+          solo: {},
+          tool: {}
+        }
+      };
+      const {summary, details} = score;
+      // For each test act:
+      testActs.forEach(act => {
+        // If the page prevented the tool from operating:
+        const {which, standardResult} = act;
+        if (! standardResult || standardResult.prevented) {
+          // Add this to the score.
+          details.prevention[which] = preventionWeight;
+        }
+        // Otherwise, if a valid standard result exists:
+        else if (
+          standardResult
+          && standardResult.totals
+          && standardResult.totals.length === 4
+          && standardResult.instances
+        ) {
+          // Add the severity totals of the tool to the score.
+          const {totals} = standardResult;
+          details.severity.byTool[which] = totals;
+          // Add the severity-weighted tool totals to the score.
+          details.tool[which] = totals.reduce(
+            (sum, current, index) => sum + severityWeights[index] * current, 0
+          );
+          // For each instance of the tool:
+          standardResult.instances.forEach(instance => {
+            // Get the rule ID.
+            const {ruleID, ordinalSeverity, count} = instance;
+            // If it is not in the table of tool rules:
+            let canonicalRuleID = ruleID;
+            if (! issueIndex[which][ruleID]) {
+              // Convert it to the variably named tool rule that it matches, if any.
+              canonicalRuleID = issueMatcher.find(pattern => {
+                const patternRE = new RegExp(pattern);
+                return patternRE.test(instance.ruleID);
+              });
+            }
+            // If the rule ID belongs to an issue:
+            if (canonicalRuleID) {
+              // Get the issue.
+              const issueName = issueIndex[which][canonicalRuleID];
+              // Add the instance to the issue details of the score data.
+              if (! details.issue[issueName]) {
+                details.issue[issueName] = {
+                  summary: issues[issueName].summary,
+                  score: 0,
+                  maxCount: 0,
+                  weight: issues[issueName].weight,
+                  countLimit: issues[issueName].max,
+                  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].tools[which][canonicalRuleID]) {
+                const ruleData = issues[issueName].tools[which][canonicalRuleID];
+                details.issue[issueName].tools[which][canonicalRuleID] = {
+                  quality: ruleData.quality,
+                  what: ruleData.what,
+                  complaints: {
+                    countTotal: 0,
+                    texts: []
+                  }
+                };
+              }
+              details
+              .issue[issueName]
+              .tools[which][canonicalRuleID]
+              .complaints
+              .countTotal += instance.count || 1;
+              if (
+                ! details
+                .issue[issueName]
+                .tools[which][canonicalRuleID]
+                .complaints
+                .texts
+                .includes(instance.what)
+              ) {
+                details
+                .issue[issueName]
+                .tools[which][canonicalRuleID]
+                .complaints
+                .texts
+                .push(instance.what);
+              }
+            }
+            // Otherwise, i.e. if the rule ID belongs to no issue:
+            else {
+              // Add the instance to the solo details of the score data.
+              if (! details.solo[which]) {
+                details.solo[which] = {};
+              }
+              if (! details.solo[which][ruleID]) {
+                details.solo[which][ruleID] = 0;
+              }
+              details.solo[which][ruleID] += (count || 1) * (ordinalSeverity + 1);
+              // Report this.
+              console.log(`ERROR: ${instance.ruleID} of ${which} not found in issues`);
+            }
+          });
+        }
+        // Otherwise, i.e. if a failed standard result exists:
+        else {
+          // Add an inferred prevention to the score.
+          details.prevention[which] = preventionWeight;
+        }
+      });
+      // For each issue with any complaints:
+      Object.keys(details.issue).forEach(issueName => {
+        const issueData = details.issue[issueName];
+        // For each tool with any complaints for the issue:
+        Object.keys(issueData.tools).forEach(toolID => {
+          // Get the sum of the quality-weighted counts of its issue rules.
+          let weightedCount = 0;
+          Object.values(issueData.tools[toolID]).forEach(ruleData => {
+            weightedCount += ruleData.quality * ruleData.complaints.countTotal;
+          });
+          // If the sum exceeds the existing maximum weighted count for the issue:
+          if (weightedCount > issueData.maxCount) {
+            // Change the maximum count for the issue to the sum.
+            issueData.maxCount = weightedCount;
+          }
+        });
+        // Get the score for the issue, including any addition for the instance count limit.
+        const maxAddition = issueData.countLimit ? maxWeight / issueData.countLimit : 0;
+        issueData.score = Math.round(issueData.weight * issueData.maxCount * (1 + maxAddition));
+      });
+      // Add the severity detail totals to the score.
+      details.severity.total = Object.keys(details.severity.byTool).reduce((severityTotals, toolID) => {
+        details.severity.byTool[toolID].forEach((severityScore, index) => {
+          severityTotals[index] += severityScore;
+        });
+        return severityTotals;
+      }, details.severity.total);
+      // Add the summary issue total to the score.
+      summary.issue = Object
+      .values(details.issue)
+      .reduce((total, current) => total + current.score, 0);
+      // Add the summary solo total to the score.
+      Object.keys(details.solo).forEach(tool => {
+        summary.solo += Object
+        .values(details.solo[tool])
+        .reduce((total, current) => total + current);
+      });
+      // Add the summary tool total to the score.
+      summary.tool = toolWeight * details.severity.total.reduce(
+        (total, current, index) => total + severityWeights[index] * current, 0
+      );
+      // Add the summary prevention total to the score.
+      summary.prevention = Object.values(details.prevention).reduce(
+        (total, current) => total + current, 0
+      );
+      // Add the summary log score to the score.
+      const {jobData} = report;
+      if (jobData) {
+        summary.log = Math.max(0, Math.round(
+          logWeights.logCount * jobData.logCount
+          + logWeights.logSize * jobData.logSize +
+          + logWeights.errorLogCount * jobData.errorLogCount
+          + logWeights.errorLogSize * jobData.errorLogSize
+          + logWeights.prohibitedCount * jobData.prohibitedCount +
+          + logWeights.visitRejectionCount * jobData.visitRejectionCount
+        ));
+        // Add the summary latency score to the score.
+        summary.latency = Math.round(
+          latencyWeight * (Math.max(0, jobData.visitLatency - normalLatency))
+        );
+      }
+      // Round the unrounded scores.
+      Object.keys(summary).forEach(summaryTypeName => {
+        summary[summaryTypeName] = Math.round(summary[summaryTypeName]);
+      });
+      details.severity.total.forEach((severityTotal, index) => {
+        details.severity.total[index] = Math.round(severityTotal);
+      });
+      // Add the summary total score to the score.
+      summary.total = summary.issue
+      + summary.solo
+      + summary.tool
+      + summary.prevention
+      + summary.log
+      + summary.latency;
+      // Add the score to the report or replace the existing score of the report.
+      report.score = score;
+    }
+    // Otherwise, i.e. if none of them is a test act:
+    else {
+      // Report this.
+      console.log('ERROR: No test acts');
+    }
+  }
+  // Otherwise, i.e. if there are no acts in the report:
+  else {
+    // Report this.
+    console.log('ERROR: No acts');
+  }
+};

package/procs/score/tsp38.js

@@ -21,6 +21,8 @@ const normalLatency = 9;
 const preventionWeight = 300;
 // Maximum instance count addition weight (divisor of max).
 const maxWeight = 30;
+// Issue count weight.
+const issueCountWeight = 15;
 // Other weights.
 const severityWeights = [1, 2, 3, 4];
 const toolWeight = 0.1;
@@ -80,6 +82,7 @@ exports.scorer = report => {
         normalLatency,
         summary: {
           total: 0,
+          issueCount: 0,
           issue: 0,
           solo: 0,
           tool: 0,
@@ -134,57 +137,59 @@ exports.scorer = report => {
                 return patternRE.test(instance.ruleID);
               });
             }
-            // If the rule ID belongs to an issue:
+            // If the rule ID belongs to a non-ignorable issue:
             if (canonicalRuleID) {
               // Get the issue.
               const issueName = issueIndex[which][canonicalRuleID];
               // Add the instance to the issue details of the score data.
-              if (! details.issue[issueName]) {
-                details.issue[issueName] = {
-                  summary: issues[issueName].summary,
-                  score: 0,
-                  maxCount: 0,
-                  weight: issues[issueName].weight,
-                  countLimit: issues[issueName].max,
-                  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].tools[which][canonicalRuleID]) {
-                const ruleData = issues[issueName].tools[which][canonicalRuleID];
-                details.issue[issueName].tools[which][canonicalRuleID] = {
-                  quality: ruleData.quality,
-                  what: ruleData.what,
-                  complaints: {
-                    countTotal: 0,
-                    texts: []
+              if (issueName !== 'ignorable') {
+                if (! details.issue[issueName]) {
+                  details.issue[issueName] = {
+                    summary: issues[issueName].summary,
+                    score: 0,
+                    maxCount: 0,
+                    weight: issues[issueName].weight,
+                    countLimit: issues[issueName].max,
+                    tools: {}
+                  };
+                  if (! details.issue[issueName].countLimit) {
+                    delete details.issue[issueName].countLimit;
                   }
-                };
-              }
-              details
-              .issue[issueName]
-              .tools[which][canonicalRuleID]
-              .complaints
-              .countTotal += instance.count || 1;
-              if (
-                ! details
-                .issue[issueName]
-                .tools[which][canonicalRuleID]
-                .complaints
-                .texts
-                .includes(instance.what)
-              ) {
+                }
+                if (! details.issue[issueName].tools[which]) {
+                  details.issue[issueName].tools[which] = {};
+                }
+                if (! details.issue[issueName].tools[which][canonicalRuleID]) {
+                  const ruleData = issues[issueName].tools[which][canonicalRuleID];
+                  details.issue[issueName].tools[which][canonicalRuleID] = {
+                    quality: ruleData.quality,
+                    what: ruleData.what,
+                    complaints: {
+                      countTotal: 0,
+                      texts: []
+                    }
+                  };
+                }
                 details
                 .issue[issueName]
                 .tools[which][canonicalRuleID]
                 .complaints
-                .texts
-                .push(instance.what);
+                .countTotal += instance.count || 1;
+                if (
+                  ! details
+                  .issue[issueName]
+                  .tools[which][canonicalRuleID]
+                  .complaints
+                  .texts
+                  .includes(instance.what)
+                ) {
+                  details
+                  .issue[issueName]
+                  .tools[which][canonicalRuleID]
+                  .complaints
+                  .texts
+                  .push(instance.what);
+                }
               }
             }
             // Otherwise, i.e. if the rule ID belongs to no issue:
@@ -208,7 +213,7 @@ exports.scorer = report => {
           details.prevention[which] = preventionWeight;
         }
       });
-      // For each issue with any complaints:
+      // For each non-ignorable issue with any complaints:
       Object.keys(details.issue).forEach(issueName => {
         const issueData = details.issue[issueName];
         // For each tool with any complaints for the issue:
@@ -235,6 +240,8 @@ exports.scorer = report => {
         });
         return severityTotals;
       }, details.severity.total);
+      // Add the summary issue-count total to the score.
+      summary.issueCount = Object.keys(details.issue).length * issueCountWeight;
       // Add the summary issue total to the score.
       summary.issue = Object
       .values(details.issue)
@@ -277,7 +284,8 @@ exports.scorer = report => {
         details.severity.total[index] = Math.round(severityTotal);
       });
       // Add the summary total score to the score.
-      summary.total = summary.issue
+      summary.total = summary.issueCount
+      + summary.issue
       + summary.solo
       + summary.tool
       + summary.prevention