Skip to content
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

Enable crash dumps for uwp/wasdk unit tests #493

Open
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

Arlodotexe
Copy link
Member

@Arlodotexe Arlodotexe commented Aug 26, 2024

This PR updates our CI to emit crash dumps and diagnostic logs, which should be uploaded as artifacts.

Needed to diagnose #347

See also https://github.com/microsoft/vstest/blob/main/docs/troubleshooting.md

@Arlodotexe Arlodotexe force-pushed the tests/diagnostics branch 2 times, most recently from 2d701d9 to 80e9b96 Compare August 26, 2024 20:59
@Arlodotexe
Copy link
Member Author

Looks like cancelling and re-running all jobs isn't re-running the successful ones. Closing and re-opening to fix.

@Arlodotexe Arlodotexe closed this Aug 26, 2024
@Arlodotexe Arlodotexe reopened this Aug 26, 2024
@Arlodotexe Arlodotexe closed this Aug 26, 2024
@Arlodotexe Arlodotexe reopened this Aug 26, 2024
@Arlodotexe
Copy link
Member Author

Ran a few times, and even when failing during various tests, the dump seems to capture an "invalid pointer read" in the dispatcher queue extensions:

image

It's not immediately obvious why this is happening. I wonder if it's only happening for tests generated with UIThreadTestMethod? Maybe that's related.

@Arlodotexe
Copy link
Member Author

Arlodotexe commented Nov 11, 2024

Strange, it's not outputting crash dumps anymore 🤔

I'm seeing this:

Terminating app with process ID '7764'.
Data collector 'Blame' message: All tests finished running, Sequence file will not be generated.

Will continue to retry, it's not clear why the tests all finish but fail anyway.

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