Skip to content

Commit 8034cc3

Browse files
scottgonzalezscottgonzalez
committed
Widget tests: Cleanup.
1 parent 2e214f9 commit 8034cc3

File tree

3 files changed

+26
-32
lines changed

3 files changed

+26
-32
lines changed

‎tests/unit/widget/widget.html

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,22 +1,22 @@
11
<!DOCTYPE html>
22
<html lang="en">
33
<head>
4-
<meta charset="UTF-8" />
4+
<meta charset="utf-8">
55
<title>jQuery UI Widget Test Suite</title>
66

7-
<script type="text/javascript" src="../../../jquery-1.4.4.js"></script>
8-
<script type="text/javascript" src="../../../ui/jquery.ui.core.js"></script>
9-
<script type="text/javascript" src="../../../ui/jquery.ui.widget.js"></script>
7+
<script src="../../../jquery-1.4.4.js"></script>
8+
<script src="../../../ui/jquery.ui.core.js"></script>
9+
<script src="../../../ui/jquery.ui.widget.js"></script>
1010

11-
<link rel="stylesheet" href="../../../external/qunit.css" type="text/css"/>
12-
<script type="text/javascript" src="../../../external/qunit.js"></script>
13-
<script type="text/javascript" src="../../jquery.simulate.js"></script>
14-
<script type="text/javascript" src="../testsuite.js"></script>
11+
<link rel="stylesheet" href="../../../external/qunit.css">
12+
<script src="../../../external/qunit.js"></script>
13+
<script src="../../jquery.simulate.js"></script>
14+
<script src="../testsuite.js"></script>
1515

16-
<script type="text/javascript" src="widget_core.js"></script>
17-
<script type="text/javascript" src="widget_tickets.js"></script>
18-
19-
<script type="text/javascript" src="../swarminject.js"></script>
16+
<script src="widget_core.js"></script>
17+
<script src="widget_tickets.js"></script>
18+
19+
<script src="../swarminject.js"></script>
2020
</head>
2121
<body>
2222

‎tests/unit/widget/widget_core.js

Lines changed: 13 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,3 @@
1-
/*
2-
* widget unit tests
3-
*/
41
(function( $ ) {
52

63
module( "widget factory", {
@@ -104,7 +101,7 @@ test( "jQuery usage", function() {
104101
});
105102

106103
shouldCreate = true;
107-
var elem = $( "<div></div>" )
104+
var elem = $( "<div>" )
108105
.bind( "testwidgetcreate", function() {
109106
ok( shouldCreate, "create event triggered on instantiation" );
110107
})
@@ -150,7 +147,7 @@ test( "direct usage", function() {
150147
}
151148
});
152149

153-
var elem = $( "<div></div>" )[ 0 ];
150+
var elem = $( "<div>" )[ 0 ];
154151

155152
shouldCreate = true;
156153
var instance = new $.ui.testWidget( {}, elem );
@@ -202,7 +199,7 @@ test("merge multiple option arguments", function() {
202199
});
203200
}
204201
});
205-
$( "<div></div>" ).testWidget({
202+
$( "<div>" ).testWidget({
206203
option1: "valuex",
207204
option2: "valuex",
208205
option3: "value3",
@@ -249,7 +246,7 @@ test( "_getCreateOptions()", function() {
249246
});
250247

251248
test( "re-init", function() {
252-
var div = $( "<div></div>" ),
249+
var div = $( "<div>" ),
253250
actions = [];
254251

255252
$.widget( "ui.testWidget", {
@@ -335,7 +332,7 @@ test( ".option() - getter", function() {
335332
_create: function() {}
336333
});
337334

338-
var div = $( "<div></div>" ).testWidget({
335+
var div = $( "<div>" ).testWidget({
339336
foo: "bar",
340337
baz: 5,
341338
qux: [ "quux", "quuux" ]
@@ -366,7 +363,7 @@ test( ".option() - delegate to ._setOptions()", function() {
366363
calls.push( options );
367364
}
368365
});
369-
var div = $( "<div></div>" ).testWidget();
366+
var div = $( "<div>" ).testWidget();
370367

371368
calls = [];
372369
div.testWidget( "option", "foo", "bar" );
@@ -392,7 +389,7 @@ test( ".option() - delegate to ._setOption()", function() {
392389
});
393390
}
394391
});
395-
var div = $( "<div></div>" ).testWidget();
392+
var div = $( "<div>" ).testWidget();
396393

397394
calls = [];
398395
div.testWidget( "option", "foo", "bar" );
@@ -419,7 +416,7 @@ test( ".enable()", function() {
419416
same( val, false, "disabled set to false" );
420417
}
421418
});
422-
$( "<div></div>" ).testWidget().testWidget( "enable" );
419+
$( "<div>" ).testWidget().testWidget( "enable" );
423420
});
424421

425422
test( ".disable()", function() {
@@ -431,26 +428,26 @@ test( ".disable()", function() {
431428
same( val, true, "disabled set to true" );
432429
}
433430
});
434-
$( "<div></div>" ).testWidget().testWidget( "disable" );
431+
$( "<div>" ).testWidget().testWidget( "disable" );
435432
});
436433

437434
test( ".widget() - base", function() {
438435
$.widget( "ui.testWidget", {
439436
_create: function() {}
440437
});
441-
var div = $( "<div></div>" ).testWidget();
438+
var div = $( "<div>" ).testWidget();
442439
same( div[0], div.testWidget( "widget" )[0]);
443440
});
444441

445442
test( ".widget() - overriden", function() {
446-
var wrapper = $( "<div></div>" );
443+
var wrapper = $( "<div>" );
447444
$.widget( "ui.testWidget", {
448445
_create: function() {},
449446
widget: function() {
450447
return wrapper;
451448
}
452449
});
453-
same( wrapper[0], $( "<div></div>" ).testWidget().testWidget( "widget" )[0] );
450+
same( wrapper[0], $( "<div>" ).testWidget().testWidget( "widget" )[0] );
454451
});
455452

456453
test( "._trigger() - no event, no ui", function() {
@@ -670,4 +667,4 @@ test( "auto-destroy - .detach()", function() {
670667
$( "#widget" ).testWidget().detach();
671668
});
672669

673-
})( jQuery );
670+
}( jQuery ) );

‎tests/unit/widget/widget_tickets.js

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,3 @@
1-
/*
2-
* widget unit tests
3-
*/
41
(function( $ ) {
52

63
module( "widget: tickets" );
@@ -43,4 +40,4 @@ test( "#5830 - Widget: Using inheritance overwrites the base classes options", f
4340
delete $.ui.testWidgetExtension;
4441
});
4542

46-
})( jQuery );
43+
}( jQuery ) );

0 commit comments

Comments
 (0)