Skip to content

Commit 53d1bc2

Browse files
committed
Effects: Use new has/lacksClasses assertions for all class checks
1 parent 15a9ed7 commit 53d1bc2

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

tests/unit/effects/effects.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
js: [

tests/unit/effects/effects_core.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -182,13 +182,14 @@ asyncTest( "animateClass clears style properties when stopped", function() {
182182
start();
183183
});
184184

185-
asyncTest( "animateClass: css and class changes during animation are not lost (#7106)", function() {
185+
asyncTest( "animateClass: css and class changes during animation are not lost (#7106)",
186+
function( assert ) {
186187
expect( 2 );
187188
var test = $( "div.ticket7106" );
188189

189190
// ensure the class stays and that the css property stays
190191
function animationComplete() {
191-
ok( test.hasClass( "testClass" ), "class change during animateClass was not lost" );
192+
assert.hasClasses( test, "testClass", "class change during animateClass was not lost" );
192193
equal( test.height(), 100, "css change during animateClass was not lost" );
193194
start();
194195
}

0 commit comments

Comments
 (0)