astral-sh/ruff

Cleanup Ruff's CLI tests

Open

#19.016 geöffnet am 28. Juni 2025

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (47.527 Stars) (2.088 Forks)batch import
help wantedtesting

Beschreibung

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

Tech Stack
rust
Domain
clitesting
Issue Type
refactor
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
half day
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
fresh
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
RustCargoCLI testing patterns
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70
Research-Richtung
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.