Build: Don't assume the browser environment; smoke test on Node w/ jsdom

Fixes jquerygh-1788
mgol · Dec 18, 2014 · 8acd10f · 8acd10f
1 parent d6c97ab
commit 8acd10f
Show file tree

Hide file tree

Showing 22 changed files with 68 additions and 26 deletions.
diff --git a/Gruntfile.js b/Gruntfile.js
@@ -157,9 +157,22 @@ module.exports = function( grunt ) {
 
 	grunt.registerTask( "lint", [ "jshint", "jscs" ] );
 
+	grunt.registerTask( "node-smoke-test", function() {
+	    var done = this.async();
+		require( "jsdom" ).env( "", function( errors, window ) {
+			if ( errors ) {
+				console.error( errors );
+				done( false );
+			}
+			require( "./" )( window );
+			done();
+		});
+	});
+
 	// Short list as a high frequency watch task
 	grunt.registerTask( "dev", [ "build:*:*", "lint" ] );
 
 	// Default grunt
-	grunt.registerTask( "default", [ "jsonlint", "dev", "uglify", "dist:*", "compare_size" ] );
+	grunt.registerTask( "default",
+		[ "jsonlint", "dev", "uglify", "dist:*", "compare_size", "node-smoke-test" ] );
 };
diff --git a/package.json b/package.json
@@ -43,6 +43,7 @@
     "grunt-jsonlint": "1.0.4",
     "grunt-npmcopy": "0.1.0",
     "gzip-js": "0.3.2",
+    "jsdom": "1.5.0",
     "load-grunt-tasks": "1.0.0",
     "npm": "2.1.12",
     "qunitjs": "1.16.0",

diff --git a/src/.jshintrc b/src/.jshintrc
@@ -15,9 +15,13 @@
 
 	"sub": true,
 
