astral-sh/ruff

Cleanup Ruff's CLI tests

Offen

#19.016 geöffnet am 28.06.2025

Auf GitHub ansehenContributor Guide
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (2.088 Forks)batch import
help wantedtesting

Repository-Metriken

Stars
 (47.527 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 3T 8h) (573 gemergte PRs in 30 T)

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

Research-Richtung
Studiere den vorhandenen CLI Test Boilerplate in crates/ruff/tests, identifiziere gemeinsame Muster und entwerfe eine wiederverwendbare Testinfrastruktur, inspiriert von tys CLI Testinfrastruktur (https://github.com/astral sh/ruff/blob/1f27d53fd576e92a7d2cd4abc1761d6ad836d85b/crates/ty/tests/cli/main.rs#L600). Implementiere die Änderungen und stelle sicher, dass alle Tests weiterhin bestanden werden.
Tech Stack
rust
Domain
cli
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
RustGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75