test_runner: do not error when getting fullName of root context

Renegade334 · targos · commit e964c4334e2b · 2025-08-24T09:09:12.000+02:00
PR-URL: #59377 Reviewed-By: Chemi Atlow <chemi@atlow.co.il> Reviewed-By: Pietro Marchini <pietro.marchini94@gmail.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
diff --git a/lib/internal/test_runner/test.js b/lib/internal/test_runner/test.js
@@ -1533,6 +1533,8 @@ class Suite extends Test {
 }
 
 function getFullName(test) {
+  if (test === test.root) return test.name;
+
   let fullName = test.name;
 
   for (let t = test.parent; t !== t.root; t = t.parent) {
diff --git a/test/parallel/test-runner-test-fullname.js b/test/parallel/test-runner-test-fullname.js
@@ -1,7 +1,11 @@
 'use strict';
 require('../common');
 const { strictEqual } = require('node:assert');
-const { suite, test } = require('node:test');
+const { before, suite, test } = require('node:test');
+
+before((t) => {
+  strictEqual(t.fullName, '<root>');
+});
 
 suite('suite', (t) => {
   strictEqual(t.fullName, 'suite');

Original file line number	Diff line number	Diff line change
`@@ -1533,6 +1533,8 @@ class Suite extends Test {`
`1533`	`1533`	`}`
`1534`	`1534`
`1535`	`1535`	`function getFullName(test) {`
	`1536`	`+ if (test === test.root) return test.name;`
	`1537`	`+`
`1536`	`1538`	`let fullName = test.name;`
`1537`	`1539`
`1538`	`1540`	`for (let t = test.parent; t !== t.root; t = t.parent) {`