fix(cli): Don't report scores on nonscored items

lighthouse-cli/printer.js

@@ -80,6 +80,7 @@ function formatScore(score, suffix) {
   if (typeof score !== 'number') {
     return `${purple}${score}${reset}`;
   }
+
   let colorChoice = red;
   if (score > 45) {
     colorChoice = yellow;
@@ -123,7 +124,7 @@ function createOutput(results, outputMode) {
       let score = (item.overall * 100).toFixed(0);
 
       if (item.name) {
-        output += `${bold}${item.name}${reset}: ${formatScore(Number(score), '%')}\n`;
+        output += `${bold}${item.name}${reset}: ${item.scored ? formatScore(score, '%') : ''}\n`;
       }
 
       item.subItems.forEach(subitem => {

lighthouse-core/aggregator/aggregate.js

@@ -37,12 +37,11 @@ class Aggregate {
    */
   static _getTotalWeight(expected) {
     const expectedNames = Object.keys(expected);
-    let weight = expectedNames.reduce((last, e) => last + expected[e].weight, 0);
-    if (weight === 0) {
-      weight = 1;
+    let totalWeight = expectedNames.reduce((last, e) => last + (expected[e].weight || 0), 0);
+    if (totalWeight === 0) {
+      totalWeight = 1;
     }
-
-    return weight;
+    return totalWeight;
   }
 
   /**
@@ -158,9 +157,10 @@ class Aggregate {
           Aggregate._remapResultsByName(
             Aggregate._filterResultsByAuditNames(results, item.audits)
           );
-      const maxScore = Aggregate._getTotalWeight(item.audits);
+
       const subItems = [];
       let overallScore = 0;
+      let maxScore = 0;
 
       // Step through each item in the expected results, and add them
       // to the overall score and add each to the subItems list.
@@ -197,6 +197,10 @@ class Aggregate {
             e);
       });
 
+      if (aggregationIsScored) {
+        maxScore = Aggregate._getTotalWeight(item.audits);
+      }
+
       return {
         overall: (overallScore / maxScore),
         name: item.name,