Skip to content

Commit 9af91bc

Browse files
committed
Selectable: Use new has/lacksClasses assertions for all class checks
1 parent 5b359d0 commit 9af91bc

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

tests/unit/selectable/selectable.html

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
<script src="../../../external/qunit/qunit.js"></script>
1010
<script src="../../../external/jquery-simulate/jquery.simulate.js"></script>
1111
<script src="../testsuite.js"></script>
12+
<script src="../../../external/qunit-assert-classes/qunit-assert-classes.js"></script>
1213
<script>
1314
TestHelpers.loadResources({
1415
css: [ "core", "selectable" ],

tests/unit/selectable/selectable_methods.js

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ test("enable", function() {
7171
equal(actual, expected, "enable is chainable");
7272
});
7373

74-
test( "disable", function() {
74+
test( "disable", function( assert ) {
7575
expect( 6 );
7676
var chainable,
7777
fired = false,
@@ -98,9 +98,10 @@ test( "disable", function() {
9898
});
9999
equal( fired, false, "start fired" );
100100

101-
ok( !element.selectable( "widget" ).hasClass( "ui-state-disabled" ), "element does not get ui-state-disabled" );
101+
assert.lacksClasses( element.selectable( "widget" ), "ui-state-disabled" );
102+
102103
ok( !element.selectable( "widget" ).attr( "aria-disabled" ), "element does not get aria-disabled" );
103-
ok( element.selectable( "widget" ).hasClass( "ui-selectable-disabled" ), "element gets ui-selectable-disabled" );
104+
assert.hasClasses( element.selectable( "widget" ), "ui-selectable-disabled" );
104105

105106
element.selectable( "destroy" );
106107

0 commit comments

Comments
 (0)