Show data usage on Site Details (MDSettings)

chrome/test/data/webui/settings/site_details_tests.js

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+function MockUsageHandler(element) {
+  this.testElement = element;
+}
+
+MockUsageHandler.prototype.fetchUsageTotal = function(origin) {
+  this.testElement.$.storedData_ = '1 KB';
+};

[Finnur, 2016/01/22 15:52:15]

This is purely here to facilitate discussion. I ran out of time before
completing the test, but would appreciate a comment on whether this is in line
with what you would expect, or if I need to take a different approach.
Currently, I see that the storedData_ is set on an HTMLElement, but the Polymer
object that I'm testing does not appear to show the UI through
hidden$="[[!storedData_]]". It therefore asserts on line 158.

[Finnur, 2016/01/25 11:19:16]

Ignore the comment above, this code is obsolete.

Instead, look at the test below and let me know if this approach is valid.

+
 /** @fileoverview Suite of tests for site-details. */
 cr.define('site_details', function() {
   function registerTests() {
     suite('SiteDetails', function() {
       /**
        * A site list element created before each test.
        * @type {SiteDetails}
        */
       var testElement;
 
@@ skipping 87 matching lines @@
       setup(function() {
         PolymerTest.clearBody();
         testElement = document.createElement('site-details');
         document.body.appendChild(testElement);
       });
 
       test('empty state', function() {
         testElement.prefs = prefsEmpty;
         testElement.origin = "http://www.google.com";
 
-        // Once actual storage numbers are shown (instead of hard-coded), these
-        // two will become hidden by default.
-        assertFalse(testElement.$.usage.hidden);
-        assertFalse(testElement.$.storage.hidden);
+        assertTrue(testElement.$.usage.hidden);
+        assertTrue(testElement.$.storage.hidden);
 
         // TODO(finnur): Check for the Permission heading hiding when no
         // permissions are showing.
 
         var msg = 'No category should be showing, height';
         assertEquals(0, testElement.$.camera.offsetHeight, msg);
         assertEquals(0, testElement.$.cookies.offsetHeight, msg);
         assertEquals(0, testElement.$.fullscreen.offsetHeight, msg);
         assertEquals(0, testElement.$.geolocation.offsetHeight, msg);
         assertEquals(0, testElement.$.javascript.offsetHeight, msg);
         assertEquals(0, testElement.$.mic.offsetHeight, msg);
         assertEquals(0, testElement.$.notification.offsetHeight, msg);
         assertEquals(0, testElement.$.popups.offsetHeight, msg);
       });
 
       test('all categories visible', function() {
         testElement.prefs = prefs;
         testElement.origin = "https://foo-allow.com:443";
 
         var msg = 'All categories should be showing';
         assertNotEquals(0, testElement.$.camera.offsetHeight, msg);
         assertNotEquals(0, testElement.$.cookies.offsetHeight, msg);
         assertNotEquals(0, testElement.$.fullscreen.offsetHeight, msg);
         assertNotEquals(0, testElement.$.geolocation.offsetHeight, msg);
         assertNotEquals(0, testElement.$.javascript.offsetHeight, msg);
         assertNotEquals(0, testElement.$.mic.offsetHeight, msg);
         assertNotEquals(0, testElement.$.notification.offsetHeight, msg);
         assertNotEquals(0, testElement.$.popups.offsetHeight, msg);
       });
+
+      test('usage heading shows on storage available', function() {
+        testElement.$.usageApi = new MockUsageHandler(testElement);
+        testElement.prefs = prefs;
+        testElement.origin = "https://foo-allow.com:443";
+        Polymer.dom.flush();
+
+        assertFalse(testElement.$.usage.hidden);
+        assertFalse(testElement.$.storage.hidden);
+      });
     });
   }
   return {
     registerTests: registerTests,
   };
 });