Browse files

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)
  • Loading branch information...
timmywil authored and gibson042 committed Oct 26, 2015
1 parent 5c086c3 commit 689270e9d8333085ae73144d5a21c244d07c8cb3
Showing with 1 addition and 3 deletions.
  1. +1 −3 test/unit/attributes.js
@@ -563,7 +563,7 @@ QUnit.test( "attr('tabindex', value)", function( assert ) {
} );
QUnit.test( "removeAttr(String)", function( assert ) {
assert.expect( 13 );
assert.expect( 12 );
var $first;
assert.equal( jQuery( "#mark" ).removeAttr( "class" ).attr( "class" ), undefined, "remove class" );
@@ -575,8 +575,6 @@ QUnit.test( "removeAttr(String)", function( assert ) {
jQuery( "#check1" ).removeAttr( "checked" ).prop( "checked", true ).removeAttr( "checked" );
assert.equal( document.getElementById( "check1" ).checked, true, "removeAttr should not set checked to false, since the checked attribute does NOT mirror the checked property" );
jQuery( "#option1b" ).attr( "selected", "selected" ).removeAttr( "selected" ).attr( "selected", "selected" );
assert.notEqual( document.getElementById( "select1" ).selectedIndex, 1, "Once the selected attribute is dirty, subsequent settings should not select the option (gh-1759)" );
jQuery( "#text1" ).prop( "readOnly", true ).removeAttr( "readonly" );
assert.equal( document.getElementById( "text1" ).readOnly, false, "removeAttr sets boolean properties to false" );

0 comments on commit 689270e

Please sign in to comment.