Support numbers in settings-checkbox

chrome/test/data/webui/settings/checkbox_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.
 
 /** @fileoverview Suite of tests for settings-checkbox. */
 cr.define('settings_checkbox', function() {
   function registerTests() {
     suite('SettingsCheckbox', function() {
       /**
        * Checkbox created before each test.
@@ skipping 23 matching lines @@
         testElement = document.createElement('settings-checkbox');
         testElement.set('pref', pref);
         document.body.appendChild(testElement);
       });
 
       test('responds to checked attribute', function() {
         assertTrue(testElement.checked);
 
         testElement.removeAttribute('checked');
         assertFalse(testElement.checked);
-        assertFalse(pref.value);
+        assertEquals(false, pref.value);
 
         testElement.setAttribute('checked', '');
         assertTrue(testElement.checked);
-        assertTrue(pref.value);
+        assertEquals(true, pref.value);

[Dan Beam, 2015/12/02 21:48:57]

why is this better?

[stevenjb, 2015/12/02 22:07:27]

It's not. I made the incorrect assumption that assertFalse(0) would pass.

       });
 
       test('fires a change event', function(done) {
         testElement.addEventListener('change', function() {
           assertFalse(testElement.checked);
           done();
         });
         MockInteractions.tap(testElement.$.checkbox);
       });
 
       test('does not change when disabled', function() {
         testElement.checked = false;
         testElement.setAttribute('disabled', '');
         assertTrue(testElement.disabled);
         assertTrue(testElement.$.checkbox.disabled);
 
         MockInteractions.tap(testElement.$.checkbox);
         assertFalse(testElement.checked);
         assertFalse(testElement.$.checkbox.checked);
       });
+
+      test('numerical pref', function() {
+        var prefNum = {
+          key: 'test',
+          type: chrome.settingsPrivate.PrefType.NUMBER,
+          value: 1
+        };
+
+        var testElementNum = document.createElement('settings-checkbox');
+        testElementNum.set('pref', prefNum);

[Dan Beam, 2015/12/02 21:48:57]

why can't we just re-use the testElement fixture and call

  testElement.set('pref', prefNum);

?

[stevenjb, 2015/12/02 22:07:27]

We can. Done.

+        document.body.appendChild(testElementNum);

[Dan Beam, 2015/12/02 21:48:57]

why does this need to be inserted into the DOM?

[stevenjb, 2015/12/02 22:07:27]

Acknowledged.

+
+        assertTrue(testElementNum.checked);
+
+        testElementNum.removeAttribute('checked');
+        assertFalse(testElementNum.checked);
+        assertEquals(0, prefNum.value);
+
+        testElementNum.setAttribute('checked', '');
+        assertTrue(testElementNum.checked);
+        assertEquals(1, prefNum.value);
+      });
     });
   }
 
   return {
     registerTests: registerTests,
   };
 });