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

Support panic backtraces, add log buffer #48

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

untitaker
Copy link

@untitaker untitaker commented Nov 1, 2024

When a test fails there is no way to use RUST_BACKTRACE=1 to show a
backtrace. This patch fixes the issue.

Additionally, it adds an explicit log buffer that can be used to work
around #9. That's a feature that the std harness doesn't have, but I
think it's appropriate to add since it's a workaround for something
libtest-mimic doesn't but should have (stdout/err capturing)

When a test fails there is no way to use RUST_BACKTRACE=1 to show a
backtrace. This patch fixes the issue.

Additionally, it adds an explicit log buffer that can be used to work
around LukasKalbertodt#9. That's a feature that the std harness doesn't have, but I
think it's appropriate to add since it's a workaround for something
libtest-mimic _doesn't_ but should have.
@untitaker
Copy link
Author

it seems this codebase doesn't pass through cargo fmt without changes (forgot to check in custom fmt config?)

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.

1 participant