Skip to content

Commit b862534

Browse files
committed
Selectable: Convert tests to new infrastructure
Ref #10119 Ref gh-1528
1 parent 4e7407c commit b862534

File tree

5 files changed

+27
-42
lines changed

5 files changed

+27
-42
lines changed

tests/unit/selectable/selectable.html

+2-25
Original file line numberDiff line numberDiff line change
@@ -4,31 +4,8 @@
44
<meta charset="utf-8">
55
<title>jQuery UI Selectable 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", "selectable" ],
16-
js: [
17-
"ui/core.js",
18-
"ui/widget.js",
19-
"ui/mouse.js",
20-
"ui/selectable.js"
21-
]
22-
});
23-
</script>
24-
25-
<script src="selectable_common.js"></script>
26-
<script src="selectable_core.js"></script>
27-
<script src="selectable_events.js"></script>
28-
<script src="selectable_methods.js"></script>
29-
<script src="selectable_options.js"></script>
30-
31-
<script src="../swarminject.js"></script>
7+
<script src="../../lib/css.js" data-modules="core selectable"></script>
8+
<script src="../../lib/bootstrap.js" data-widget="selectable"></script>
329
</head>
3310
<body>
3411

tests/unit/selectable/selectable_common.js

+8-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,9 @@
1-
TestHelpers.commonWidgetTests("selectable", {
1+
define( [
2+
"lib/common",
3+
"ui/selectable"
4+
], function( common ) {
5+
6+
common.testWidget( "selectable", {
27
defaults: {
38
appendTo: "body",
49
autoRefresh: true,
@@ -20,3 +25,5 @@ TestHelpers.commonWidgetTests("selectable", {
2025
unselecting: null
2126
}
2227
});
28+
29+
} );

tests/unit/selectable/selectable_events.js

+7-6
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
1-
/*
2-
* selectable_events.js
3-
*/
4-
(function( $ ) {
1+
define( [
2+
"jquery",
3+
"lib/helper",
4+
"ui/selectable"
5+
], function( $, testHelpers ) {
56

67
module("selectable: events");
78

@@ -40,7 +41,7 @@ test( "mousedown: initial position of helper", function() {
4041

4142
var helperOffset,
4243
element = $( "#selectable1" ).selectable(),
43-
contentToForceScroll = TestHelpers.forceScrollableWindow( "body" );
44+
contentToForceScroll = testHelpers.forceScrollableWindow( "body" );
4445

4546
$( window ).scrollTop( 100 ).scrollLeft( 100 );
4647

@@ -59,4 +60,4 @@ test( "mousedown: initial position of helper", function() {
5960
$( window ).scrollTop( 0 ).scrollLeft( 0 );
6061
});
6162

62-
})( jQuery );
63+
} );

tests/unit/selectable/selectable_methods.js

+5-5
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
1-
/*
2-
* selectable_methods.js
3-
*/
4-
(function($) {
1+
define( [
2+
"jquery",
3+
"ui/selectable"
4+
], function( $ ) {
55

66
module("selectable: methods");
77

@@ -108,4 +108,4 @@ test( "disable", function( assert ) {
108108
equal( chainable, element, "disable is chainable" );
109109
});
110110

111-
})(jQuery);
111+
} );

tests/unit/selectable/selectable_options.js

+5-5
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
1-
/*
2-
* selectable_options.js
3-
*/
4-
(function($) {
1+
define( [
2+
"jquery",
3+
"ui/selectable"
4+
], function( $ ) {
55

66
module("selectable: options");
77

@@ -61,4 +61,4 @@ test("filter", function() {
6161
el.selectable("destroy");
6262
});
6363

64-
})(jQuery);
64+
} );

0 commit comments

Comments
 (0)