diff --git a/configure.py b/configure.py index 69841d624f37b0..b6810a2f507c69 100755 --- a/configure.py +++ b/configure.py @@ -1113,7 +1113,7 @@ def try_check_compiler(cc, lang): proc = subprocess.Popen(shlex.split(cc) + ['-E', '-P', '-x', lang, '-'], stdin=subprocess.PIPE, stdout=subprocess.PIPE) except OSError: - return (False, False, '', '') + return (False, False, '', '', False) with proc: proc.stdin.write(b'__clang__ __GNUC__ __GNUC_MINOR__ __GNUC_PATCHLEVEL__ ' diff --git a/lib/internal/test_runner/reporter/junit.js b/lib/internal/test_runner/reporter/junit.js index 0b2efe119fedee..24bbd9ffc3e1cc 100644 --- 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) { diff --git a/src/node_constants.cc b/src/node_constants.cc index aebda8fb44b919..fd28e0904d05e2 100644 --- a/src/node_constants.cc +++ b/src/node_constants.cc @@ -1105,10 +1105,6 @@ NODE_DEFINE_CONSTANT(target, UV_FS_O_FILEMAP); NODE_DEFINE_CONSTANT(target, O_DIRECTORY); #endif -#ifdef O_EXCL - NODE_DEFINE_CONSTANT(target, O_EXCL); -#endif - #ifdef O_NOATIME NODE_DEFINE_CONSTANT(target, O_NOATIME); #endif diff --git a/test/parallel/test-abortsignal-cloneable.js b/test/parallel/test-abortsignal-cloneable.js index ffffcc5f43fa0d..cfd194909db38e 100644 --- a/test/parallel/test-abortsignal-cloneable.js +++ b/test/parallel/test-abortsignal-cloneable.js @@ -35,7 +35,7 @@ test('Can create a transferable abort controller', async () => { mc.port2.postMessage(ac.signal, [ac.signal]); - // Can be cloned/transferd multiple times and they all still work + // Can be cloned/transferred multiple times and they all still work mc.port2.postMessage(ac.signal, [ac.signal]); // Although we're using transfer semantics, the local AbortSignal diff --git a/test/parallel/test-abortsignal-drop-settled-signals.mjs b/test/parallel/test-abortsignal-drop-settled-signals.mjs index b300b0e223fc93..bb7448eacb1c34 100644 --- a/test/parallel/test-abortsignal-drop-settled-signals.mjs +++ b/test/parallel/test-abortsignal-drop-settled-signals.mjs @@ -104,17 +104,17 @@ const limit = 10_000; describe('when there is a long-lived signal', () => { it('drops settled dependant signals', (t, done) => { - makeSubsequentCalls(limit, (signal, depandantSignalsKey) => { + makeSubsequentCalls(limit, (signal, dependantSignalsKey) => { setImmediate(() => { - t.assert.strictEqual(signal[depandantSignalsKey].size, 0); + t.assert.strictEqual(signal[dependantSignalsKey].size, 0); done(); }); }); }); it('keeps all active dependant signals', (t, done) => { - makeSubsequentCalls(limit, (signal, depandantSignalsKey) => { - t.assert.strictEqual(signal[depandantSignalsKey].size, limit); + makeSubsequentCalls(limit, (signal, dependantSignalsKey) => { + t.assert.strictEqual(signal[dependantSignalsKey].size, limit); done(); }, true); diff --git a/test/parallel/test-runner-reporters.js b/test/parallel/test-runner-reporters.js index b557cef1b9bef8..50a47578a1da7e 100644 --- a/test/parallel/test-runner-reporters.js +++ b/test/parallel/test-runner-reporters.js @@ -191,4 +191,17 @@ describe('node:test reporters', { concurrency: true }, () => { assert.match(fileConent, /ℹ skipped 0/); assert.match(fileConent, /ℹ todo 0/); }); + + it('should correctly report pass/fail for junit reporter using reporters.js', async () => { + const file = tmpdir.resolve(`${tmpFiles++}.xml`); + const child = spawnSync(process.execPath, + ['--test', '--test-reporter', 'junit', '--test-reporter-destination', file, testFile]); + assert.strictEqual(child.stderr.toString(), ''); + assert.strictEqual(child.stdout.toString(), ''); + const fileContents = fs.readFileSync(file, 'utf8'); + assert.match(fileContents, //); + assert.match(fileContents, /\s*/); + assert.match(fileContents, //); + assert.match(fileContents, //); + }); });