Permalink
Browse files

Attributes: fix failing test for new return value

  • Loading branch information...
timmywil committed Mar 16, 2015
1 parent aaeed53 commit 5dc4616ca0fdbf2a80890bd9b236e796b84db8c1
Showing with 12 additions and 3 deletions.
  1. +1 −1 src/attributes/attr.js
  2. +11 −2 test/unit/attributes.js
View
@@ -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
View
@@ -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;
});

0 comments on commit 5dc4616

Please sign in to comment.