Skip to content
Permalink
Browse files

Revert "Attributes: remove flakey test for selected attribute"

This reverts commit 87bd130.
  • Loading branch information
markelog committed Nov 11, 2015
1 parent 112f612 commit 38261772a4054621b24135a5b2266eb2a12798b9
Showing with 3 additions and 1 deletion.
  1. +3 −1 test/unit/attributes.js
@@ -564,7 +564,7 @@ QUnit.test( "attr('tabindex', value)", function( assert ) {
} );

QUnit.test( "removeAttr(String)", function( assert ) {
assert.expect( 12 );
assert.expect( 13 );
var $first;

assert.equal( jQuery( "#mark" ).removeAttr( "class" ).attr( "class" ), undefined, "remove class" );
@@ -576,6 +576,8 @@ 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 3826177

Please sign in to comment.
You can’t perform that action at this time.