test_runner: add skip, todo, only, and expectFailure to subtest context

lib/internal/test_runner/test.js

@@ -1,6 +1,7 @@
 'use strict';
 const {
   ArrayPrototypeEvery,
+  ArrayPrototypeForEach,
   ArrayPrototypePush,
   ArrayPrototypePushApply,
   ArrayPrototypeShift,
@@ -259,6 +260,39 @@ class TestContext {
 
   constructor(test) {
     this.#test = test;
+
+    const runSubtest = (name, options, fn, loc, extraOverrides) => {
+      const overrides = {
+        __proto__: null,
+        ...extraOverrides,
+        loc,
+      };
+
+      const { plan } = this.#test;
+      if (plan !== null) {
+        plan.count();
+      }
+
+      const subtest = this.#test.createSubtest(
+        // eslint-disable-next-line no-use-before-define
+        Test, name, options, fn, overrides,
+      );
+
+      return subtest.start();
+    };
+
+    this.test = (name, options, fn) =>
+      runSubtest(name, options, fn, getCallerLocation());
+    ArrayPrototypeForEach(
+      ['expectFailure', 'skip', 'todo', 'only'],
+      (keyword) => {
+        this.test[keyword] = (name, options, fn) =>
+          runSubtest(name, options, fn, getCallerLocation(), {
+            __proto__: null,
+            [keyword]: true,
+          });
+      },
+    );
   }
 
   get signal() {
@@ -358,25 +392,6 @@ class TestContext {
     this.#test.todo(message);
   }
 
-  test(name, options, fn) {
-    const overrides = {
-      __proto__: null,
-      loc: getCallerLocation(),
-    };
-
-    const { plan } = this.#test;
-    if (plan !== null) {
-      plan.count();
-    }
-
-    const subtest = this.#test.createSubtest(
-      // eslint-disable-next-line no-use-before-define
-      Test, name, options, fn, overrides,
-    );
-
-    return subtest.start();
-  }
-
   before(fn, options) {
     this.#test.createHook('before', fn, {
       __proto__: null,

test/parallel/test-runner-subtest-skip-todo-only.js

@@ -0,0 +1,48 @@
+'use strict';
+const common = require('../common');
+const assert = require('node:assert');
+const { test } = require('node:test');
+
+// Verify that all subtest variants exist on TestContext.
+test('subtest variants exist on TestContext', common.mustCall(async (t) => {
+  assert.strictEqual(typeof t.test, 'function');
+  assert.strictEqual(typeof t.test.skip, 'function');
+  assert.strictEqual(typeof t.test.todo, 'function');
+  assert.strictEqual(typeof t.test.only, 'function');
+  assert.strictEqual(typeof t.test.expectFailure, 'function');
+}));
+
+// t.test.skip: callback must NOT be called.
+test('t.test.skip prevents callback execution', common.mustCall(async (t) => {
+  await t.test.skip('skipped subtest', common.mustNotCall());
+}));
+
+// t.test.todo without callback: subtest is marked as todo and skipped.
+test('t.test.todo without callback', common.mustCall(async (t) => {
+  await t.test.todo('todo subtest without callback');
+}));
+
+// t.test.todo with callback: callback runs but subtest is marked as todo.
+test('t.test.todo with callback runs the callback', common.mustCall(async (t) => {
+  await t.test.todo('todo subtest with callback', common.mustCall());
+}));
+
+// Plan counting works with subtest variants.
+test('plan counts subtest variants', common.mustCall(async (t) => {
+  t.plan(3);
+  await t.test('normal subtest', common.mustCall());
+  await t.test.skip('skipped subtest');
+  await t.test.todo('todo subtest');
+}));
+
+// Nested subtests also expose the variants.
+test('nested subtests have variants', common.mustCall(async (t) => {
+  await t.test('level 1', common.mustCall(async (t2) => {
+    assert.strictEqual(typeof t2.test.skip, 'function');
+    assert.strictEqual(typeof t2.test.todo, 'function');
+    assert.strictEqual(typeof t2.test.only, 'function');
+    assert.strictEqual(typeof t2.test.expectFailure, 'function');
+
+    await t2.test.skip('nested skipped', common.mustNotCall());
+  }));
+}));