Skip to content
Permalink
Browse files

Add test for jQuery(jQueryObj) cloning and simplify new get() code

  • Loading branch information
wycats committed Jul 17, 2009
1 parent 88bd74c commit 1a7f72e7fe96654838074f88c79e021d37c5c30a
Showing with 6 additions and 4 deletions.
  1. +2 −3 src/core.js
  2. +4 −1 test/unit/core.js
@@ -43,9 +43,8 @@ jQuery.fn = jQuery.prototype = {

// Handle $(DOMElement)
if ( selector.nodeType ) {
this[0] = selector;
this.context = this[0] = selector;
this.length++;
this.context = selector;
return this;
}

@@ -132,7 +131,7 @@ jQuery.fn = jQuery.prototype = {
this.toArray() :

// Return just the object
( num < 0 ? this.toArray.call(this, num)[0] : this[ num ] );
( num < 0 ? this.toArray(num)[ 0 ] : this[ num ] );
},

// Take an array of elements and push it onto the stack
@@ -12,7 +12,7 @@ test("Basic requirements", function() {
});

test("jQuery()", function() {
expect(11);
expect(12);

// Basic constructor's behavior

@@ -21,6 +21,9 @@ test("jQuery()", function() {
equals( jQuery(null).length, 0, "jQuery(null) === jQuery([])" );
equals( jQuery("").length, 0, "jQuery('') === jQuery([])" );

var obj = jQuery("div")
equals( jQuery(obj).selector, "div", "jQuery(jQueryObj) == jQueryObj" );

// can actually yield more than one, when iframes are included, the window is an array as well
equals( 1, jQuery(window).length, "Correct number of elements generated for jQuery(window)" );

0 comments on commit 1a7f72e

Please sign in to comment.
You can’t perform that action at this time.