-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
Port backtrace-rs test-crates to rustc repo #122899
Comments
I'll take a stab at porting line-tables-only |
The tests using C may be able to make use of |
Hi, I'll pick this up cpp_smoke_test |
… r=workingjubilee Port backtrace dylib-dep test to a ui test Original test: [dylib-dep](https://github.com/rust-lang/backtrace-rs/tree/6fa4b85b9962c3e1be8c2e5cc605cd078134152b/crates/dylib-dep) Part of rust-lang#122899.
I don't think |
@jieyouxu I think I'll have to reimagine the test somewhat and port it myself, yeah! it can be run under the rustc test suite but it requires doing something mildly silly because what it is really doing is testing that you "don't lose track" of the debuginfo. |
… r=workingjubilee Port backtrace dylib-dep test to a ui test Original test: [dylib-dep](https://github.com/rust-lang/backtrace-rs/tree/6fa4b85b9962c3e1be8c2e5cc605cd078134152b/crates/dylib-dep) Part of rust-lang#122899.
The |
…=workingjubilee Port backtrace dylib-dep test to a ui test Original test: [dylib-dep](https://github.com/rust-lang/backtrace-rs/tree/6fa4b85b9962c3e1be8c2e5cc605cd078134152b/crates/dylib-dep) Part of rust-lang#122899.
That test is also unfortunately a bit awkward because it's checking if each frame with a non-null IP has resolved symbols even if |
We could add more methods to |
…only, r=workingjubilee Port backtrace's `line-tables-only` test over to rustc Part of rust-lang#122899.
…only, r=workingjubilee Port backtrace's `line-tables-only` test over to rustc Part of rust-lang#122899.
backtrace-rs has a number of subcrates which it uses to test more complicated logic that is harder to observe from inside a program, leading to this exciting yaml: https://github.com/rust-lang/backtrace-rs/blob/6fa4b85b9962c3e1be8c2e5cc605cd078134152b/.github/workflows/main.yml#L54-L69
while tests inside backtrace's library itself will be run as part of the stdlib's test suite, these are not. backtrace-rs failed to run nightly testing for Windows, which would have caught this. however, many of these tests are directly relevant to rustc's functionality as well, and changes should not be landed in rustc which break them without very good reason. failing to include these tests led to a severe regression in backtraces on Windows. these tests should be ported to run inside rustc's existing test infrastructure where they are relevant. I have excepted one which was already based on a rustc test and one which is only relevant for backtrace-rs.
if you would like to work on porting one of these tests, please link to this issue in your PR and leave a comment to claim the test. if you are stuck, please don't hesitate to open a thread on the rust zulip! and be aware, some may be complex enough to need the new recipes infrastructure, but hopefully not. I expect all may need, however, this compiletest annotation:
//@ needs-unwind
The text was updated successfully, but these errors were encountered: