test_runner: fix isSkipped check in junit reporter

mete0rfish · mete0rfish · commit 5816778d970d · 2025-08-01T15:20:54.000+09:00
The `isSkipped` function in the JUnit reporter was incorrectly
checking for `node?.attrs.failures` instead of `node?.attrs.skipped`.
diff --git a/lib/internal/test_runner/reporter/junit.js b/lib/internal/test_runner/reporter/junit.js
@@ -58,7 +58,7 @@ function isFailure(node) {
 }
 
 function isSkipped(node) {
-  return (node?.children && ArrayPrototypeSome(node.children, (c) => c.tag === 'skipped')) || node?.attrs?.failures;
+  return (node?.children && ArrayPrototypeSome(node.children, (c) => c.tag === 'skipped')) || node?.attrs?.skipped;
 }
 
 module.exports = async function* junitReporter(source) {

Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,7 @@ function isFailure(node) {`
`58`	`58`	`}`
`59`	`59`
`60`	`60`	`function isSkipped(node) {`
`61`		`- return (node?.children && ArrayPrototypeSome(node.children, (c) => c.tag === 'skipped')) \|\| node?.attrs?.failures;`
	`61`	`+ return (node?.children && ArrayPrototypeSome(node.children, (c) => c.tag === 'skipped')) \|\| node?.attrs?.skipped;`
`62`	`62`	`}`
`63`	`63`
`64`	`64`	`module.exports = async function* junitReporter(source) {`