Skip to content

Commit 689270e

Browse files
timmywilgibson042
authored andcommitted
Attributes: remove flakey test for selected attribute
- The change is already covered by the "checked" test. Since this test is redundant, just drop it. (cherry picked from commit 87bd130)
1 parent 5c086c3 commit 689270e

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

test/unit/attributes.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -563,7 +563,7 @@ QUnit.test( "attr('tabindex', value)", function( assert ) {
563563
} );
564564

565565
QUnit.test( "removeAttr(String)", function( assert ) {
566-
assert.expect( 13 );
566+
assert.expect( 12 );
567567
var $first;
568568

569569
assert.equal( jQuery( "#mark" ).removeAttr( "class" ).attr( "class" ), undefined, "remove class" );
@@ -575,8 +575,6 @@ QUnit.test( "removeAttr(String)", function( assert ) {
575575

576576
jQuery( "#check1" ).removeAttr( "checked" ).prop( "checked", true ).removeAttr( "checked" );
577577
assert.equal( document.getElementById( "check1" ).checked, true, "removeAttr should not set checked to false, since the checked attribute does NOT mirror the checked property" );
578-
jQuery( "#option1b" ).attr( "selected", "selected" ).removeAttr( "selected" ).attr( "selected", "selected" );
579-
assert.notEqual( document.getElementById( "select1" ).selectedIndex, 1, "Once the selected attribute is dirty, subsequent settings should not select the option (gh-1759)" );
580578
jQuery( "#text1" ).prop( "readOnly", true ).removeAttr( "readonly" );
581579
assert.equal( document.getElementById( "text1" ).readOnly, false, "removeAttr sets boolean properties to false" );
582580

0 commit comments

Comments
 (0)