Fix OverrideResult skip reason not being displayed

[Copilot]

OverrideResult(TestState.Skipped, "reason") ignores the provided reason and displays "Skipped" instead.

[Test]
public async Task TestSkip()
{
    TestContext.Current!.Execution.OverrideResult(TestState.Skipped, "test-skip foo bar baz.");
}
// Output: "Skipped" ❌
// Expected: "test-skip foo bar baz." ✅

Changes

• TestCoordinator.cs: When reporting skipped tests, check OverrideReason from overridden results before falling back to "Skipped"

The skip reason resolution order is now: SkipReason → OverrideReason (if overridden) → "Skipped"

Original prompt

---

This section details on the original issue you should resolve

<issue_title>OverrideResult does not reflect the specified reason even when Skipped is set</issue_title>
<issue_description>I used OverrideResult to dynamically skip tests based on certain conditions.
The test was skipped as expected, but the reason was not reflected and just shows "Skipped".

Environment:

• .NET 10.0.100
• TUnit 1.5.6

Code:

[Test]
public async Task TestSkip()
{
    TestContext.Current!.Execution.OverrideResult(TestState.Skipped, "test-skip foo bar baz.");
}

Console:

Visual Studio 2026 Test Explorer:

(Sorry for the message of images in Japanese)</issue_description>

Comments on the Issue (you are @copilot in this section)

@thomhurst Thanks I'll take a look.

As a work around, you should be able to use the static method Skip.Test(reason)</comment_new>
<comment_new>@thomhurst
Docs will be updated 😄
https://github.com/thomhurst/TUnit/pull/4018/files</comment_new>

• Fixes OverrideResult does not reflect the specified reason even when Skipped is set #4015

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[Copilot]

Pull request overview

This PR fixes a bug where calling OverrideResult(TestState.Skipped, "reason") would ignore the provided reason and display "Skipped" instead of the custom message. The fix adds logic to check the OverrideReason property when reporting skipped tests, falling back to the default "Skipped" message only if no override reason is available.

Key Changes:

• Modified skip reason resolution logic to prioritize SkipReason → OverrideReason (if overridden) → default "Skipped" message

Comments suppressed due to low confidence (1)

TUnit.Engine/Services/TestExecution/TestCoordinator.cs:307

• This foreach loop implicitly filters its target sequence - consider filtering the sequence explicitly using '.Where(...)'.

        foreach (var dependency in test.Dependencies)
        {
            if (collected.Add(dependency.Test.Context.Metadata.TestDetails))
            {
                CollectAllDependencies(dependency.Test, collected, visited);
            }
        }