Fix OverrideResult skip reason not being displayed (#4264)

* Initial plan

* Fix OverrideResult to reflect skip reason when Skipped is set

When OverrideResult is called with TestState.Skipped and a reason,
the reason was not being reflected in test output. The code was only
checking test.Context.SkipReason but not the OverrideReason from the
overridden result.

Updated TestCoordinator to check OverrideReason when the result is
overridden to Skipped.

Co-authored-by: thomhurst <30480171+thomhurst@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: thomhurst <30480171+thomhurst@users.noreply.github.com>

Author: Copilot

TUnit.Engine/Services/TestExecution/TestCoordinator.cs

@@ -276,7 +276,10 @@ await _eventReceiverOrchestrator.InvokeLastTestInSessionEventReceiversAsync(
                     await _messageBus.Failed(test.Context, test.Context.Execution.Result?.Exception!, test.StartTime.GetValueOrDefault()).ConfigureAwait(false);
                     break;
                 case TestState.Skipped:
-                    await _messageBus.Skipped(test.Context, test.Context.SkipReason ?? "Skipped").ConfigureAwait(false);
+                    var skipReason = test.Context.SkipReason
+                                     ?? (test.Context.Execution.Result?.IsOverridden == true ? test.Context.Execution.Result.OverrideReason : null)
+                                     ?? "Skipped";
+                    await _messageBus.Skipped(test.Context, skipReason).ConfigureAwait(false);
                     break;
                 case TestState.Cancelled:
                     await _messageBus.Cancelled(test.Context, test.StartTime.GetValueOrDefault()).ConfigureAwait(false);