rust-lang/rust-clippy

Extensions to should_assert_eq

Open

#1810 aperta il 3 giu 2017

Vedi su GitHub
 (1 commento) (1 reazione) (1 assegnatario)Rust (1391 fork)batch import
C-enhancementgood first issue

Metriche repository

Star
 (10.406 star)
Metriche merge PR
 (Merge medio 19g 22h) (113 PR mergiate in 30 g)

Descrizione

I found this in the standard library:

assert!(o == owned && b == borrowed)

This should be a case for should_assert_eq. Additionally, it might be nice to have a lint that suggests splitting asserts with && to multiple assert statements, for better debugging.

Guida contributor

Direzione di ricerca
Studia il lint `should assert eq` esistente nel codice sorgente di Clippy per comprenderne il pattern matching. Poi modificalo per gestire l'AND logico all'interno delle macro assert. Inoltre, ricerca come creare un nuovo lint che suggerisca di dividere le asserzioni con `&&` in più asserzioni per un debugging migliore.
Tech stack
rust
Dominio
testing
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
RustGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65