Skip to content
Browse files

Fix attribute names in aliased form property test. Close gh-951.

Test expects input elements having name='id', name='name', and name='target'. Additionally, these should have id='id', id='name', and id='target' respectively. No element was provided with id='id' or name='id', but rather one element had two name attributes (illegal) with the values 'id' and 'name' respectively.
  • Loading branch information...
1 parent 408e5e0 commit 144b8bfeadc5316656105e1a10a31ba5348ef5d6 @jonathansampson jonathansampson committed with dmethvin Oct 11, 2012
Showing with 1 addition and 1 deletion.
  1. +1 −1 test/unit/attributes.js
View
2 test/unit/attributes.js
@@ -73,7 +73,7 @@ test( "attr(String)", function() {
equal( jQuery("#foo").attr("height"), undefined, "Non existent height attribute should return undefined" );
// [7472] & [3113] (form contains an input with name="action" or name="id")
- var extras = jQuery("<input name='id' name='name' /><input id='target' name='target' />").appendTo("#testForm");
+ var extras = jQuery("<input id='id' name='id' /><input id='name' name='name' /><input id='target' name='target' />").appendTo("#testForm");
equal( jQuery("#form").attr("action","newformaction").attr("action"), "newformaction", "Check that action attribute was changed" );
equal( jQuery("#testForm").attr("target"), undefined, "Retrieving target does not equal the input with name=target" );
equal( jQuery("#testForm").attr("target", "newTarget").attr("target"), "newTarget", "Set target successfully on a form" );

0 comments on commit 144b8bf

Please sign in to comment.
Something went wrong with that request. Please try again.