MD Settings: cr-slider: Make display consistent and clean up.

chrome/test/data/webui/cr_elements/cr_slider_tests.js

 // Copyright 2016 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 cr-slider. */
 cr.define('cr_slider', function() {
   function registerTests() {
     suite('cr-slider', function() {
       /** @type {!CrSliderElement} */
       var slider;
 
       /**
        * paper-slider instance wrapped by cr-slider.
        * @type {!PaperSliderElement}
        */
       var paperSlider;
 
       var tickValues = [2, 4, 8, 16, 32, 64, 128];
 
       setup(function() {
         PolymerTest.clearBody();
         slider = document.createElement('cr-slider');
         document.body.appendChild(slider);
         paperSlider = slider.$$('paper-slider');
       });
 
-      test('basic properties', function() {
-        // Default properties.
-        expectEquals(1, paperSlider.step);
-        expectEquals(0, paperSlider.min);
-
-        // Configurable properties.
-        expectFalse(paperSlider.snaps);
-        slider.setAttribute('snaps', '');
-        expectTrue(paperSlider.snaps);
-
-        slider.setAttribute('max-markers', 7);
-        expectEquals(7, paperSlider.maxMarkers);
-      });
-
       test('set value', function() {
         slider.tickValues = tickValues;
         slider.value = 16;
         expectEquals(6, paperSlider.max);
         expectEquals(3, paperSlider.value);
         expectEquals(3, paperSlider.immediateValue);
 
-        // Setting to an in-between value should choose an index but not change
-        // the value.
+        // cr-slider only supports snapping to a range of tick values. Setting
+        // to an in-between value should snap to an indexed value.
         slider.value = 70;
         expectEquals(5, paperSlider.value);
         expectEquals(5, paperSlider.immediateValue);
-        expectEquals(70, slider.value);
+        expectEquals(64, slider.value);
 
         // Setting the value out-of-range should clamp the slider.
         slider.value = -100;
         expectEquals(0, paperSlider.value);
         expectEquals(0, paperSlider.immediateValue);
-        expectEquals(-100, slider.value);
+        expectEquals(2, slider.value);
       });
 
       test('move slider', function() {
         slider.tickValues = tickValues;
         slider.value = 30;
         expectEquals(4, paperSlider.value);
 
         MockInteractions.pressAndReleaseKeyOn(
             paperSlider, 39 /* right */);
         expectEquals(5, paperSlider.value);
@@ skipping 41 matching lines @@
         testFindNearestIndex(5, 100);
         testFindNearestIndex(5, 91);
       });
     });
   }
 
   return {
     registerTests: registerTests,
   };
 });