Skip to content

Regression: panic messages about invalid mdtests are now swallowed by mdtest #15317

@AlexWaygood

Description

@AlexWaygood

Following #15241, some of the internal error messages from red_knot_test are now swallowed by the framework, for example this one here:

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:

image

If I revert 75015b0 on that PR branch, I get this instead, which is much more helpful:

image

Metadata

Metadata

Assignees

Labels

testingRelated to testing Ruff itselftyMulti-file analysis & type inference

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions