Skip to content

[WIP] Use dotnet vstest runner instead of xunit.console#3333

Closed
ViktorHofer wants to merge 1 commit intodotnet:masterfrom
ViktorHofer:VSTestRunner
Closed

[WIP] Use dotnet vstest runner instead of xunit.console#3333
ViktorHofer wants to merge 1 commit intodotnet:masterfrom
ViktorHofer:VSTestRunner

Conversation

@ViktorHofer
Copy link
Member

@ViktorHofer ViktorHofer commented Jul 21, 2019

Switching from the unsupported xunit.console to dotnet test.

I had to bring the major version roll-forward policy back as the 3.0 SDK expects a 3.0 runtime to be present in the testhost folder. I don't know how to work around that until we have a 5.0 SDK and how to prevent that in future when we again switch the major version. Maybe we should coordinate the SDK branching with corefx/coreclr/core-setup more closely going forward.

Keeping the WIP marker until I validated all verticals and configurations in corefx. The consuming corefx branch to be cloned locally can be found here: https://github.com/ViktorHofer/corefx/tree/DotNetTest

cc @ericstj @safern @danmosemsft

@ViktorHofer ViktorHofer self-assigned this Jul 21, 2019
@ViktorHofer ViktorHofer changed the title [WIP] Use dotnet vstest runner instead of xunit.console Use dotnet vstest runner instead of xunit.console Jul 24, 2019
@ViktorHofer ViktorHofer changed the title Use dotnet vstest runner instead of xunit.console [WIP] Use dotnet vstest runner instead of xunit.console Aug 11, 2019
@ViktorHofer
Copy link
Member Author

ViktorHofer commented Aug 11, 2019

Remaining work:

  • Fix NetFx leg
  • Remove duplicate test cases in corefx
  • Make sure that code coverage works with dotnet test

@ViktorHofer ViktorHofer force-pushed the VSTestRunner branch 4 times, most recently from d95aadd to f101b40 Compare October 17, 2019 14:48
@tmat
Copy link
Member

tmat commented Oct 17, 2019

FYI: Not relevant for CoreFX, but definitely blocking Roslyn and other repos from using vstest: https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1002535/

@ViktorHofer
Copy link
Member Author

Thanks for letting me know :)

@ViktorHofer
Copy link
Member Author

Ported to runtime: dotnet/runtime@8790252

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.

2 participants