GitHubActionsTestLogger icon indicating copy to clipboard operation
GitHubActionsTestLogger copied to clipboard

Migrate from `vstest` to `Microsoft.Testing.Platform`

Open Evangelink opened this issue 10 months ago • 4 comments

First draft at changing the codebase to be an MTP GH reporter

Closes #41

Evangelink avatar Mar 07 '25 21:03 Evangelink

Would be nice if we could also update the Demo project to use MTP as well 🙏🏻

Tyrrrz avatar Mar 09 '25 13:03 Tyrrrz

Hehe, I was actually working on the code to support both at the same time 😁

nohwnd avatar Mar 10 '25 09:03 nohwnd

Because I missed this comment, saying that you are okay to migrate:

https://github.com/Tyrrrz/GitHubActionsTestLogger/issues/41#issuecomment-2669057401

fwiw code is here (the MTP part is heavily not done) https://github.com/Tyrrrz/GitHubActionsTestLogger/pull/45 feel free to close.

nohwnd avatar Mar 10 '25 09:03 nohwnd

Blocked by https://github.com/microsoft/testfx/issues/5265

I'll address the other comments and do some ccleanup this week,

Evangelink avatar Mar 19 '25 08:03 Evangelink

This should be unblocked now: https://github.com/microsoft/testfx/issues/5265#issuecomment-3124235442

martincostello avatar Jul 27 '25 08:07 martincostello

@Evangelink do you think we can get this moving again?

Tyrrrz avatar Sep 10 '25 18:09 Tyrrrz

An app where I'm already using .NET 10 RC.1 has been updated to MSTest.Sdk 4.0.0, and now I'm starting to see messages like this:

/home/runner/.nuget/packages/microsoft.testing.platform.msbuild/2.0.0/buildMu
ltiTargeting/Microsoft.Testing.Platform.MSBuild.targets(263,5): error : Testing 
with VSTest target is no longer supported by Microsoft.Testing.Platform on .NET 
10 SDK and later. If you use dotnet test, you should opt-in to the new dotnet 
test experience. For more information, see https://aka.ms/dotnet-test-mtp-error 

It would be unfortunate if I'm forced to drop the GitHub Actions test logging because I have to choose between .NET 10 or VSTest support.

martincostello avatar Oct 08 '25 16:10 martincostello

@martincostello hm, that's harsh, I was hoping for a softer migration process.

I'll take a look and try to restart this PR myself 👍🏻

Tyrrrz avatar Oct 11 '25 13:10 Tyrrrz

I have merged the code into a separate branch so I can continue working on it. Thanks @Evangelink and everyone else for the effort thus far!

Tyrrrz avatar Oct 11 '25 17:10 Tyrrrz