Skip to content

Commit b5028f0

Browse files
committed
Spinner: Shift to use no globals
1 parent 749f95e commit b5028f0

File tree

5 files changed

+257
-250
lines changed

5 files changed

+257
-250
lines changed

tests/unit/spinner/core.js

Lines changed: 88 additions & 86 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,16 @@
11
define( [
2+
"qunit",
23
"jquery",
34
"./helper",
45
"ui/widgets/spinner"
5-
], function( $, testHelper ) {
6+
], function( QUnit, $, testHelper ) {
67

78
var simulateKeyDownUp = testHelper.simulateKeyDownUp;
89

9-
module( "spinner: core" );
10+
QUnit.module( "spinner: core" );
1011

11-
test( "markup structure", function( assert ) {
12-
expect( 6 );
12+
QUnit.test( "markup structure", function( assert ) {
13+
assert.expect( 6 );
1314
var element = $( "#spin" ).spinner(),
1415
spinner = element.spinner( "widget" ),
1516
up = spinner.find( ".ui-spinner-up" ),
@@ -18,101 +19,102 @@ test( "markup structure", function( assert ) {
1819
assert.hasClasses( element, "ui-spinner-input" );
1920
assert.hasClasses( spinner, "ui-spinner ui-widget ui-widget-content" );
2021
assert.hasClasses( up, "ui-spinner-button ui-spinner-up ui-widget" );
21-
equal( up.length, 1, "Spinner contains exactly one up button" );
22+
assert.equal( up.length, 1, "Spinner contains exactly one up button" );
2223
assert.hasClasses( down, "ui-spinner-button ui-spinner-down ui-widget" );
23-
equal( down.length, 1, "Spinner contains exactly one down button" );
24+
assert.equal( down.length, 1, "Spinner contains exactly one down button" );
2425
} );
2526

26-
test( "keydown UP on input, increases value not greater than max", function() {
27-
expect( 5 );
27+
QUnit.test( "keydown UP on input, increases value not greater than max", function( assert ) {
28+
assert.expect( 5 );
2829
var element = $( "#spin" ).val( 70 ).spinner( {
2930
max: 100,
3031
step: 10
3132
} );
3233

3334
simulateKeyDownUp( element, $.ui.keyCode.UP );
34-
equal( element.val(), 80 );
35+
assert.equal( element.val(), 80 );
3536
simulateKeyDownUp( element, $.ui.keyCode.UP );
36-
equal( element.val(), 90 );
37+
assert.equal( element.val(), 90 );
3738
simulateKeyDownUp( element, $.ui.keyCode.UP );
38-
equal( element.val(), 100 );
39+
assert.equal( element.val(), 100 );
3940
simulateKeyDownUp( element, $.ui.keyCode.UP );
40-
equal( element.val(), 100 );
41+
assert.equal( element.val(), 100 );
4142
simulateKeyDownUp( element, $.ui.keyCode.UP );
42-
equal( element.val(), 100 );
43+
assert.equal( element.val(), 100 );
4344
} );
4445

45-
test( "keydown DOWN on input, decreases value not less than min", function() {
46-
expect( 5 );
46+
QUnit.test( "keydown DOWN on input, decreases value not less than min", function( assert ) {
47+
assert.expect( 5 );
4748
var element = $( "#spin" ).val( 50 ).spinner( {
4849
min: 20,
4950
step: 10
5051
} );
5152

5253
simulateKeyDownUp( element, $.ui.keyCode.DOWN );
53-
equal( element.val(), 40 );
54+
assert.equal( element.val(), 40 );
5455
simulateKeyDownUp( element, $.ui.keyCode.DOWN );
55-
equal( element.val(), 30 );
56+
assert.equal( element.val(), 30 );
5657
simulateKeyDownUp( element, $.ui.keyCode.DOWN );
57-
equal( element.val(), 20 );
58+
assert.equal( element.val(), 20 );
5859
simulateKeyDownUp( element, $.ui.keyCode.DOWN );
59-
equal( element.val(), 20 );
60+
assert.equal( element.val(), 20 );
6061
simulateKeyDownUp( element, $.ui.keyCode.DOWN );
61-
equal( element.val(), 20 );
62+
assert.equal( element.val(), 20 );
6263
} );
6364

64-
test( "keydown PAGE_UP on input, increases value not greater than max", function() {
65-
expect( 5 );
65+
QUnit.test( "keydown PAGE_UP on input, increases value not greater than max", function( assert ) {
66+
assert.expect( 5 );
6667
var element = $( "#spin" ).val( 70 ).spinner( {
6768
max: 100,
6869
page: 10
6970
} );
7071

7172
simulateKeyDownUp( element, $.ui.keyCode.PAGE_UP );
72-
equal( element.val(), 80 );
73+
assert.equal( element.val(), 80 );
7374
simulateKeyDownUp( element, $.ui.keyCode.PAGE_UP );
74-
equal( element.val(), 90 );
75+
assert.equal( element.val(), 90 );
7576
simulateKeyDownUp( element, $.ui.keyCode.PAGE_UP );
76-
equal( element.val(), 100 );
77+
assert.equal( element.val(), 100 );
7778
simulateKeyDownUp( element, $.ui.keyCode.PAGE_UP );
78-
equal( element.val(), 100 );
79+
assert.equal( element.val(), 100 );
7980
simulateKeyDownUp( element, $.ui.keyCode.PAGE_UP );
80-
equal( element.val(), 100 );
81+
assert.equal( element.val(), 100 );
8182
} );
8283

83-
test( "keydown PAGE_DOWN on input, decreases value not less than min", function() {
84-
expect( 5 );
84+
QUnit.test( "keydown PAGE_DOWN on input, decreases value not less than min", function( assert ) {
85+
assert.expect( 5 );
8586
var element = $( "#spin" ).val( 50 ).spinner( {
8687
min: 20,
8788
page: 10
8889
} );
8990

9091
simulateKeyDownUp( element, $.ui.keyCode.PAGE_DOWN );
91-
equal( element.val(), 40 );
92+
assert.equal( element.val(), 40 );
9293
simulateKeyDownUp( element, $.ui.keyCode.PAGE_DOWN );
93-
equal( element.val(), 30 );
94+
assert.equal( element.val(), 30 );
9495
simulateKeyDownUp( element, $.ui.keyCode.PAGE_DOWN );
95-
equal( element.val(), 20 );
96+
assert.equal( element.val(), 20 );
9697
simulateKeyDownUp( element, $.ui.keyCode.PAGE_DOWN );
97-
equal( element.val(), 20 );
98+
assert.equal( element.val(), 20 );
9899
simulateKeyDownUp( element, $.ui.keyCode.PAGE_DOWN );
99-
equal( element.val(), 20 );
100+
assert.equal( element.val(), 20 );
100101
} );
101102

102-
asyncTest( "blur input while spinning with UP", function() {
103-
expect( 3 );
103+
QUnit.test( "blur input while spinning with UP", function( assert ) {
104+
var ready = assert.async();
105+
assert.expect( 3 );
104106
var value,
105107
element = $( "#spin" ).val( 10 ).spinner();
106108

107109
function step1() {
108110
element.simulate( "keydown", { keyCode: $.ui.keyCode.UP } );
109-
equal( element.val(), 11 );
111+
assert.equal( element.val(), 11 );
110112
setTimeout( step2, 750 );
111113
}
112114

113115
function step2() {
114116
value = element.val();
115-
ok( value > 11, "repeating while key is down" );
117+
assert.ok( value > 11, "repeating while key is down" );
116118

117119
element.on( "blur", function() {
118120
value = element.val();
@@ -121,137 +123,137 @@ asyncTest( "blur input while spinning with UP", function() {
121123
}
122124

123125
function step3() {
124-
equal( element.val(), value, "stopped repeating on blur" );
125-
start();
126+
assert.equal( element.val(), value, "stopped repeating on blur" );
127+
ready();
126128
}
127129

128130
element[ 0 ].focus();
129131
setTimeout( step1 );
130132
} );
131133

132-
test( "mouse click on up button, increases value not greater than max", function() {
133-
expect( 3 );
134+
QUnit.test( "mouse click on up button, increases value not greater than max", function( assert ) {
135+
assert.expect( 3 );
134136
var element = $( "#spin" ).val( 18 ).spinner( {
135137
max: 20
136138
} ),
137139
button = element.spinner( "widget" ).find( ".ui-spinner-up" );
138140

139141
button.trigger( "mousedown" ).trigger( "mouseup" );
140-
equal( element.val(), 19 );
142+
assert.equal( element.val(), 19 );
141143
button.trigger( "mousedown" ).trigger( "mouseup" );
142-
equal( element.val(), 20 );
144+
assert.equal( element.val(), 20 );
143145
button.trigger( "mousedown" ).trigger( "mouseup" );
144-
equal( element.val(), 20 );
146+
assert.equal( element.val(), 20 );
145147
} );
146148

147-
test( "mouse click on up button, increases value not greater than max", function() {
148-
expect( 3 );
149+
QUnit.test( "mouse click on up button, increases value not greater than max", function( assert ) {
150+
assert.expect( 3 );
149151
var element = $( "#spin" ).val( 2 ).spinner( {
150152
min: 0
151153
} ),
152154
button = element.spinner( "widget" ).find( ".ui-spinner-down" );
153155

154156
button.trigger( "mousedown" ).trigger( "mouseup" );
155-
equal( element.val(), 1 );
157+
assert.equal( element.val(), 1 );
156158
button.trigger( "mousedown" ).trigger( "mouseup" );
157-
equal( element.val(), 0 );
159+
assert.equal( element.val(), 0 );
158160
button.trigger( "mousedown" ).trigger( "mouseup" );
159-
equal( element.val(), 0 );
161+
assert.equal( element.val(), 0 );
160162
} );
161163

162-
test( "mousewheel on input", function() {
163-
expect( 4 );
164+
QUnit.test( "mousewheel on input", function( assert ) {
165+
assert.expect( 4 );
164166

165167
var element = $( "#spin" ).val( 0 ).spinner( {
166168
step: 2
167169
} );
168170

169171
element.trigger( "mousewheel" );
170-
equal( element.val(), 0, "mousewheel event without delta does not change value" );
172+
assert.equal( element.val(), 0, "mousewheel event without delta does not change value" );
171173

172174
element.trigger( "mousewheel", 1 );
173-
equal( element.val(), 2 );
175+
assert.equal( element.val(), 2 );
174176

175177
element.trigger( "mousewheel", -0.2 );
176-
equal( element.val(), 0 );
178+
assert.equal( element.val(), 0 );
177179

178180
element.trigger( "mousewheel", -15 );
179-
equal( element.val(), -2 );
181+
assert.equal( element.val(), -2 );
180182
} );
181183

182-
test( "reading HTML5 attributes", function() {
183-
expect( 6 );
184+
QUnit.test( "reading HTML5 attributes", function( assert ) {
185+
assert.expect( 6 );
184186
var markup = "<input type='number' min='-100' max='100' value='5' step='2'>",
185187
element = $( markup ).spinner();
186-
equal( element.spinner( "option", "min" ), -100, "min from markup" );
187-
equal( element.spinner( "option", "max" ), 100, "max from markup" );
188-
equal( element.spinner( "option", "step" ), 2, "step from markup" );
188+
assert.equal( element.spinner( "option", "min" ), -100, "min from markup" );
189+
assert.equal( element.spinner( "option", "max" ), 100, "max from markup" );
190+
assert.equal( element.spinner( "option", "step" ), 2, "step from markup" );
189191

190192
element = $( markup ).spinner( {
191193
min: -200,
192194
max: 200,
193195
step: 5
194196
} );
195-
equal( element.spinner( "option", "min" ), -200, "min from options" );
196-
equal( element.spinner( "option", "max" ), 200, "max from options" );
197-
equal( element.spinner( "option", "step" ), 5, "stop from options" );
197+
assert.equal( element.spinner( "option", "min" ), -200, "min from options" );
198+
assert.equal( element.spinner( "option", "max" ), 200, "max from options" );
199+
assert.equal( element.spinner( "option", "step" ), 5, "stop from options" );
198200
} );
199201

200-
test( "ARIA attributes", function() {
201-
expect( 9 );
202+
QUnit.test( "ARIA attributes", function( assert ) {
203+
assert.expect( 9 );
202204
var element = $( "#spin" ).val( 2 ).spinner( { min: -5, max: 5 } );
203205

204-
equal( element.attr( "role" ), "spinbutton", "role" );
205-
equal( element.attr( "aria-valuemin" ), "-5", "aria-valuemin" );
206-
equal( element.attr( "aria-valuemax" ), "5", "aria-valuemax" );
207-
equal( element.attr( "aria-valuenow" ), "2", "aria-valuenow" );
206+
assert.equal( element.attr( "role" ), "spinbutton", "role" );
207+
assert.equal( element.attr( "aria-valuemin" ), "-5", "aria-valuemin" );
208+
assert.equal( element.attr( "aria-valuemax" ), "5", "aria-valuemax" );
209+
assert.equal( element.attr( "aria-valuenow" ), "2", "aria-valuenow" );
208210

209211
element.spinner( "stepUp" );
210212

211-
equal( element.attr( "aria-valuenow" ), "3", "stepUp 1 step changes aria-valuenow" );
213+
assert.equal( element.attr( "aria-valuenow" ), "3", "stepUp 1 step changes aria-valuenow" );
212214

213215
element.spinner( "option", { min: -10, max: 10 } );
214216

215-
equal( element.attr( "aria-valuemin" ), "-10", "min option changed aria-valuemin changes" );
216-
equal( element.attr( "aria-valuemax" ), "10", "max option changed aria-valuemax changes" );
217+
assert.equal( element.attr( "aria-valuemin" ), "-10", "min option changed aria-valuemin changes" );
218+
assert.equal( element.attr( "aria-valuemax" ), "10", "max option changed aria-valuemax changes" );
217219

218220
element.spinner( "option", "min", null );
219-
equal( element.attr( "aria-valuemin" ), undefined, "aria-valuemin not set when no min" );
221+
assert.equal( element.attr( "aria-valuemin" ), undefined, "aria-valuemin not set when no min" );
220222

221223
element.spinner( "option", "max", null );
222-
equal( element.attr( "aria-valuemax" ), undefined, "aria-valuemax not set when no max" );
224+
assert.equal( element.attr( "aria-valuemax" ), undefined, "aria-valuemax not set when no max" );
223225
} );
224226

225-
test( "focus text field when pressing button", function() {
226-
expect( 2 );
227+
QUnit.test( "focus text field when pressing button", function( assert ) {
228+
assert.expect( 2 );
227229
var element = $( "#spin" ).spinner();
228230
$( "body" ).trigger( "focus" );
229-
ok( element[ 0 ] !== document.activeElement, "not focused before" );
231+
assert.ok( element[ 0 ] !== document.activeElement, "not focused before" );
230232
element.spinner( "widget" ).find( ".ui-spinner-up" ).trigger( "mousedown" );
231-
ok( element[ 0 ] === document.activeElement, "focused after" );
233+
assert.ok( element[ 0 ] === document.activeElement, "focused after" );
232234
} );
233235

234-
test( "don't clear invalid value on blur", function() {
235-
expect( 1 );
236+
QUnit.test( "don't clear invalid value on blur", function( assert ) {
237+
assert.expect( 1 );
236238
var element = $( "#spin" ).spinner();
237239
element.trigger( "focus" ).val( "a" ).trigger( "blur" );
238-
equal( element.val(), "a" );
240+
assert.equal( element.val(), "a" );
239241
} );
240242

241-
test( "precision", function() {
242-
expect( 2 );
243+
QUnit.test( "precision", function( assert ) {
244+
assert.expect( 2 );
243245
var element = $( "#spin" ).val( 0.05 ).spinner( {
244246
step: 0.0001
245247
} );
246248
element.spinner( "stepUp" );
247-
equal( element.val(), "0.0501", "precision from step" );
249+
assert.equal( element.val(), "0.0501", "precision from step" );
248250

249251
element.val( 1.05 ).spinner( "option", {
250252
step: 1,
251253
min: -9.95
252254
} );
253255
element.spinner( "stepDown" );
254-
equal( element.val(), "0.05", "precision from min" );
256+
assert.equal( element.val(), "0.05", "precision from min" );
255257
} );
256258

257259
} );

tests/unit/spinner/deprecated.js

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,12 @@
11
define( [
2+
"qunit",
23
"jquery",
34
"ui/widgets/spinner"
4-
], function( $ ) {
5+
], function( QUnit, $ ) {
56

67
var originalSpinner = $.ui.spinner.prototype;
7-
module( "spinner: deprecated", {
8-
setup: function() {
8+
QUnit.module( "spinner: deprecated", {
9+
beforeEach: function() {
910
$.widget( "ui.spinner", $.ui.spinner, {
1011
_uiSpinnerHtml: function() {
1112
return "<span class='spin-wrap'>";
@@ -23,13 +24,13 @@ module( "spinner: deprecated", {
2324
} );
2425
},
2526

26-
teardown: function() {
27+
afterEach: function() {
2728
$.ui.spinner.prototype = originalSpinner;
2829
}
2930
} );
3031

31-
test( "markup structure - custom", function( assert ) {
32-
expect( 2 );
32+
QUnit.test( "markup structure - custom", function( assert ) {
33+
assert.expect( 2 );
3334
var element = $( "#spin" ).spinner(),
3435
spinner = element.spinner( "widget" ),
3536
up = spinner.find( ".ui-spinner-up" );

0 commit comments

Comments
 (0)