fix: Refactor TestOutcome enum with updated summaries

[BenjaminMichaelis]

This pull request updates the TestOutcome enum in TrxLib/TestOutcome.cs to better align with the official outcomes defined by the Microsoft VSTest framework. The enum now includes a more comprehensive set of test result states, with revised documentation for each value.

Key changes to test outcome representation:

• Expanded the TestOutcome enum to include additional states such as Error, Timeout, Aborted, PassedButRunAborted, NotRunnable, Disconnected, Warning, Completed, and InProgress, providing more granular information about test execution results.
• Updated existing enum value names and their documentation to match the official VSTest definitions, ensuring greater compatibility and clarity.

[Copilot]

Pull request overview

This PR refactors TestOutcome to expand the set of possible outcomes and updates the XML doc summaries to more closely match the VSTest/TRX outcome vocabulary.

Changes:

• Expanded TestOutcome with additional VSTest-aligned states (e.g., Error, Aborted, Warning, Completed, InProgress).
• Updated/rewrote enum value summaries to describe the revised semantics.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

Comments suppressed due to low confidence (2)

TrxLib/TestOutcome.cs:12

• This member summary uses first-person wording ("we were trying"), which is unusual for public API docs and reads less clearly in IntelliSense. Consider rephrasing to third-person (e.g., "A system error occurred while executing the test.").

    /// <summary>
    /// There was a system error while we were trying to execute a test.
    /// </summary>
    Error = 6,

TrxLib/TestOutcome.cs:35

• The Inconclusive summary uses first-person wording ("we can't say") and an ellipsis ("...") which can be unclear in IntelliSense. Consider rewriting as a concise third-person description without ellipses.

    /// <summary>
    /// Test has completed, but we can't say if it passed or failed.
    /// May be used for aborted tests...
    /// </summary>
    Inconclusive = 3,