Skip to content

Run VSTest tests with MTP #15079

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 23 commits into from
Jul 4, 2025
Merged

Run VSTest tests with MTP #15079

merged 23 commits into from
Jul 4, 2025

Conversation

Youssef1313
Copy link
Member

@Youssef1313 Youssef1313 commented May 28, 2025

TODO:

  • Move to .NET 10 and Arcade 10 (@nohwnd is that part okay for you? - I assume moving to MTP generally is good, unless you see any technical blockers but there probably none?) - I'll follow-up in a separate PR.
  • Add dotnet.config
  • Set OutputType to Exe in test projects

@nohwnd
Copy link
Member

nohwnd commented May 29, 2025

moving to net10 should be okay. Moving to mtp should also be okay. We are using an already released version to run the tests anyway, so we are not losing "coverage" by moving to another test runner.

@Youssef1313 Youssef1313 force-pushed the dev/ygerges/mtp branch 3 times, most recently from 282d282 to c14f4ba Compare June 3, 2025 09:05
@@ -112,6 +112,7 @@ stages:
-nobl
-integrationTest
-performanceTest
/bl:$(Build.SourcesDirectory)/artifacts/TestResults/$(_BuildConfig)/Test.binlog
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: move this below the -nobl or the otherway around

@Evangelink Evangelink enabled auto-merge (squash) July 1, 2025 07:38
@Youssef1313
Copy link
Member Author

failed RunAllTestsWithMixedTFMsWillProvideAdditionalInformationToTheDebugger (Row: 19, Every runner, Runner = net8.0, TargetFramework = net8.0, InIsolation,  vstest.console = 17.6.2 [MostDownloaded],  Testhost = 17.15.0-ci [Latest],  MSTest = 3.9.3 [LatestStable]) (16s 791ms)
  Expected collection to be empty, but found at least one item {"The active Tests Run was aborted."}.
    at FluentAssertions.Execution.LateBoundTestFramework.Throw(String message)
    at FluentAssertions.Execution.AssertionChain.FailWith(Func`1 getFailureReason)
    at FluentAssertions.Execution.GivenSelector`1.FailWith(String message, Object[] args)
    at FluentAssertions.Execution.AssertionChain.WithExpectation(String message, Action`1 chain, Object[] args)
    at FluentAssertions.Collections.GenericCollectionAssertions`3.BeEmpty(String because, Object[] becauseArgs)
    at Microsoft.TestPlatform.AcceptanceTests.TranslationLayerTests.CustomTestHostTests.RunAllTestsWithMixedTFMsWillProvideAdditionalInformationToTheDebugger(RunnerInfo runnerInfo) in /_/test/Microsoft.TestPlatform.Acceptance.IntegrationTests/TranslationLayerTests/CustomTestHostTests.cs:158
    
  Standard output
    Available space for TEMP: D:\ 85151 MB
    Logging diagnostics in D:\a\_work\_temp\vstest\kTfAa\log.txt
    Console runner path: D:\a\_work\1\s\.packages\microsoft.testplatform.cli\17.6.2\contentFiles\any\netcoreapp3.1\vstest.console.dll

😕

@nohwnd
Copy link
Member

nohwnd commented Jul 2, 2025

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Evangelink Evangelink merged commit e2a4415 into main Jul 4, 2025
7 checks passed
@Evangelink Evangelink deleted the dev/ygerges/mtp branch July 4, 2025 15:24
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