Build: Refactor Node smoke tests

Utilize the assert module, avoid inline JSHint comments.

build/tasks/node_smoke_tests.js

@@ -15,7 +15,8 @@ module.exports = function( grunt ) {
 
 	fs.readdirSync( testsDir )
 		.filter( function( testFilePath ) {
-			return fs.statSync( testsDir + testFilePath ).isFile();
+			return fs.statSync( testsDir + testFilePath ).isFile() &&
+				/\.js$/.test( testFilePath );
 		} )
 		.forEach( function( testFilePath ) {
 			var taskName = "node_" + testFilePath.replace( /\.js$/, "" );

test/node_smoke_tests/.jshintrc

@@ -0,0 +1,14 @@
+{
+	"boss": true,
+	"curly": true,
+	"eqeqeq": true,
+	"eqnull": true,
+	"expr": true,
+	"immed": true,
+	"noarg": true,
+	"quotmark": "double",
+	"undef": true,
+	"unused": true,
+
+	"node": true
+}

test/node_smoke_tests/document_missing.js

@@ -1,19 +1,11 @@
-/* jshint node: true */
-
 "use strict";
 
-var ensureGlobalNotCreated = require( "./lib/ensure_global_not_created" ),
+var assert = require( "assert" ),
+	ensureGlobalNotCreated = require( "./lib/ensure_global_not_created" ),
 	jQueryFactory = require( "../../dist/jquery.js" );
 
-try {
+assert.throws( function () {
 	jQueryFactory( {} );
-	console.error( "The jQuery factory should reject window without a document" );
+}, /jQuery requires a window with a document/ );
-	process.exit( 1 );
+
-} catch ( e ) {
+ensureGlobalNotCreated( module.exports );
-	if ( e.message ===  "jQuery requires a window with a document" ) {
-		ensureGlobalNotCreated( module.exports );
-		process.exit( 0 );
-	}
-	console.error( "An unexpected error thrown; message: ", e.message );
-	process.exit( 1 );
-}

test/node_smoke_tests/document_passed.js

@@ -1,12 +1,9 @@
-/* jshint node: true */
-
 "use strict";
 
+var assert = require( "assert" );
+
 require( "jsdom" ).env( "", function( errors, window ) {
-	if ( errors ) {
+	assert.ifError( errors );
-		console.error( errors );
-		process.exit( 1 );
-	}
 
 	var ensureJQuery = require( "./lib/ensure_jquery" ),
 		ensureGlobalNotCreated = require( "./lib/ensure_global_not_created" ),

test/node_smoke_tests/document_present_originally.js

@@ -1,12 +1,9 @@
-/* jshint node: true */
-
 "use strict";
 
+var assert = require( "assert" );
+
 require( "jsdom" ).env( "", function( errors, window ) {
-	if ( errors ) {
+	assert.ifError( errors );
-		console.error( errors );
-		process.exit( 1 );
-	}
 
 	// Pretend the window is a global.
 	global.window = window;

test/node_smoke_tests/lib/ensure_global_not_created.js

@@ -1,7 +1,7 @@
-/* jshint node: true */
-
 "use strict";
 
+var assert = require( "assert" );
+
 // Ensure the jQuery property on global/window/module.exports/etc. was not
 // created in a CommonJS environment.
 // `global` is always checked in addition to passed parameters.
@@ -9,9 +9,7 @@ module.exports = function ensureGlobalNotCreated() {
 	var args = [].slice.call( arguments ).concat( global );
 
 	args.forEach( function( object ) {
-		if ( object.jQuery ) {
+		assert.strictEqual( object.jQuery, undefined,
-			console.error( "A jQuery global was created in a CommonJS environment." );
+			"A jQuery global was created in a CommonJS environment." );
-			process.exit( 1 );
-		}
 	} );
 };

test/node_smoke_tests/lib/ensure_jquery.js

@@ -1,11 +1,9 @@
-/* jshint node: true */
-
 "use strict";
 
+var assert = require( "assert" );
+
 // Check if the object we got is the jQuery object by invoking a basic API.
 module.exports = function ensureJQuery( jQuery ) {
-	if ( !/^jQuery/.test( jQuery.expando ) ) {
+	assert( /^jQuery/.test( jQuery.expando ),
-		console.error( "jQuery.expando was not detected, the jQuery bootstrap process has failed" );
+		"jQuery.expando was not detected, the jQuery bootstrap process has failed" );
-		process.exit( 1 );
-	}
 };