Unit test for this case

Codestyle fixes

diff --git a/src/core.js b/src/core.js
index db832a0bf21091f93bf681f00df74104892e6971..eb24a69a4a735b1b313b469cb1ec860acdc09ddd 100644 (file)
--- a/src/core.js
+++ b/src/core.js
@@ -678,8 +678,7 @@ jQuery.extend({
        },
 
        inArray: function( elem, array ) {
-               if (!jQuery.isArray(array))
-               {
+               if ( !array ) {
                        return -1;
                }
 

diff --git a/test/unit/core.js b/test/unit/core.js
index 8c285f6dd89411ad539e8b3efde45b2392a09449..6e245adf7b43a519100d9a90e96ebc5f358c5712 100644 (file)
--- a/test/unit/core.js
+++ b/test/unit/core.js
@@ -916,6 +916,16 @@ test("jQuery.makeArray", function(){
        same( jQuery.makeArray({length: "5"}), [], "Make sure object is coerced properly.");
 });
 
+test("jQuery.inArray", function(){
+       expect(3);
+
+       equals( jQuery.inArray( 0, false ), -1 , "Search in 'false' as array returns -1 and doesn't throw exception" );
+
+       equals( jQuery.inArray( 0, null ), -1 , "Search in 'null' as array returns -1 and doesn't throw exception" );
+
+       equals( jQuery.inArray( 0, undefined ), -1 , "Search in 'undefined' as array returns -1 and doesn't throw exception" );
+});
+
 test("jQuery.isEmptyObject", function(){
        expect(2);