-	"browser": true,
-
 	"globals": {
+		"window": true,
+		"setTimeout": true,
+		"clearTimeout": true,
+		"setInterval": true,
+		"clearInterval": true,
+
 		"jQuery": true,
 		"define": true,
 		"module": true,

diff --git a/src/ajax.js b/src/ajax.js
@@ -1,13 +1,15 @@
 define([
 	"./core",
+	"./var/document",
 	"./var/rnotwhite",
+	"./ajax/var/location",
 	"./ajax/var/nonce",
 	"./ajax/var/rquery",
 	"./core/init",
 	"./ajax/parseJSON",
 	"./ajax/parseXML",
 	"./deferred"
-], function( jQuery, rnotwhite, nonce, rquery ) {
+], function( jQuery, document, rnotwhite, location, nonce, rquery ) {
 
 var
 	rhash = /#.*$/,

diff --git a/src/ajax/parseXML.js b/src/ajax/parseXML.js
@@ -11,7 +11,7 @@ jQuery.parseXML = function( data ) {
 
 	// Support: IE9
 	try {
-		tmp = new DOMParser();
+		tmp = new window.DOMParser();
 		xml = tmp.parseFromString( data, "text/xml" );
 	} catch ( e ) {
 		xml = undefined;

diff --git a/src/ajax/script.js b/src/ajax/script.js
@@ -1,7 +1,8 @@
 define([
 	"../core",
+	"../var/document",
 	"../ajax"
-], function( jQuery ) {
+], function( jQuery, document ) {
 
 // Install script dataType
 jQuery.ajaxSetup({

diff --git a/src/ajax/var/location.js b/src/ajax/var/location.js
@@ -0,0 +1,3 @@
+define(function() {
+	return window.location;
+});
diff --git a/src/ajax/xhr.js b/src/ajax/xhr.js
@@ -6,7 +6,7 @@ define([
 
 jQuery.ajaxSettings.xhr = function() {
 	try {
-		return new XMLHttpRequest();
+		return new window.XMLHttpRequest();
 	} catch ( e ) {}
 };
 

diff --git a/src/attributes/support.js b/src/attributes/support.js
@@ -1,6 +1,7 @@
 define([
+	"../var/document",
 	"../var/support"
-], function( support ) {
+], function( document, support ) {
 
 (function() {
 	var input = document.createElement( "input" ),

diff --git a/src/core.js b/src/core.js
@@ -1,5 +1,6 @@
 define([
 	"./var/arr",
+	"./var/document",
 	"./var/slice",
 	"./var/concat",
 	"./var/push",
@@ -8,12 +9,9 @@ define([
 	"./var/toString",
 	"./var/hasOwn",
 	"./var/support"
-], function( arr, slice, concat, push, indexOf, class2type, toString, hasOwn, support ) {
+], function( arr, document, slice, concat, push, indexOf, class2type, toString, hasOwn, support ) {
 
 var
-	// Use the correct document accordingly with window argument (sandbox)
-	document = window.document,
-
 	version = "@VERSION",
 
 	// Define a local copy of jQuery

diff --git a/src/core/init.js b/src/core/init.js
@@ -1,9 +1,10 @@
 // Initialize a jQuery object
 define([
 	"../core",
+	"../var/document",
 	"./var/rsingleTag",
 	"../traversing/findFilter"
-], function( jQuery, rsingleTag ) {
+], function( jQuery, document, rsingleTag ) {
 
 // A central reference to the root jQuery(document)
 var rootjQuery,

diff --git a/src/core/parseHTML.js b/src/core/parseHTML.js
@@ -1,13 +1,14 @@
 define([
 	"../core",
+	"../var/document",
 	"./var/rsingleTag",
 
 	// This is the only module that needs core/support
 	"./support",
 
 	// buildFragment
 	"../manipulation"
-], function( jQuery, rsingleTag, support ) {
+], function( jQuery, document, rsingleTag, support ) {
 
 // data: string of html
 // context (optional): If specified, the fragment will be created in this context,

diff --git a/src/core/ready.js b/src/core/ready.js
@@ -1,8 +1,9 @@
 define([
 	"../core",
+	"../var/document",
 	"../core/init",
 	"../deferred"
-], function( jQuery ) {
+], function( jQuery, document ) {
 
 // The deferred used on DOM ready
 var readyList;

diff --git a/src/core/support.js b/src/core/support.js
@@ -1,9 +1,15 @@
 define([
+	"../var/document",
 	"../var/support"
-], function( support ) {
+], function( document, support ) {
 
 support.createHTMLDocument = (function() {
 	var doc = document.implementation.createHTMLDocument( "" );
+	// Support: Node with jsdom<=1.5.0+
+	// jsdom's document created via the above method doesn't contain the body
+	if ( !doc.body ) {
+		return false;
+	}
 	doc.body.innerHTML = "<form></form><form></form>";
 	return doc.body.childNodes.length === 2;
 })();

diff --git a/src/css/defaultDisplay.js b/src/css/defaultDisplay.js
@@ -1,7 +1,8 @@
 define([
 	"../core",
+	"../var/document",
 	"../manipulation" // appendTo
-], function( jQuery ) {
+], function( jQuery, document ) {
 
 var iframe,
 	elemdisplay = {

diff --git a/src/css/support.js b/src/css/support.js
@@ -1,7 +1,8 @@
 define([
 	"../core",
+	"../var/document",
 	"../var/support"
-], function( jQuery, support ) {
+], function( jQuery, document, support ) {
 
 (function() {
 	var pixelPositionVal, boxSizingReliableVal,

diff --git a/src/effects.js b/src/effects.js
@@ -1,5 +1,6 @@
 define([
 	"./core",
+	"./var/document",
 	"./var/pnum",
 	"./css/var/cssExpand",
 	"./css/var/isHidden",
@@ -12,7 +13,7 @@ define([
 	"./css",
 	"./deferred",
 	"./traversing"
-], function( jQuery, pnum, cssExpand, isHidden, defaultDisplay, dataPriv ) {
+], function( jQuery, document, pnum, cssExpand, isHidden, defaultDisplay, dataPriv ) {
 
 var
 	fxNow, timerId,

diff --git a/src/event.js b/src/event.js
@@ -1,5 +1,6 @@
 define([
 	"./core",
+	"./var/document",
 	"./var/rnotwhite",
 	"./var/hasOwn",
 	"./var/slice",
@@ -9,7 +10,7 @@ define([
 	"./core/init",
 	"./data/accepts",
 	"./selector"
-], function( jQuery, rnotwhite, hasOwn, slice, support, dataPriv ) {
+], function( jQuery, document, rnotwhite, hasOwn, slice, support, dataPriv ) {
 
 var
 	rkeyEvent = /^key/,

diff --git a/src/manipulation/support.js b/src/manipulation/support.js
@@ -1,6 +1,7 @@
 define([
+	"../var/document",
 	"../var/support"
-], function( support ) {
+], function( document, support ) {
 
 (function() {
 	var fragment = document.createDocumentFragment(),

diff --git a/src/offset.js b/src/offset.js
@@ -1,6 +1,7 @@
 define([
 	"./core",
 	"./core/access",
+	"./var/document",
 	"./css/var/rnumnonpx",
 	"./css/curCSS",
 	"./css/addGetHookIf",
@@ -9,9 +10,9 @@ define([
 	"./core/init",
 	"./css",
 	"./selector" // contains
-], function( jQuery, access, rnumnonpx, curCSS, addGetHookIf, support ) {
+], function( jQuery, access, document, rnumnonpx, curCSS, addGetHookIf, support ) {
 
-var docElem = window.document.documentElement;
+var docElem = document.documentElement;
 
 /**
  * Gets a window from an element

diff --git a/src/selector-native.js b/src/selector-native.js
@@ -1,6 +1,7 @@
 define([
-	"./core"
-], function( jQuery ) {
+	"./core",
+	"./var/document"
+], function( jQuery, document ) {
 
 /*
  * Optional (non-Sizzle) selector module for custom builds.
@@ -28,7 +29,7 @@ define([
  */
 
 var hasDuplicate,
-	docElem = window.document.documentElement,
+	docElem = document.documentElement,
 	matches = docElem.matches ||
 		docElem.webkitMatchesSelector ||
 		docElem.mozMatchesSelector ||

diff --git a/src/var/document.js b/src/var/document.js
@@ -0,0 +1,3 @@
+define(function() {
+	return window.document;
+});