Fix test property lifecycle scope

[Amaury Levé (Evangelink)]

Summary

• keep [TestProperty], test categories, and host-provided TCM properties scoped to the individual test lifecycle
• seed assembly/class initialization and cleanup contexts only with source/run-level properties
• preserve downward flow of properties written by AssemblyInitialize and ClassInitialize
• cover constructor, test initialize, test body, test cleanup, dispose, class cleanup, assembly cleanup, and sibling-test isolation

Validation

• MSTestAdapter.PlatformServices.UnitTests: 925 passed
• TestContextPropertyFlowTests: 6 passed across net462, net8.0, and net10.0
• Debug package build: 0 warnings, 0 errors

Fixes #10041

[Copilot]

Pull request overview

Separates per-test metadata from assembly/class lifecycle properties, preventing property leakage between sibling tests while preserving initializer property flow.

Changes:

• Adds distinct per-test and lifecycle property dictionaries.
• Uses lifecycle-only properties for assembly/class initialization and cleanup.
• Adds unit and acceptance regression coverage across the complete test lifecycle.

Show a summary per file

File	Description
TestExecutionManagerTests.cs	Tests TCM property isolation across lifecycle contexts and sibling tests.
TestContextPropertyFlowTests.cs	Adds end-to-end property-scope coverage.
InternalAPI.Unshipped.txt	Tracks the new internal runner overloads.
UnitTestRunner.TestFilter.cs	Uses lifecycle properties during filtered-test cleanup.
UnitTestRunner.RunSingleTest.cs	Separates test and lifecycle property inputs.
TestExecutionManager.Runner.cs	Seeds and passes source-only lifecycle properties.

Review details

• Files reviewed: 6/6 changed files
• Comments generated: 0
• Review effort level: Medium

[github-actions]

🧪 Test quality grade — PR #10080

Grade	Test	Notes
A (90–100)	mod TestContextPropertyFlowTests. PropertiesSetInAssemblyInit AndClassInitAreVisibleEverywhere	Updated expected summary count from 5 to 7; clean AAA with exit-code + summary assertions. No issues found.
A (90–100)	new TestContextPropertyFlowTests. PerTestPropertiesAreScopedToTheirTest	Focused integration test with two meaningful assertions; single behavior under test. No issues found.
A (90–100)	new TestExecutionManagerTests. RunTestsShouldKeepTcmPropertiesOut OfClassLifecycleAndSiblingTestContexts	Clear AAA; AwesomeAssertions equality check verifies both sibling tests passed with correct scoping. No issues found.

_{This advisory comment was generated automatically. Grades are heuristic
and informational — they do not block merging. Re-run with
/grade-tests.}

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

• awmgmcpg

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "awmgmcpg"

See Network Configuration for more information.

🤖 Automated content by GitHub Copilot. Generated by the Grade Tests on PR (on open / sync) workflow. · 49.9 AIC · ⌖ 5.74 AIC · ⊞ 8.9K · [◷]( · ◷)

[Amaury Levé (Evangelink)]

/backport to rel/4.3

[github-actions]

Started backporting to rel/4.3: https://github.com/microsoft/testfx/actions/runs/29831531182

[github-actions]

Amaury Levé (@Evangelink) backporting to rel/4.3 failed, the patch most likely resulted in conflicts:

$ git am --3way --empty=keep --ignore-whitespace --keep-non-patch changes.patch

Applying: Fix test property lifecycle scope
Using index info to reconstruct a base tree...
M	src/Adapter/MSTestAdapter.PlatformServices/Execution/TestExecutionManager.Runner.cs
M	src/Adapter/MSTestAdapter.PlatformServices/Execution/UnitTestRunner.RunSingleTest.cs
M	src/Adapter/MSTestAdapter.PlatformServices/Execution/UnitTestRunner.TestFilter.cs
A	src/Adapter/MSTestAdapter.PlatformServices/InternalAPI/InternalAPI.Unshipped.txt
M	test/UnitTests/MSTestAdapter.PlatformServices.UnitTests/Execution/TestExecutionManagerTests.cs
Falling back to patching base and 3-way merge...
Auto-merging src/Adapter/MSTestAdapter.PlatformServices/Execution/TestExecutionManager.Runner.cs
CONFLICT (content): Merge conflict in src/Adapter/MSTestAdapter.PlatformServices/Execution/TestExecutionManager.Runner.cs
Auto-merging src/Adapter/MSTestAdapter.PlatformServices/Execution/UnitTestRunner.RunSingleTest.cs
CONFLICT (content): Merge conflict in src/Adapter/MSTestAdapter.PlatformServices/Execution/UnitTestRunner.RunSingleTest.cs
Auto-merging src/Adapter/MSTestAdapter.PlatformServices/Execution/UnitTestRunner.TestFilter.cs
CONFLICT (content): Merge conflict in src/Adapter/MSTestAdapter.PlatformServices/Execution/UnitTestRunner.TestFilter.cs
CONFLICT (modify/delete): src/Adapter/MSTestAdapter.PlatformServices/InternalAPI/InternalAPI.Unshipped.txt deleted in HEAD and modified in Fix test property lifecycle scope.  Version Fix test property lifecycle scope of src/Adapter/MSTestAdapter.PlatformServices/InternalAPI/InternalAPI.Unshipped.txt left in tree.
Auto-merging test/UnitTests/MSTestAdapter.PlatformServices.UnitTests/Execution/TestExecutionManagerTests.cs
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 Fix test property lifecycle scope
Error: The process '/usr/bin/git' failed with exit code 128

NOTE: A PR will be created, but needs to be revised manually!