r2987 commited - spinner: updated unit and visual tests

r2987 commited - spinner: updated unit and visual tests


Revision: 2987
Author: pazu2k@gmail.com
Date: Sun Jul 26 17:00:38 2009
Log: spinner: updated unit and visual tests
http://code.google.com/p/jquery-ui/source/detail?r=2987
Modified:
/branches/dev/spinner/tests/unit/spinner/spinner.js
/branches/dev/spinner/tests/unit/spinner/spinner_defaults.js
/branches/dev/spinner/tests/visual/spinner/spinner_option_readOnly.html
=======================================
--- /branches/dev/spinner/tests/unit/spinner/spinner.js    Sun Jul 19 04:21:07
2009
+++ /branches/dev/spinner/tests/unit/spinner/spinner.js    Sun Jul 26 17:00:38
2009
@@ -6,7 +6,6 @@
var el,
    simulateKeyAction = function(el,key) {
        el.simulate("keydown",{keyCode:$.simulate[key]})
-            .simulate("keypress",{keyCode:$.simulate[key]})
            .simulate("keyup",{keyCode:$.simulate[key]});
    };
@@ -129,15 +128,13 @@
    equals(el.val(), "-$50.00", "Home key to min");
-    el.simulate("keydown",{keyCode:$.simulate.VK_UP});
-
    for ( var i = 1 ; i<=120 ; i++ ) {
-        el.simulate("keypress",{keyCode:$.simulate.VK_UP});
+        el.simulate("keydown",{keyCode:$.simulate.VK_UP});
    }
    el.simulate("keyup",{keyCode:$.simulate.VK_UP});
-    equals(el.val(), "-$14.00", "keypress 120 times");
+    equals(el.val(), "-$14.00", "keydown 120 times");
});
@@ -152,13 +149,12 @@
    equals(el.val(), "-0.7", "stepping 0.7");
-    el.simulate("keydown",{keyCode:$.simulate.VK_UP});
    for ( var i = 1 ; i<=11 ; i++ ) {
-        el.simulate("keypress",{keyCode:$.simulate.VK_UP});
+        el.simulate("keydown",{keyCode:$.simulate.VK_UP});
    }
    el.simulate("keyup",{keyCode:$.simulate.VK_UP});
-    equals(el.val(), "7.0", "keypress 11 times");
+    equals(el.val(), "7.0", "keydown 11 times");
});
test("spin without auto-incremental stepping", function() {
@@ -166,21 +162,19 @@
    el = $("#spin").spinner({ incremental:false });
-    el.simulate("keydown",{keyCode:$.simulate.VK_UP});
    for ( var i = 1 ; i<=120 ; i++ ) {
-        el.simulate("keypress",{keyCode:$.simulate.VK_UP});
+        el.simulate("keydown",{keyCode:$.simulate.VK_UP});
    }
    el.simulate("keyup",{keyCode:$.simulate.VK_UP});
-    equals(el.val(), 120, "keypress 120 times");
-
-    el.simulate("keydown",{keyCode:$.simulate.VK_DOWN});
+    equals(el.val(), 120, "keydown 120 times");
+
    for ( var i = 1 ; i<=210 ; i++ ) {
-        el.simulate("keypress",{keyCode:$.simulate.VK_DOWN});
+        el.simulate("keydown",{keyCode:$.simulate.VK_DOWN});
    }
    el.simulate("keyup",{keyCode:$.simulate.VK_DOWN});
-    equals(el.val(), -90, "keypress 210 times");
+    equals(el.val(), -90, "keydown 210 times");
});
@@ -189,43 +183,46 @@
    el = $("#spin").spinner();
-    el.simulate("keydown",{keyCode:$.simulate.VK_UP});
    for ( var i = 1 ; i<=120 ; i++ ) {
-        el.simulate("keypress",{keyCode:$.simulate.VK_UP});
+            el.simulate("keydown",{keyCode:$.simulate.VK_UP});
    }
    el.simulate("keyup",{keyCode:$.simulate.VK_UP});
-    equals(el.val(), 300, "keypress 120 times (100+20*10)");
-
-    el.simulate("keydown",{keyCode:$.simulate.VK_DOWN});
+    equals(el.val(), 300, "keydown 120 times (100+20*10)");
+
    for ( var i = 1 ; i<=210 ; i++ ) {
-        el.simulate("keypress",{keyCode:$.simulate.VK_DOWN});
+            el.simulate("keydown",{keyCode:$.simulate.VK_DOWN});
    }
    el.simulate("keyup",{keyCode:$.simulate.VK_DOWN});
