Code Quality: Migrate from VSTest to Microsoft.Testing.Platform

[Youssef1313]

Resolved / Related Issues

To prevent extra work, all changes to the Files codebase must link to an approved issue marked as Ready to build. Please insert the issue number following the hashtag with the issue number that this Pull Request resolves.

• Closes Code Quality: Migrate tests from VSTest to Microsoft.Testing.Platform #17981

Steps used to test these changes

Stability is a top priority for Files and all changes are required to go through testing before being merged into the repo. Please include a list of steps that you used to test this PR.

1. Looking at the test run in CI confirming that all tests are still running as before. (Still waiting for workflows to be approved to run)

[marcelwgn]

Except for the global.json, LGTM

[hez2010]

Looks fine to me. Thank you!

[sentry]

Bug: The test command in the CI workflow uses a glob pattern that looks for a .dll, but the build now produces an .exe, which will cause tests to fail.
_{Severity: CRITICAL | Confidence: High}

🔍 Detailed Analysis

The CI workflow command dotnet test --test-modules $env:AUTOMATED_TESTS_ASSEMBLY_DIR\**\Files.InteractionTests.dll will fail. The project Files.InteractionTests.csproj has been changed to produce an executable (<OutputType>Exe</OutputType>), resulting in a Files.InteractionTests.exe file. The glob pattern in the test command explicitly searches for a .dll file, which will not match the .exe output. Consequently, the test runner will not find any test modules to execute, causing the test job to fail.

💡 Suggested Fix

Update the glob pattern in the dotnet test command to search for the .exe file instead of the .dll. Change --test-modules $env:AUTOMATED_TESTS_ASSEMBLY_DIR\**\Files.InteractionTests.dll to --test-modules $env:AUTOMATED_TESTS_ASSEMBLY_DIR\**\Files.InteractionTests.exe.

🤖 Prompt for AI Agent

Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.

Location: .github/workflows/ci.yml#L240-L241

Potential issue: The CI workflow command `dotnet test --test-modules
$env:AUTOMATED_TESTS_ASSEMBLY_DIR\**\Files.InteractionTests.dll` will fail. The project
`Files.InteractionTests.csproj` has been changed to produce an executable
(`<OutputType>Exe</OutputType>`), resulting in a `Files.InteractionTests.exe` file. The
glob pattern in the test command explicitly searches for a `.dll` file, which will not
match the `.exe` output. Consequently, the test runner will not find any test modules to
execute, causing the test job to fail.

_{Did we get this right? 👍 / 👎 to inform future reviews.
Reference ID: 8199804}

[yaira2]

@Youssef1313 it looks like the ci pipeline is failing.