]> source.dussan.org Git - jquery.git/commitdiff
Attributes: remove flakey test for selected attribute
authorTimmy Willison <timmywillisn@gmail.com>
Mon, 26 Oct 2015 15:38:04 +0000 (11:38 -0400)
committerRichard Gibson <richard.gibson@gmail.com>
Mon, 26 Oct 2015 20:08:59 +0000 (16:08 -0400)
- The change is already covered by the "checked" test.
  Since this test is redundant, just drop it.

(cherry picked from commit 87bd130289c6ed9bfc355c1f8587ae6ce00a4776)

test/unit/attributes.js

index 79b191663b064e60a017e5d2dcef5bf8b97d65e8..d5826566f934c35db5a7470015c8fac3626f35c1 100644 (file)
@@ -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" );