Skip to content

Run host tests with dotnet test and enable crash/hang dumps - #69340

Merged
elinor-fung merged 5 commits into
dotnet:mainfrom
elinor-fung:hostTestDump
May 19, 2022
Merged

elinor-fung merged 5 commits into
dotnet:mainfrom
elinor-fung:hostTestDump

Conversation

@elinor-fung

@elinor-fung elinor-fung commented May 14, 2022 •

Copy link
Copy Markdown
Member
  • Switch host tests to run using dotnet test
  • Enable blame sequence, crash dumps, and hang dumps
    • These are uploaded during publish of test results as an attachment to the test run - example
  • Upload host binaries/symbols on failure so that they can be used to investigate any dumps - example

Related: #69114

@ghost ghost added the area-Host label May 14, 2022
@ghost ghost assigned elinor-fung May 14, 2022
@ghost

ghost commented May 14, 2022

Copy link
Copy Markdown

Tagging subscribers to this area: @vitek-karas, @agocke, @VSadov
See info in area-owners.md if you want to be subscribed.

Issue Details

null

Author: elinor-fung
Assignees: elinor-fung
Labels:

area-Host

Milestone: -

@elinor-fung
elinor-fung force-pushed the hostTestDump branch 6 times, most recently from 45cabf4 to d761328 Compare May 16, 2022 18:36
@elinor-fung
elinor-fung force-pushed the hostTestDump branch 3 times, most recently from 7d303ef to cc8966e Compare May 17, 2022 03:32
@elinor-fung
elinor-fung marked this pull request as ready for review May 18, 2022 22:10
@elinor-fung
elinor-fung requested review from a team and vitek-karas May 18, 2022 22:12
@elinor-fung
elinor-fung merged commit 55090e6 into dotnet:main May 19, 2022
@elinor-fung
elinor-fung deleted the hostTestDump branch May 19, 2022 02:02
@ViktorHofer

Copy link
Copy Markdown
Member

As I didn't see a change of the actual test runner invocation, does this use Arcade's dotnet test support?

@elinor-fung

Copy link
Copy Markdown
Member Author

Yes, it is using Arcade's support (via setting UseVSTestRunner).

@ghost ghost locked as resolved and limited conversation to collaborators Jun 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants