Core: Skip tests with no/falsy callbacks

Gratuitous demonstration
qunitjs · Dec 9, 2014 · c018044 · c018044
1 parent 609ed37
commit c018044
Showing 1 changed file with 3 additions and 8 deletions.
diff --git a/test/logs.js b/test/logs.js
@@ -12,6 +12,7 @@ var totalTests,
 	testStart = 0,
 	testDone = 0,
 	log = 0,
+	isBrowser = typeof window !== "undefined" && window.document,
 	module1Context = {
 		name: "logs1",
 		tests: [
@@ -337,14 +338,8 @@ QUnit.done(function() {
 
 QUnit.module( "deprecated log methods" );
 
-QUnit.test( "QUnit.reset()", function( assert ) {
-
-	// Skip non-browsers
-	if ( typeof window === "undefined" || !window.document ) {
-		assert.expect( 0 );
-		return;
-	}
-
+// Browser-only QUnit.reset test
+QUnit.test( "QUnit.reset()", isBrowser && function( assert ) {
 	var myFixture = document.getElementById( "qunit-fixture" );
 
 	myFixture.innerHTML = "<em>something different from QUnit.config.fixture</em>";