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

tests: run tests under a minimal debugger that can create minidumps #10072

Merged
merged 1 commit into from
Jun 21, 2024

Conversation

madebr
Copy link
Contributor

@madebr madebr commented Jun 20, 2024

Tests on ci are run using this dumper, and will upload the minidumps.

This (currently) only supports win32 Windows, and should hopefully help us with finding the cause for #8576

Description

Existing Issue(s)

Tests on ci are run using this dumper, and will upload the minidumps.
@madebr madebr force-pushed the run-tests-under-sdlprocdump branch from bcab4ae to 657abfa Compare June 21, 2024 10:24
@madebr madebr merged commit 0f27686 into libsdl-org:main Jun 21, 2024
39 checks passed
@madebr madebr deleted the run-tests-under-sdlprocdump branch June 21, 2024 10:39
@slouken
Copy link
Collaborator

slouken commented Jun 21, 2024

Great, do we have dumps for the testatomic crash now?

@madebr
Copy link
Contributor Author

madebr commented Jun 21, 2024

Not yet.
When it actually crashes, then hopefully this debugger will create a dump.

I say "hopefully" because I'm not 100% sure the exception of the testatomic crashes will trigger a dump.

@madebr
Copy link
Contributor Author

madebr commented Jun 21, 2024

Additionally, I don't know what influence the (single-threaded) debugger has on multi-threading.
It's possible the debugger will cause the bug to never appear (a Heisenbug).

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