gfx-rs/wgpu

Enable `clippy::tests_outside_test_module` lint

Open

#6106 aperta il 12 ago 2024

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)Rust (703 fork)batch import
area: infrastructuregood first issuetype: enhancement

Metriche repository

Star
 (9290 star)
Metriche merge PR
 (Merge medio 10g 7h) (61 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Abilita il lint `clippy::tests outside test module` nella configurazione clippy dell'area di lavoro (probabilmente in `Cargo.toml` o `.cargo/config.toml`). Quindi esegui `cargo clippy all targets` per identificare tutte le funzioni di test attualmente al di fuori di un modulo `#[cfg(test)]`. Sposta ogni test identificato in un modulo di test appropriato all'interno del suo file. Assicurati che i test passino ancora con `cargo test`. Nessun PR collegato o commento del maintainer indica blocchi. L'issue è aperta senza assegnatari, quindi un nuovo arrivato può prenderla in carico.
Tech stack
rust
Dominio
testingdeveloper experience
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Rust basicscargo clippy familiarity
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50