-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Closed
Closed
Copy link
Labels
testingRelated to testing Ruff itselfRelated to testing Ruff itselftyMulti-file analysis & type inferenceMulti-file analysis & type inference
Description
Following #15241, some of the internal error messages from red_knot_test are now swallowed by the framework, for example this one here:
ruff/crates/red_knot_test/src/lib.rs
Lines 32 to 37 in b264489
| let suite = match test_parser::parse(short_title, &source) { | |
| Ok(suite) => suite, | |
| Err(err) => { | |
| panic!("Error parsing `{path}`: {err:?}") | |
| } | |
| }; |
On a PR branch I was working on, I got this (not very informative!) test failure:
If I revert 75015b0 on that PR branch, I get this instead, which is much more helpful:
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
testingRelated to testing Ruff itselfRelated to testing Ruff itselftyMulti-file analysis & type inferenceMulti-file analysis & type inference

