Core: Connected to #1434: run all only tests

src/module.js

@@ -104,14 +104,12 @@ export default function module( name, options, executeNow ) {
 }
 
 module.only = function() {
-	if ( focused ) {
-		return;
+	if ( !focused ) {
+		config.modules.length = 0;
+		config.queue.length = 0;
 	}
 
-	config.modules.length = 0;
-	config.queue.length = 0;
-
-	module( ...arguments );
+	processModule( ...arguments );
 
 	focused = true;
 };

src/test.js

@@ -719,13 +719,11 @@ export function skip( testName ) {
 
 // Will be exposed as QUnit.only
 export function only( testName, callback ) {
-	if ( focused ) {
-		return;
+	if ( !focused ) {
+		config.queue.length = 0;
+		focused = true;
 	}
 
-	config.queue.length = 0;
-	focused = true;
-
 	const newTest = new Test( {
 		testName: testName,
 		callback: callback

test/module-only.js

@@ -35,6 +35,10 @@ QUnit.done( function() {
 			"75e1bf3f": {
 				skipped: false,
 				todo: false
+			},
+			"c7ae85c2": {
+				skipped: false,
+				todo: false
 			}
 		} );
 	} );
@@ -79,3 +83,9 @@ QUnit.module( "module C", function() {
 		assert.ok( false, "this test should not run" );
 	} );
 } );
+
+QUnit.module.only( "module D", function() {
+	QUnit.test( "test D", function( assert ) {
+		assert.ok( true, "this test should run as well" );
+	} );
+} );

test/only.js

@@ -1,17 +1,25 @@
-QUnit.module( "QUnit.only" );
+QUnit.module( "QUnit.only", function( hooks ) {
+	let testsRun = 0;
 
-QUnit.test( "implicitly skipped test", function( assert ) {
-	assert.ok( false, "test should be skipped" );
-} );
+	hooks.after( function( assert ) {
+		assert.strictEqual( testsRun, 2 );
+	} );
 
-QUnit.only( "only run this test", function( assert ) {
-	assert.ok( true, "only this test should run" );
-} );
+	QUnit.test( "implicitly skipped test", function( assert ) {
+		assert.ok( false, "test should be skipped" );
+	} );
 
-QUnit.test( "another implicitly skipped test", function( assert ) {
-	assert.ok( false, "test should be skipped" );
-} );
+	QUnit.only( "run this test", function( assert ) {
+		testsRun += 1;
+		assert.ok( true, "only this test should run" );
+	} );
+
+	QUnit.test( "another implicitly skipped test", function( assert ) {
+		assert.ok( false, "test should be skipped" );
+	} );
 
-QUnit.only( "ignore the subsequent calls to only", function( assert ) {
-	assert.ok( false, "this test should be skipped" );
+	QUnit.only( "all tests with only run", function( assert ) {
+		testsRun += 1;
+		assert.ok( true, "this test should run as well" );
+	} );
 } );