Skip to content

Commit 3522fd9

Browse files
committed
Selectmenu: Convert tests to new infrastructure
Ref #10119 Ref jquerygh-1528
1 parent b862534 commit 3522fd9

File tree

6 files changed

+32
-37
lines changed

6 files changed

+32
-37
lines changed

tests/unit/selectmenu/selectmenu.html

+2-26
Original file line numberDiff line numberDiff line change
@@ -4,32 +4,8 @@
44
<meta charset="utf-8">
55
<title>jQuery UI Selectmenu Test Suite</title>
66

7-
<script src="../../jquery.js"></script>
8-
<link rel="stylesheet" href="../../../external/qunit/qunit.css">
9-
<script src="../../../external/qunit/qunit.js"></script>
10-
<script src="../../../external/jquery-simulate/jquery.simulate.js"></script>
11-
<script src="../testsuite.js"></script>
12-
<script src="../../../external/qunit-assert-classes/qunit-assert-classes.js"></script>
13-
<script>
14-
TestHelpers.loadResources({
15-
css: [ "core", "menu" , "selectmenu" ],
16-
js: [
17-
"ui/core.js",
18-
"ui/widget.js",
19-
"ui/position.js",
20-
"ui/menu.js",
21-
"ui/selectmenu.js"
22-
]
23-
});
24-
</script>
25-
26-
<script src="selectmenu_common.js"></script>
27-
<script src="selectmenu_core.js"></script>
28-
<script src="selectmenu_events.js"></script>
29-
<script src="selectmenu_methods.js"></script>
30-
<script src="selectmenu_options.js"></script>
31-
32-
<script src="../swarminject.js"></script>
7+
<script src="../../lib/css.js" data-modules="core menu selectmenu"></script>
8+
<script src="../../lib/bootstrap.js" data-widget="selectmenu"></script>
339
</head>
3410
<body>
3511

tests/unit/selectmenu/selectmenu_common.js

+8-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,9 @@
1-
TestHelpers.commonWidgetTests( "selectmenu", {
1+
define( [
2+
"lib/common",
3+
"ui/selectmenu"
4+
], function( common ) {
5+
6+
common.testWidget( "selectmenu", {
27
defaults: {
38
appendTo: null,
49
classes: {
@@ -25,3 +30,5 @@ TestHelpers.commonWidgetTests( "selectmenu", {
2530
select: null
2631
}
2732
});
33+
34+
} );

tests/unit/selectmenu/selectmenu_core.js

+5-2
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,7 @@
1-
(function( $ ) {
1+
define( [
2+
"jquery",
3+
"ui/selectmenu"
4+
], function( $ ) {
25

36
module( "selectmenu: core" );
47

@@ -326,4 +329,4 @@ $.each([
326329
});
327330
});
328331

329-
})( jQuery );
332+
} );

tests/unit/selectmenu/selectmenu_events.js

+5-2
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,7 @@
1-
(function ( $ ) {
1+
define( [
2+
"jquery",
3+
"ui/selectmenu"
4+
], function( $ ) {
25

36
module( "selectmenu: events", {
47
setup: function() {
@@ -130,4 +133,4 @@ asyncTest( "select", function() {
130133
});
131134
});
132135

133-
})( jQuery );
136+
} );

tests/unit/selectmenu/selectmenu_methods.js

+7-4
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,13 @@
1-
(function( $ ) {
1+
define( [
2+
"jquery",
3+
"ui/selectmenu"
4+
], function( $ ) {
25

36
module( "selectmenu: methods" );
47

5-
test( "destroy", function() {
8+
test( "destroy", function( assert ) {
69
expect( 1 );
7-
domEqual( "#speed", function() {
10+
assert.domEqual( "#speed", function() {
811
$( "#speed" ).selectmenu().selectmenu( "destroy" );
912
});
1013
});
@@ -183,4 +186,4 @@ test( "widget and menuWidget", function( assert ) {
183186
ok( menu.is( "ul.ui-menu" ), "Menu Widget: element and class" );
184187
});
185188

186-
})( jQuery );
189+
} );

tests/unit/selectmenu/selectmenu_options.js

+5-2
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,7 @@
1-
(function ( $ ) {
1+
define( [
2+
"jquery",
3+
"ui/selectmenu"
4+
], function( $ ) {
25

36
module( "selectmenu: options" );
47

@@ -122,4 +125,4 @@ test( "width", function() {
122125
equal( button.outerWidth(), 300, "button width fills container" );
123126
});
124127

125-
})( jQuery );
128+
} );

0 commit comments

Comments
 (0)