Skip to content

Migrate from VSTest to MTP#124

Merged
elantiguamsft merged 8 commits intomicrosoft:mainfrom
Youssef1313:mtp
Jul 16, 2025
Merged

Migrate from VSTest to MTP#124
elantiguamsft merged 8 commits intomicrosoft:mainfrom
Youssef1313:mtp

Conversation

@Youssef1313
Copy link
Copy Markdown
Member

@Youssef1313 Youssef1313 commented Mar 2, 2025

This migrates from VSTest, to the newer Microsoft.Testing.Platform.

ValidateIndirectSucceedsWithRootPassedIn is failing in this PR, but it's also failing on current main when I try locally. Looks like the test involves some certificate that expired on 28/02/2025.

@Youssef1313
Copy link
Copy Markdown
Member Author

@elantiguamsft Can you please review this PR? Thank you!

Signed-off-by: Youssef Victor <youssefvictor00@gmail.com>
Comment thread CoseHandler.Tests/CoseHandler.Tests.csproj Outdated
Comment thread CoseIndirectSignature.Tests/CoseIndirectSignature.Tests.csproj Outdated
Comment thread CoseSign1.Certificates.Tests/CoseSign1.Certificates.Tests.csproj Outdated
Comment thread CoseSign1.Headers.Tests/CoseSign1.Headers.Tests.csproj Outdated
Comment thread CoseSign1.Tests/CoseSign1.Tests.csproj Outdated
Comment thread CoseSignTool.Tests/CoseSignTool.Tests.csproj Outdated
Signed-off-by: Youssef Victor <youssefvictor00@gmail.com>
Signed-off-by: Youssef Victor <youssefvictor00@gmail.com>
Comment thread CoseHandler.Tests/CoseSignValidateTests.cs Outdated
Signed-off-by: Youssef Victor <youssefvictor00@gmail.com>
@Youssef1313
Copy link
Copy Markdown
Member Author

@elantiguamsft I think the current failure is unrelated to my changes.

error: pathspec 'mtp' did not match any file(s) known to git

The GitHub workflow seems to not be working for PRs from forks. Could we get this merged? Thanks!

@elantiguamsft elantiguamsft merged commit 7f40677 into microsoft:main Jul 16, 2025
14 of 16 checks passed
@Youssef1313 Youssef1313 deleted the mtp branch July 16, 2025 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants