From ee2cdef3347dfc439708b6841e9c44d778e215bb Mon Sep 17 00:00:00 2001
From: cjihrig <cjihrig@gmail.com>
Date: Sun, 26 May 2024 12:36:46 -0400
Subject: [PATCH] test_runner: add context.fullName

This commit adds a fullName getter to the TestContext and
SuiteContext classes. This is similar to the existing name getter,
but also includes the name of all ancestor tests/suites.

PR-URL: https://github.com/nodejs/node/pull/53169
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Co-authored-by: Jacob Smith <jacob@frende.me>
---
 doc/api/test.md                            |  8 ++++++++
 lib/internal/test_runner/test.js           | 18 ++++++++++++++++
 test/parallel/test-runner-test-fullname.js | 24 ++++++++++++++++++++++
 3 files changed, 50 insertions(+)
 create mode 100644 test/parallel/test-runner-test-fullname.js

diff --git a/doc/api/test.md b/doc/api/test.md
index 7069dbc38ed9a8..5ce1d8f3d2286f 100644
--- a/doc/api/test.md
+++ b/doc/api/test.md
@@ -3044,6 +3044,14 @@ test('top level test', (t) => {
 });
 ```
 
+### `context.fullName`
+
+<!-- YAML
+added: REPLACEME
+-->
+
+The name of the test and each of its ancestors, separated by `>`.
+
 ### `context.name`
 
 <!-- YAML
diff --git a/lib/internal/test_runner/test.js b/lib/internal/test_runner/test.js
index effc73c0f2ca7d..c32ef4a7acd961 100644
--- a/lib/internal/test_runner/test.js
+++ b/lib/internal/test_runner/test.js
@@ -218,6 +218,10 @@ class TestContext {
     return this.#test.name;
   }
 
+  get fullName() {
+    return getFullName(this.#test);
+  }
+
   get error() {
     return this.#test.error;
   }
@@ -331,6 +335,10 @@ class SuiteContext {
   get name() {
     return this.#suite.name;
   }
+
+  get fullName() {
+    return getFullName(this.#suite);
+  }
 }
 
 class Test extends AsyncResource {
@@ -1216,6 +1224,16 @@ class Suite extends Test {
   }
 }
 
+function getFullName(test) {
+  let fullName = test.name;
+
+  for (let t = test.parent; t !== t.root; t = t.parent) {
+    fullName = `${t.name} > ${fullName}`;
+  }
+
+  return fullName;
+}
+
 module.exports = {
   kCancelledByParent,
   kSubtestsFailed,
diff --git a/test/parallel/test-runner-test-fullname.js b/test/parallel/test-runner-test-fullname.js
new file mode 100644
index 00000000000000..40e8abccd36b24
--- /dev/null
+++ b/test/parallel/test-runner-test-fullname.js
@@ -0,0 +1,24 @@
+'use strict';
+require('../common');
+const { strictEqual } = require('node:assert');
+const { suite, test } = require('node:test');
+
+suite('suite', (t) => {
+  strictEqual(t.fullName, 'suite');
+
+  test('test', (t) => {
+    strictEqual(t.fullName, 'suite > test');
+
+    t.test('subtest', (t) => {
+      strictEqual(t.fullName, 'suite > test > subtest');
+
+      t.test('subsubtest', (t) => {
+        strictEqual(t.fullName, 'suite > test > subtest > subsubtest');
+      });
+    });
+  });
+});
+
+test((t) => {
+  strictEqual(t.fullName, '<anonymous>');
+});