From 6833a3169775d4c15dd5e68c96bc63ad0187035e Mon Sep 17 00:00:00 2001 From: Jyoti Deka Date: Sat, 19 Oct 2013 00:56:20 -0400 Subject: Slider: Don't allow a slider's value to exceed its max Fixes #9376 Closes gh-1016 --- tests/unit/slider/slider_options.js | 22 +++++++++++++++++++--- 1 file changed, 19 insertions(+), 3 deletions(-) (limited to 'tests') diff --git a/tests/unit/slider/slider_options.js b/tests/unit/slider/slider_options.js index 587f847cf..defb6f3b0 100644 --- a/tests/unit/slider/slider_options.js +++ b/tests/unit/slider/slider_options.js @@ -40,7 +40,7 @@ test( "disabled", function(){ }); test( "max", function() { - expect( 2 ); + expect( 4 ); element = $( "
" ); options = { @@ -52,8 +52,24 @@ test( "max", function() { }; element.slider( options ); - ok(element.slider( "option", "value" ) === options.value, "value option is not contained by max" ); - ok(element.slider( "value" ) === options.max, "value method is contained by max" ); + ok( element.slider( "option", "value" ) === options.value, "value option is not contained by max" ); + ok( element.slider( "value" ) === options.max, "value method is contained by max" ); + + options = { + max: 9, + min: 1, + orientation: "horizontal", + step: 3, + value: 8.75 + }; + + element.slider( options ); + ok( element.slider( "value" ) === 7, "value method is within max, edge Case" ); + + options.step = 2; + + element.slider( options ); + ok( element.slider( "value" ) === options.max, "value method will max, step is changed" ); element.slider( "destroy" ); }); -- cgit v1.2.3