[Feature]: MSTest `WorkerAwareTest.TestContext` should be virtual

[MattKotsenas]

🚀 Feature Request

In the MSTest harness, WorkerAwareTest has a TestContext property, see

playwright-dotnet/src/Playwright.MSTest/WorkerAwareTest.cs

Line 13 in a878a63

public TestContext TestContext { get; set; } = null!;

. This property should be marked virtual so that it can be overridden by inheritors as necessary.

Example

No response

Motivation

Verify relies on hooking into TestContext to provide snapshot testing. It does this with a source generator to override the TestContext property and storing the property in its own AsyncLocal during set.

However, because WorkerAwareTest.TestContext isn't marked virtual, the source generator isn't able to add the setter hook. See VerifyTests/Verify#1482 for the bug on the Verify side as well.