diff options
author | Timmy Willison <timmywillisn@gmail.com> | 2015-03-16 11:52:05 -0400 |
---|---|---|
committer | Timmy Willison <timmywillisn@gmail.com> | 2015-03-16 11:52:05 -0400 |
commit | 5dc4616ca0fdbf2a80890bd9b236e796b84db8c1 (patch) | |
tree | 0f49e28bf583579b7368d8334b801eb8324b5e23 | |
parent | aaeed53e9f1e299975cb6f697c8038ec3a83fa4d (diff) | |
download | jquery-5dc4616ca0fdbf2a80890bd9b236e796b84db8c1.tar.gz jquery-5dc4616ca0fdbf2a80890bd9b236e796b84db8c1.zip |
Attributes: fix failing test for new return value
-rw-r--r-- | src/attributes/attr.js | 2 | ||||
-rw-r--r-- | test/unit/attributes.js | 13 |
2 files changed, 12 insertions, 3 deletions
diff --git a/src/attributes/attr.js b/src/attributes/attr.js index 3bb05a5a3..201a2d2a6 100644 --- a/src/attributes/attr.js +++ b/src/attributes/attr.js @@ -28,7 +28,7 @@ jQuery.extend({ // don't get/set attributes on text, comment and attribute nodes if ( !elem || nType === 3 || nType === 8 || nType === 2 ) { - return; + return null; } // Fallback to prop when attributes are not supported diff --git a/test/unit/attributes.js b/test/unit/attributes.js index c4bd17c17..1163ff4e1 100644 --- a/test/unit/attributes.js +++ b/test/unit/attributes.js @@ -377,8 +377,17 @@ test( "attr(String, Object)", function() { jQuery.each( [ window, document, obj, "#firstp" ], function( i, elem ) { var oldVal = elem.nonexisting, $elem = jQuery( elem ); - strictEqual( $elem.attr("nonexisting"), null, "attr works correctly for non existing attributes (bug #7500)." ); - equal( $elem.attr( "nonexisting", "foo" ).attr("nonexisting"), "foo", "attr falls back to prop on unsupported arguments" ); + // Falls back to prop, which returns undefined + strictEqual( + $elem.attr( "nonexisting" ), + typeof $elem[0].getAttribute === "undefined" ? undefined : null, + "attr works correctly for non existing attributes (bug #7500)." + ); + equal( + $elem.attr( "nonexisting", "foo" ).attr( "nonexisting" ), + "foo", + "attr falls back to prop on unsupported arguments" + ); elem.nonexisting = oldVal; }); |