Skip to content
Permalink
Browse files
Tests: Fix code style issues
  • Loading branch information
markelog committed Jan 16, 2014
1 parent 659ac9c commit a2250b0
Showing 1 changed file with 4 additions and 4 deletions.
@@ -941,8 +941,8 @@ var testVal = function( valueObj ) {
equal( document.getElementById("text1").value, "", "Check for modified (via val(null)) value of input element" );

var j,
select = jQuery( "<select multiple><option value='1'/><option value='2'/></select>" );
$select1 = jQuery("#select1"),
$select = jQuery( "<select multiple><option value='1'/><option value='2'/></select>" ),
$select1 = jQuery("#select1");

This comment has been minimized.

Copy link
@mgol

mgol Jan 16, 2014

Member

As for code style... There should be spaces after ( and before ).

This comment has been minimized.

Copy link
@mgol

mgol Jan 16, 2014

Member

But probably not worth correcting it now, until we have jscs for tests.

This comment has been minimized.

Copy link
@markelog

markelog Jan 16, 2014

Author Member

This only fixes issues created by the previous commit, specifically the jshint ones.


$select1.val( valueObj("3") );
equal( $select1.val(), "3", "Check for modified (via val(String)) value of select element" );
@@ -960,8 +960,8 @@ var testVal = function( valueObj ) {
equal( j.val(), "asdf", "Check node,textnode,comment with val()" );
j.removeAttr("value");

select.val( valueObj( [ "1", "2" ] ) );
deepEqual( select.val(), [ "1", "2" ], "Should set array of values" );
$select.val( valueObj( [ "1", "2" ] ) );
deepEqual( $select.val(), [ "1", "2" ], "Should set array of values" );
};

test( "val(String/Number)", function() {

0 comments on commit a2250b0

Please sign in to comment.