gfx-rs/wgpu

Enable `clippy::tests_outside_test_module` lint

Open

#6.106 geöffnet am 12. Aug. 2024

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (703 Forks)batch import
area: infrastructuregood first issuetype: enhancement

Repository-Metriken

Stars
 (9.290 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 10T 7h) (61 gemergte PRs in 30 T)

Beschreibung

There's a whole bunch of tests that aren't in test modules. This should be solved once and for all using this clippy lint https://rust-lang.github.io/rust-clippy/master/index.html#/tests_outside_test_module

Not doing so means that the tests will be unnecessarily compiled as part of non test builds

Contributor Guide

Research-Richtung
Aktiviere den Lint `clippy::tests outside test module` in der Clippy Konfiguration des Workspaces (wahrscheinlich in `Cargo.toml` oder `.cargo/config.toml`). Führe dann `cargo clippy all targets` aus, um alle Testfunktionen zu identifizieren, die sich derzeit außerhalb eines `#[cfg(test)]` Moduls befinden. Verschiebe jeden identifizierten Test in ein geeignetes Testmodul innerhalb seiner Datei. Stelle sicher, dass die Tests mit `cargo test` weiterhin bestehen. Verknüpfte PRs oder Maintainer Kommentare deuten auf keine Blockaden hin. Das Issue ist offen und hat keine Zuweisungen, daher kann ein Neuling es übernehmen.
Tech Stack
rust
Domain
testingdeveloper experience
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-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Rust basicscargo clippy familiarity
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50