Skip to content

Commit 87bd130

Browse files
committed
Attributes: remove flakey test for selected attribute
- The change is already covered by the "checked" test. Since this test is redundant, just drop it.
1 parent f931786 commit 87bd130

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
@@ -564,7 +564,7 @@ QUnit.test( "attr('tabindex', value)", function( assert ) {
564564
} );
565565

566566
QUnit.test( "removeAttr(String)", function( assert ) {
567-
assert.expect( 13 );
567+
assert.expect( 12 );
568568
var $first;
569569

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

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

0 commit comments

Comments
 (0)