test_runner: suite timeout doesn't overwrite run() timeout

[ILikeRubberDuckies]

Version

24.7.0

Platform

Microsoft Windows NT 10.0.19045.0 x64

Subsystem

No response

What steps will reproduce the bug?

runner.js

import { run } from "node:test";
import { spec } from "node:test/reporters";

run({
    timeout: 1000
}).compose(spec)
    .pipe(process.stdout);

tee.test.js

import { test, suite } from "node:test";

suite("baa", { timeout: 2000 }, () => {
    test("tee", async () => {
        console.log("tee");
        await new Promise((res) => { setTimeout(res, 1500) });
    })
})

Run node runner.js
=> 'test timed out after 1000ms'

How often does it reproduce? Is there a required condition?

Always. No.

What is the expected behavior? Why is that the expected behavior?

Test should pass.

What do you see instead?

=> 'test timed out after 1000ms'

Additional information

Similar to #58802

[pmarchini]

Hey @ILikeRubberDuckies, thanks for raising this issue.

I need to check the implementation, but while reproducing the behaviour, I noticed that this is not related to the run function only, but also occurs when running via --test --test-timeout.

I will take a look as soon as possible!

cc @jakecastelli