astral-sh/ruff

Cleanup Ruff's CLI tests

Open

#19 016 ouverte le 28 juin 2025

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Rust (47 527 stars) (2 088 forks)batch import
help wantedtesting

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

Guide contributeur

Stack technique
rust
Domaine
clitesting
Type d'issue
refactor
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
half day
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
RustCargoCLI testing patterns
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
Examine the existing test files in crates/ruff/tests to identify repeated boilerplate. Review the referenced ty CLI test infrastructure at https://github.com/astral sh/ruff/blob/1f27d53fd576e92a7d2cd4abc1761d6ad836d85b/crates/ty/tests/cli/main.rs#L600 for structural inspiration. Extract common patterns into reusable functions or modules, and consider consulting maintainer feedback if scope is unclear.