-    equals(el.val(), -1800, "keypress 210 times (300-100-100*10-10*100)");
+    equals(el.val(), -1800, "keydown 210 times (300-100-100*10-10*100)");
});
test("mouse click on buttons", function() {
-    expect(4);
-    el = $("#spin").spinner();
-
+
+    el = $("#spin").spinner();
+    val = 0;
+
    $(".ui-spinner-up").trigger("mousedown").trigger("mouseup");
-    equals(el.val(), 1, "mouse click to up");
-
-    $(".ui-spinner-up").trigger("dblclick");
-
-    equals(el.val(), 2, "mouse double click to up");
+    equals(el.val(), ++val, "mouse click to up");
+
+    if ($.browser.msie) {
+        $(".ui-spinner-up").trigger("dblclick");
+
+        equals(el.val(), ++val, "mouse double click to up");
+    }
    $(".ui-spinner-down").trigger("mousedown").trigger("mouseup");
-    equals(el.val(), 1, "mouse click to down");
-
-    $(".ui-spinner-down").trigger("dblclick");
-
-    equals(el.val(), 0, "mouse double click to down");
+    equals(el.val(), --val, "mouse click to down");
+
+    if ($.browser.msie) {
+        $(".ui-spinner-down").trigger("dblclick");
+
+        equals(el.val(), --val, "mouse double click to down");
+    }
});
@@ -317,16 +314,38 @@
});
-test("reading from HTML5 attributes", function() {
-    expect(0);
-});
-
-test("method - paging", function() {
-    expect(0);
+test("reading HTML5 attributes", function() {
+
+    el = $('<input id="spinner" type="number" min="-100" max="100" value="5"
step="2">').spinner();
+
+    equals(el.spinner('option', 'value'), 5, 'value');
+    equals(el.spinner('option', 'max'), 100, 'max');
+    equals(el.spinner('option', 'min'), -100, 'min');
+    equals(el.spinner('option', 'step'), 2, 'step');
+    equals(el.spinner('option', 'page'), 5, 'page');
+
});
-test("method - stepping", function() {
-    expect(0);
+test("methods - paging and stepping", function() {
+
+    el = $('#spin').spinner({ step: 2, page: 5, value: 0 });
+
+    el.spinner('pageUp');
+
+    equals(el.val(), 10, "pageUp 1 page");
+
+    el.spinner('pageDown', 2);
+
+    equals(el.val(), -10, "pageDown 2 pages");
+
+    el.spinner('stepUp', 10)
+
+    equals(el.val(), 10, "stepUp 10 steps");
+
+    el.spinner('stepDown', 5)
+
+    equals(el.val(), 0, "stepDown 5 steps");
+
});
=======================================
--- /branches/dev/spinner/tests/unit/spinner/spinner_defaults.js    Wed Jul 15
23:42:33 2009
+++ /branches/dev/spinner/tests/unit/spinner/spinner_defaults.js    Sun Jul 26
17:00:38 2009
@@ -3,16 +3,14 @@
*/
var spinner_defaults = {
+    buttons: 'show',
    disabled: false,
    currency: false,
    dir: 'ltr',
-    format: '%',
    groupSeparator: '',
-    hide: false,
    incremental: true,
-    items: null,
-    max: Number.MAX_VALUE,
-    min: -Number.MAX_VALUE,
+    max: null,
+    min: null,
    mouseWheel: true,
    padLength: 0,
    page: 5,
@@ -21,7 +19,7 @@
    radixPoint: '.',
    readOnly: null,
    spinnerClass: null,
-    step: 1,
+    step: null,
    value: 0,
    width: false
};
=======================================
--- /branches/dev/spinner/tests/visual/spinner/spinner_option_readOnly.html    
Sat Jun 6 12:07:15 2009
+++ /branches/dev/spinner/tests/visual/spinner/spinner_option_readOnly.html    
Sun Jul 26 17:00:38 2009
@@ -19,7 +19,7 @@
<body>
<div class="ui-widget">
-    

<label>HTML attribute readOnly:</label> <input id="spinner1"
type="text" readonly="readonly">


+    

<label>HTML attribute readOnly:</label> <input id="spinner1"
type="text" readonly="readonly" value="100">


    

<label>Spinner option readOnly:</label> <input id="spinner2"
type="text">


</div>