Skip to content

Commit

Permalink
[red-knot] Disable linter-corpus tests (astral-sh#14391)
Browse files Browse the repository at this point in the history
## Summary

Disable the no-panic tests for the linter corpus, as there are too many
problems right now, requiring linter-contributors to add their test
files to the allow-list.

We can still run the tests using `cargo test -p red_knot_workspace --
--ignored linter_af linter_gz`. This is also why I left the
`crates/ruff_linter/` entries in the allow list for now, even if they
will get out of sync. But let me know if I should rather remove them.
  • Loading branch information
sharkdp authored Nov 16, 2024
1 parent 1fbed6c commit d470f29
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions crates/red_knot_workspace/tests/check.rs
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ fn parser_no_panic() -> anyhow::Result<()> {
}

#[test]
#[ignore = "Enable running once there are fewer failures"]
fn linter_af_no_panic() -> anyhow::Result<()> {
let workspace_root = get_workspace_root()?;
run_corpus_tests(&format!(
Expand All @@ -50,6 +51,7 @@ fn linter_af_no_panic() -> anyhow::Result<()> {
}

#[test]
#[ignore = "Enable running once there are fewer failures"]
fn linter_gz_no_panic() -> anyhow::Result<()> {
let workspace_root = get_workspace_root()?;
run_corpus_tests(&format!(
Expand Down

0 comments on commit d470f29

Please sign in to comment.