astral-sh/ruff

Cleanup Ruff's CLI tests

Open

#19,016 opened on Jun 28, 2025

View on GitHubContributor guide
 (2 comments) (0 reactions) (0 assignees)Rust (2,088 forks)batch import
help wantedtesting

Repository metrics

Stars
 (47,527 stars)
PR merge metrics
 (PR metrics pending)

Description

Ruff's CLI integration tests (crates/ruff/tests) contain a lot of shared boilerplate code (shared between test functions but also test files). We should try to extract those into a more reusable testing infra. I think ty's CLI test infra works pretty well and we should try if we can use the same structure for ruff.

https://github.com/astral-sh/ruff/blob/1f27d53fd576e92a7d2cd4abc1761d6ad836d85b/crates/ty/tests/cli/main.rs#L600

Contributor guide

Research direction
Study the existing CLI test boilerplate in crates/ruff/tests, identify common patterns, and design a reusable test infrastructure inspired by ty's CLI test infra at https://github.com/astral sh/ruff/blob/1f27d53fd576e92a7d2cd4abc1761d6ad836d85b/crates/ty/tests/cli/main.rs#L600. Implement the changes and ensure all tests still pass.
Tech stack
rust
Domain
cli
Issue type
Refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
RustGit
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
75