add test for nonscored items on CLI printer

GoogleChrome · Oct 11, 2016 · 1082ed3 · 1082ed3
1 parent 1c081c9
commit 1082ed3
Show file tree

Hide file tree

Showing 4 changed files with 41 additions and 16 deletions.
diff --git a/lighthouse-cli/test/cli/printer-test.js b/lighthouse-cli/test/cli/printer-test.js
@@ -57,6 +57,11 @@ describe('Printer', () => {
     // Just check there's no HTML / JSON there.
     assert.throws(_ => JSON.parse(prettyOutput));
     assert.equal(/<!doctype/gim.test(prettyOutput), false);
+
+    const hasScoreOnNonScoredItem = /Using modern offline features.*?(0%|NaN)/.test(prettyOutput);
+    const hasAggregationPresent = prettyOutput.includes('Using modern offline features');
+    assert.equal(hasScoreOnNonScoredItem, false, 'non-scored item was scored');
+    assert.equal(hasAggregationPresent, true, 'non-scored aggregation item is not there');
   });
 
   it('creates HTML for results', () => {

diff --git a/lighthouse-cli/test/fixtures/sample.json b/lighthouse-cli/test/fixtures/sample.json
@@ -60,6 +60,24 @@
       "category": "JavaScript",
       "description": "Page contains some content when its scripts are not available"
     },
+    "appcache-manifest": {
+      "score": true,
+      "displayValue": "",
+      "rawValue": true,
+      "name": "appcache-manifest",
+      "category": "Offline",
+      "description": "Site is not using Application Cache",
+      "helpText": "Application Cache has been <a href=\"https://html.spec.whatwg.org/multipage/browsers.html#offline\" target=\"_blank\">deprecated</a> by <a href=\"https://developer.mozilla.org/en-US/docs/Web/API/Service_Worker_API/Using_Service_Workers\" target=\"_blank\">Service Workers</a>. Consider implementing an offline solution using the <a href=\"https://developer.mozilla.org/en-US/docs/Web/API/Cache\" target=\"_blank\">Cache Storage API</a>."
+    },
+    "no-websql": {
+      "score": true,
+      "displayValue": "",
+      "rawValue": true,
+      "name": "no-websql",
+      "category": "Offline",
+      "description": "Site is not using WebSQL DB.",
+      "helpText": "Web SQL Database is <a href=\"https://dev.w3.org/html5/webdatabase/\" target=\"_blank\">deprecated</a>. Consider implementing an offline solution using <a href=\"https://developer.mozilla.org/en-US/docs/Web/API/IndexedDB_API/Using_IndexedDB\" target=\"_blank\">IndexedDB</a>."
+    },
     "first-meaningful-paint": {
       "score": 100,
       "displayValue": "615.8ms",
@@ -610,6 +628,22 @@
           ]
         }
       ]
+    },
+     {
+      "name": "Do Better Web",
+      "description": "We've compiled some recommendations for modernizing your web app and avoiding performance pitfalls.",
+      "scored": false,
+      "categorizable": true,
+      "score": [
+        {
+          "overall": null,
+          "name": "Using modern offline features",
+          "subItems": [
+            "appcache-manifest",
+            "no-websql"
+          ]
+        }
+      ]
     }
   ]
 }
diff --git a/lighthouse-core/aggregator/aggregate.js b/lighthouse-core/aggregator/aggregate.js
@@ -38,9 +38,6 @@ class Aggregate {
   static _getTotalWeight(expected) {
     const expectedNames = Object.keys(expected);
     let totalWeight = expectedNames.reduce((last, e) => last + (expected[e].weight || 0), 0);
-    if (totalWeight === 0) {
-      totalWeight = 1;
-    }
     return totalWeight;
   }
 
@@ -160,7 +157,7 @@ class Aggregate {
 
       const subItems = [];
       let overallScore = 0;
-      let maxScore = 0;
+      let maxScore = 1;
 
       // Step through each item in the expected results, and add them
       // to the overall score and add each to the subItems list.

diff --git a/lighthouse-core/test/aggregator/aggregate-test.js b/lighthouse-core/test/aggregator/aggregate-test.js
@@ -54,18 +54,7 @@ describe('Aggregate', () => {
     const a = {};
 
     const weight = Aggregate._getTotalWeight(a);
-    return assert.equal(weight, 1);
-  });
-
-  it('returns a weight of at least 1', () => {
-    const a = {
-      x: {
-        weight: 0
-      }
-    };
-
-    const weight = Aggregate._getTotalWeight(a);
-    return assert.equal(weight, 1);
+    return assert.equal(weight, 0);
   });
 
   it('generates the correct total weight', () => {