Permalink
Browse files

Build: fix tests in AMD mode

  • Loading branch information...
timmywil committed Dec 9, 2014
1 parent 58c2460 commit 6051609df35ef5e478c79c76534c03e4b46100bf
Showing with 32 additions and 32 deletions.
  1. +5 −8 build/tasks/build.js
  2. +2 −0 src/core/parseHTML.js
  3. +3 −1 src/core/support.js
  4. +0 −6 src/exports/global.js
  5. +2 −3 src/jquery.js
  6. +1 −0 src/outro.js
  7. +3 −1 test/data/testinit.js
  8. +16 −13 test/unit/core.js
View
@@ -26,7 +26,7 @@ module.exports = function( grunt ) {
skipSemiColonInsertion: true,
wrap: {
startFile: "src/intro.js",
endFile: "src/outro.js"
endFile: [ "src/exports/global.js", "src/outro.js" ]
},
paths: {
sizzle: "../external/sizzle/dist/sizzle"
@@ -61,13 +61,10 @@ module.exports = function( grunt ) {
} else {
// Ignore jQuery's exports (the only necessary one)
if ( name !== "jquery" ) {
contents = contents
.replace( /\s*return\s+[^\}]+(\}\);[^\w\}]*)$/, "$1" )
// Multiple exports
.replace( /\s*exports\.\w+\s*=\s*\w+;/g, "" );
}
contents = contents
.replace( /\s*return\s+[^\}]+(\}\);[^\w\}]*)$/, "$1" )
// Multiple exports
.replace( /\s*exports\.\w+\s*=\s*\w+;/g, "" );
// Remove define wrappers, closure ends, and empty declarations
contents = contents
View
@@ -1,6 +1,8 @@
define([
"../core",
"./var/rsingleTag",
"./support",
"../manipulation" // buildFragment
], function( jQuery, rsingleTag, support ) {
View
@@ -1,6 +1,8 @@
define([
"../var/support"
], function( jQuery, support ) {
], function( support ) {
// window.document is used here as it's before the sandboxed document
support.createHTMLDocument = !!window.document.implementation.createHTMLDocument;
return support;
});
View
@@ -1,7 +1,3 @@
define([
"../core"
], function( jQuery ) {
var
// Map over jQuery in case of overwrite
_jQuery = window.jQuery,
@@ -27,5 +23,3 @@ jQuery.noConflict = function( deep ) {
if ( !noGlobal ) {
window.jQuery = window.$ = jQuery;
}
});
View
@@ -28,10 +28,9 @@ define([
"./offset",
"./dimensions",
"./deprecated",
"./exports/amd",
"./exports/global"
"./exports/amd"
], function( jQuery ) {
return jQuery;
return (window.jQuery = window.$ = jQuery);
});
View
@@ -1 +1,2 @@
return jQuery;
}));
View
@@ -301,7 +301,9 @@ this.loadTests = function() {
/**
* Run in noConflict mode
*/
jQuery.noConflict();
if (jQuery.noConflict) {

This comment has been minimized.

Show comment
Hide comment
@gibson042

gibson042 Dec 9, 2014

Member

This is missing spaces.

@gibson042

gibson042 Dec 9, 2014

Member

This is missing spaces.

jQuery.noConflict();
}
// Load the TestSwarm listener if swarmURL is in the address.
if ( loadSwarm ) {
View
@@ -234,24 +234,27 @@ test( "globalEval execution after script injection (#7862)", 1, function() {
ok( window.strictEvalTest - now < 500, "Code executed synchronously" );
});
test("noConflict", function() {
expect(7);
// This is not run in AMD mode
if (jQuery.noConflict) {

This comment has been minimized.

Show comment
Hide comment
@gibson042

gibson042 Dec 9, 2014

Member

So is this.

@gibson042

gibson042 Dec 9, 2014

Member

So is this.

test("noConflict", function() {
expect(7);
var $$ = jQuery;
var $$ = jQuery;
strictEqual( jQuery, jQuery.noConflict(), "noConflict returned the jQuery object" );
strictEqual( window["jQuery"], $$, "Make sure jQuery wasn't touched." );
strictEqual( window["$"], original$, "Make sure $ was reverted." );
strictEqual( jQuery, jQuery.noConflict(), "noConflict returned the jQuery object" );
strictEqual( window["jQuery"], $$, "Make sure jQuery wasn't touched." );
strictEqual( window["$"], original$, "Make sure $ was reverted." );
jQuery = $ = $$;
jQuery = $ = $$;
strictEqual( jQuery.noConflict(true), $$, "noConflict returned the jQuery object" );
strictEqual( window["jQuery"], originaljQuery, "Make sure jQuery was reverted." );
strictEqual( window["$"], original$, "Make sure $ was reverted." );
ok( $$().pushStack([]), "Make sure that jQuery still works." );
strictEqual( jQuery.noConflict(true), $$, "noConflict returned the jQuery object" );
strictEqual( window["jQuery"], originaljQuery, "Make sure jQuery was reverted." );
strictEqual( window["$"], original$, "Make sure $ was reverted." );
ok( $$().pushStack([]), "Make sure that jQuery still works." );
window["jQuery"] = jQuery = $$;
});
window["jQuery"] = jQuery = $$;
});
}
test("trim", function() {
expect(13);

0 comments on commit 6051609

Please sign in to comment.